[Buildroot] [PATCH] utils/size-stats-compare: add package name in detail output

Thomas De Schampheleire patrickdepinguin at gmail.com
Wed Mar 3 15:00:45 UTC 2021


El mar, 2 mar 2021 a las 22:14, Yann E. MORIN
(<yann.morin.1998 at free.fr>) escribió:
[..]
> >        66752   added usr/sbin/ebtables-legacy (ebtables)
> >        66752   added usr/sbin/ebtablesu (ebtables)
> >       200840   added usr/lib/libebtc.so.0.0.0 (ebtables)
>
> I think it would make more sense to have the package name before the
> file name, so that it is easier to filter. Indeed, the package name will
> never have spaces, while the filenames may:
>
>     -66452 ebtables removed lib/ebtables/libebtable_nat.so
>      66752 ebtables added   usr/sbin/ebtablesd
>
> Yes, this means that parsers of this will have to be adapted. But even
> with the package name at the end, they would have to be adapted.
>
> Unless you have a strong argument to keep it at the end?

No I have no strong argument for that layout.
In fact, what you propose makes more sense because the package name
will always be in the third column, while with my approach it would
swap to the fourth column in the detailed mode.

I will send a v2.

Thanks!
Thomas


More information about the buildroot mailing list