Message ID | 20210216115609.24656-1-patrickdepinguin@gmail.com |
---|---|
State | Superseded |
Headers | show |
Series | utils/size-stats-compare: add package name in detail output | expand |
Thomas, All, On 2021-02-16 12:56 +0100, Thomas De Schampheleire spake thusly: > From: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com> > > size-stats-compare gives an overview of the size increase/decrease between > two cases, based on packages-file-list.txt. The 'detail' mode gives info per > file, otherwise per package. > > But sometimes, you want the detailed per-file info, but only for a specific > package. Since the detailed output no longer lists the package name, you > cannot simply grep for it. A workaround was to filter the input > packages-file-list.txt's first, and then pass these filtered versions to > size-stats-compare. > > Make this easier by adding the package name next to the filename in detailed > output. This allows grep'ing normally. > For example: > > $ utils/size-stats-compare orig new -t 100 -d | grep ebtables > -67712 removed lib/ebtables/libebtc.so (ebtables) > -66764 removed lib/ebtables/libebt_nat.so (ebtables) > -66752 removed sbin/ebtables (ebtables) > -66704 removed lib/ebtables/libebt_arp.so (ebtables) > -66700 removed lib/ebtables/libebt_stp.so (ebtables) > -66700 removed lib/ebtables/libebt_among.so (ebtables) > -66684 removed lib/ebtables/libebt_ip.so (ebtables) > -66676 removed lib/ebtables/libebt_limit.so (ebtables) > -66656 removed lib/ebtables/libebt_log.so (ebtables) > -66648 removed lib/ebtables/libebt_mark.so (ebtables) > -66636 removed lib/ebtables/libebt_pkttype.so (ebtables) > -66604 removed lib/ebtables/libebt_vlan.so (ebtables) > -66588 removed lib/ebtables/libebt_ulog.so (ebtables) > -66588 removed lib/ebtables/libebt_nflog.so (ebtables) > -66584 removed lib/ebtables/libebt_arpreply.so (ebtables) > -66544 removed lib/ebtables/libebt_ip6.so (ebtables) > -66540 removed lib/ebtables/libebt_802_3.so (ebtables) > -66536 removed lib/ebtables/libebt_standard.so (ebtables) > -66524 removed lib/ebtables/libebt_mark_m.so (ebtables) > -66524 removed lib/ebtables/libebt_redirect.so (ebtables) > -66452 removed lib/ebtables/libebtable_broute.so (ebtables) > -66452 removed lib/ebtables/libebtable_filter.so (ebtables) > -66452 removed lib/ebtables/libebtable_nat.so (ebtables) > 66752 added usr/sbin/ebtablesd (ebtables) > 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? Regards, Yann E. MORIN. > Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com> > --- > utils/size-stats-compare | 17 +++++++++++------ > 1 file changed, 11 insertions(+), 6 deletions(-) > > diff --git a/utils/size-stats-compare b/utils/size-stats-compare > index a3d7f250c6..7a6cb9d258 100755 > --- a/utils/size-stats-compare > +++ b/utils/size-stats-compare > @@ -40,9 +40,9 @@ def read_file_size_csv(inputf, detail=None): > > for row in reader: > if detail: > - sizes[row[0]] = int(row[2]) > + sizes[(row[0], row[1])] = int(row[2]) > else: > - sizes[row[1]] = int(row[3]) > + sizes[(None, row[1])] = int(row[3]) > > return sizes > > @@ -73,13 +73,18 @@ def print_results(result, threshold): > > from six import iteritems > list_result = list(iteritems(result)) > - # result is a dictionary: name -> (flag, size difference) > - # list_result is a list of tuples: (name, (flag, size difference)) > + # result is a dictionary: (filename, pkgname) -> (flag, size difference) > + # list_result is a list of tuples: ((filename, pkgname), (flag, size difference)) > + # filename may be None if no detail is requested. > > for entry in sorted(list_result, key=lambda entry: entry[1][1]): > if threshold is not None and abs(entry[1][1]) <= threshold: > continue > - print('%12s %7s %s' % (entry[1][1], entry[1][0], entry[0])) > + if entry[0][0]: > + name = '%s (%s)' % (entry[0][0], entry[0][1]) > + else: > + name = '%s' % entry[0][1] > + print('%12s %7s %s' % (entry[1][1], entry[1][0], name)) > > > # main ######################################################################### > @@ -126,5 +131,5 @@ print('Size difference per %s (bytes), threshold = %s' % (keyword, args.threshol > print(80*'-') > print_results(delta, args.threshold) > print(80*'-') > -print_results({'TOTAL': ('', sum(new_sizes.values()) - sum(old_sizes.values()))}, > +print_results({(None, 'TOTAL'): ('', sum(new_sizes.values()) - sum(old_sizes.values()))}, > threshold=None) > -- > 2.26.2 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
El mar, 2 mar 2021 a las 22:14, Yann E. MORIN (<yann.morin.1998@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
diff --git a/utils/size-stats-compare b/utils/size-stats-compare index a3d7f250c6..7a6cb9d258 100755 --- a/utils/size-stats-compare +++ b/utils/size-stats-compare @@ -40,9 +40,9 @@ def read_file_size_csv(inputf, detail=None): for row in reader: if detail: - sizes[row[0]] = int(row[2]) + sizes[(row[0], row[1])] = int(row[2]) else: - sizes[row[1]] = int(row[3]) + sizes[(None, row[1])] = int(row[3]) return sizes @@ -73,13 +73,18 @@ def print_results(result, threshold): from six import iteritems list_result = list(iteritems(result)) - # result is a dictionary: name -> (flag, size difference) - # list_result is a list of tuples: (name, (flag, size difference)) + # result is a dictionary: (filename, pkgname) -> (flag, size difference) + # list_result is a list of tuples: ((filename, pkgname), (flag, size difference)) + # filename may be None if no detail is requested. for entry in sorted(list_result, key=lambda entry: entry[1][1]): if threshold is not None and abs(entry[1][1]) <= threshold: continue - print('%12s %7s %s' % (entry[1][1], entry[1][0], entry[0])) + if entry[0][0]: + name = '%s (%s)' % (entry[0][0], entry[0][1]) + else: + name = '%s' % entry[0][1] + print('%12s %7s %s' % (entry[1][1], entry[1][0], name)) # main ######################################################################### @@ -126,5 +131,5 @@ print('Size difference per %s (bytes), threshold = %s' % (keyword, args.threshol print(80*'-') print_results(delta, args.threshold) print(80*'-') -print_results({'TOTAL': ('', sum(new_sizes.values()) - sum(old_sizes.values()))}, +print_results({(None, 'TOTAL'): ('', sum(new_sizes.values()) - sum(old_sizes.values()))}, threshold=None)