[PATCH] isar-image-base: purge installation logs of installed packages

[PATCH] isar-image-base: purge installation logs of installed packages

[Christian Storm]

Signed-off-by: Christian Storm <christian.storm@siemens.com>
---
 meta-isar/recipes-core/images/files/debian-configscript.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta-isar/recipes-core/images/files/debian-configscript.sh b/meta-isar/recipes-core/images/files/debian-configscript.sh
index 33b2b53..b0e3f6d 100755
--- a/meta-isar/recipes-core/images/files/debian-configscript.sh
+++ b/meta-isar/recipes-core/images/files/debian-configscript.sh
@@ -76,6 +76,9 @@ fi
 localepurge
 apt-get clean
 
+# Purge installation logs of installed packages
+rm -f /var/log/dpkg.log /var/log/alternatives.log
+
 # Undo setup script changes
 if [ -x "$TARGET/sbin/start-stop-daemon.REAL" ]; then
     mv -f $TARGET/sbin/start-stop-daemon.REAL $TARGET/sbin/start-stop-daemon
-- 
2.15.0

Re: [PATCH] isar-image-base: purge installation logs of installed packages

[Alexander Smirnov]

Hi,

What is the benefit of this patch? These logs take about 40KB for 
default build.

Alex

> Signed-off-by: Christian Storm <christian.storm@siemens.com>
> ---
>   meta-isar/recipes-core/images/files/debian-configscript.sh | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/meta-isar/recipes-core/images/files/debian-configscript.sh b/meta-isar/recipes-core/images/files/debian-configscript.sh
> index 33b2b53..b0e3f6d 100755
> --- a/meta-isar/recipes-core/images/files/debian-configscript.sh
> +++ b/meta-isar/recipes-core/images/files/debian-configscript.sh
> @@ -76,6 +76,9 @@ fi
>   localepurge
>   apt-get clean
>   
> +# Purge installation logs of installed packages
> +rm -f /var/log/dpkg.log /var/log/alternatives.log
> +
>   # Undo setup script changes
>   if [ -x "$TARGET/sbin/start-stop-daemon.REAL" ]; then
>       mv -f $TARGET/sbin/start-stop-daemon.REAL $TARGET/sbin/start-stop-daemon
>

Re: [PATCH] isar-image-base: purge installation logs of installed packages

[Christian Storm]

> What is the benefit of this patch? These logs take about 40KB for 
> default build.

Yes, but it sums up :)
Joking aside, I think this information is not necessary in the image and
can hence be deleted to save space, even if not much.
That said, you may also feel free to use this patch for the
isar-image-tiny [1] if not for the default "fat" one...


Kind regards,
   Christian

[1] https://github.com/ilbers/isar/issues/24

-- 
Dr. Christian Storm
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Otto-Hahn-Ring 6, 81739 M�nchen, Germany

Re: [PATCH] isar-image-base: purge installation logs of installed packages

[Henning Schild]

It does not matter how much space we can save with individual
directories and we might as well start the discussion on 40k. I think
we will need a cleanup script to run after multistrap, or (better) as a
hook to the last package we install.

That mechanism needs to be customizable from a layer and it
needs to be optional. So the configscript is the wrong place.

Henning

Am Fri, 10 Nov 2017 11:28:08 +0100
schrieb "[ext] Christian Storm" <christian.storm@siemens.com>:

> Signed-off-by: Christian Storm <christian.storm@siemens.com>
> ---
>  meta-isar/recipes-core/images/files/debian-configscript.sh | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git
> a/meta-isar/recipes-core/images/files/debian-configscript.sh
> b/meta-isar/recipes-core/images/files/debian-configscript.sh index
> 33b2b53..b0e3f6d 100755 ---
> a/meta-isar/recipes-core/images/files/debian-configscript.sh +++
> b/meta-isar/recipes-core/images/files/debian-configscript.sh @@ -76,6
> +76,9 @@ fi localepurge apt-get clean
>  
> +# Purge installation logs of installed packages
> +rm -f /var/log/dpkg.log /var/log/alternatives.log
> +
>  # Undo setup script changes
>  if [ -x "$TARGET/sbin/start-stop-daemon.REAL" ]; then
>      mv -f $TARGET/sbin/start-stop-daemon.REAL
> $TARGET/sbin/start-stop-daemon

Re: [PATCH] isar-image-base: purge installation logs of installed packages

[Jan Kiszka]

On 2017-11-20 16:59, [ext] Henning Schild wrote:
> It does not matter how much space we can save with individual
> directories and we might as well start the discussion on 40k. I think
> we will need a cleanup script to run after multistrap, or (better) as a
> hook to the last package we install.
> 
> That mechanism needs to be customizable from a layer and it
> needs to be optional. So the configscript is the wrong place.

...and modularized so that you can select something like "remove package
data base" on a per project basis and without having to write and
maintain the rules for that yourself.

Jan

> 
> Henning
> 
> Am Fri, 10 Nov 2017 11:28:08 +0100
> schrieb "[ext] Christian Storm" <christian.storm@siemens.com>:
> 
>> Signed-off-by: Christian Storm <christian.storm@siemens.com>
>> ---
>>  meta-isar/recipes-core/images/files/debian-configscript.sh | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git
>> a/meta-isar/recipes-core/images/files/debian-configscript.sh
>> b/meta-isar/recipes-core/images/files/debian-configscript.sh index
>> 33b2b53..b0e3f6d 100755 ---
>> a/meta-isar/recipes-core/images/files/debian-configscript.sh +++
>> b/meta-isar/recipes-core/images/files/debian-configscript.sh @@ -76,6
>> +76,9 @@ fi localepurge apt-get clean
>>  
>> +# Purge installation logs of installed packages
>> +rm -f /var/log/dpkg.log /var/log/alternatives.log
>> +
>>  # Undo setup script changes
>>  if [ -x "$TARGET/sbin/start-stop-daemon.REAL" ]; then
>>      mv -f $TARGET/sbin/start-stop-daemon.REAL
>> $TARGET/sbin/start-stop-daemon
> 

-- 
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Corporate Competence Center Embedded Linux