Message ID | CAPnjgZ2FyZ_iqHZrgr+ePARzwOxc4nQOcKRW9LTi4aaikfRf_Q@mail.gmail.com |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
Series | Please pull u-boot-dm | expand |
On Wed, Jan 26, 2022 at 08:35:43PM -0700, Simon Glass wrote: > Hi Tom, > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/10771 > > > The following changes since commit 6146cd62aedc4849fec66f10ab0aa57f1dc64b8e: > > Merge branch '2022-01-24-assorted-updates' (2022-01-25 08:01:43 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-26jan22 > > for you to fetch changes up to 2d2384bbaff0ab84c868b553c74048a5f6acc9e3: > > tools: mkimage: Show where signatures/keys are written (2022-01-26 > 08:50:44 -0700) > I'm going to push back on this slightly. I ran "make pylint" (which yes, is new) locally, and it complains about needing to update scripts/pylint.base. Did that, and here's the less than obvious to me results: diff --git a/scripts/pylint.base b/scripts/pylint.base index cefdc2355506..f5a045a384a9 100644 --- a/scripts/pylint.base +++ b/scripts/pylint.base @@ -1,71 +1,105 @@ +0.00 patman.checkpatch +1.43 buildman.test +1.71 buildman.builder +10.00 binman.fmap_util +3.12 rkmux +3.25 patman.control +4.15 buildman.board +4.48 binman.main +4.50 dtoc.fdt_util +4.69 patman.tout +4.74 patman.commit +4.86 binman.setup +5.00 binman.state +5.00 dtoc.src_scan +5.00 patman.status +5.62 patman.main +5.89 patman.setup +6.16 patman.settings +6.17 buildman.toolchain +6.44 concurrencytest +6.47 patman.gitutil +6.67 +6.67 patman.series +6.70 dtoc.main +6.71 endian-swap +6.81 patman.test_util +6.88 binman.ftest +6.89 patman.tools +6.90 rmboard +6.91 buildman.builderthread +6.96 dtoc.test_src_scan +7.05 patman.test_checkpatch +7.10 binman.image_test +7.18 buildman.kconfiglib +7.25 moveconfig +7.26 dtoc.dtb_platdata +7.39 buildman.cmdline +7.45 binman.image +7.49 buildman.main +7.56 patman.func_test +7.76 zynqmp_pm_cfg_obj_convert +7.78 dtoc.setup +7.82 buildman.bsettings +7.90 dtoc.fdt +7.95 microcode-tool +8.04 patman.command +8.10 buildman.func_test +8.14 patman.cros_subprocess +8.14 patman.get_maintainer +8.23 patman.patchstream +8.32 patman +8.56 dtoc.test_fdt +8.62 patman.terminal +8.91 dtoc.test_dtoc +9.04 buildman.control +9.11 patman.project +9.29 genboardscfg +9.43 efivar _testing 0.83 +_testing 6.09 atf_bl31 -6.00 atf_fip 0.29 -binman.cbfs_util 8.38 -binman.cbfs_util_test 9.30 -binman.cmdline 9.09 -binman.control 4.92 +binman.bintool 8.59 +binman.bintool_test 9.87 +binman.cbfs_util 8.46 +binman.cbfs_util_test 9.38 +binman.cmdline 9.03 +binman.comp_util 6.88 +binman.control 5.01 binman.elf 6.73 binman.elf_test 5.41 -binman.entry 3.38 +binman.entry 3.36 binman.entry_test 5.34 binman.fdt_test 3.23 -binman.fip_util 9.86 -binman.fip_util_test 9.75 -binman.fmap_util 6.88 -binman.ftest 7.46 -binman.image 7.05 -binman.image_test 4.48 -binman.main 5.00 -binman.setup 5.00 -binman.state 4.15 +binman.fip_util 9.85 blob -1.58 blob_dtb -10.00 blob_ext -19.09 -blob_ext_list -0.32 +blob_ext_list 0.00 blob_named_by_arg -7.78 blob_phase -5.00 -buildman.board 7.82 -buildman.bsettings 1.71 -buildman.builder 6.91 -buildman.builderthread 7.39 -buildman.cmdline 9.04 -buildman.control 8.10 -buildman.func_test 7.18 -buildman.kconfiglib 7.49 -buildman.main 1.43 -buildman.test 6.17 -buildman.toolchain 6.55 capsule_defs 5.00 cbfs -1.44 +cbfstool 7.83 collection 2.67 -concurrencytest 7.26 conftest -3.29 conftest 1.88 conftest 5.13 conftest 6.56 cros_ec_rw -6.00 defs 6.67 -dtoc.dtb_platdata 7.90 -dtoc.fdt 4.50 -dtoc.fdt_util 6.70 -dtoc.main 7.78 -dtoc.setup 5.00 -dtoc.src_scan 8.91 -dtoc.test_dtoc 8.56 -dtoc.test_fdt 6.96 -dtoc.test_src_scan 9.43 -efivar 6.71 -endian-swap 9.29 fdtmap -3.28 files -7.43 fill -6.43 -fit 5.32 +fiptool 7.62 +fit 5.26 fmap -0.29 fstest_defs 8.33 fstest_helpers 4.29 -gbb -0.30 -genboardscfg 7.95 +futility 7.39 +gbb 0.83 +ifwitool 3.81 image_header 5.77 intel_cmc -12.50 intel_descriptor 4.62 @@ -75,43 +109,21 @@ intel_fsp -12.50 intel_fsp_m -12.50 intel_fsp_s -12.50 intel_fsp_t -12.50 -intel_ifwi 2.71 +intel_ifwi 2.88 intel_me -12.50 intel_mrc -10.00 intel_refcode -10.00 intel_vbt -12.50 intel_vga -12.50 -microcode-tool 7.25 -mkimage 2.57 -moveconfig 8.32 +lz4 6.30 +lzma_alone 6.97 +mkimage 2.31 +mkimage 7.86 multiplexed_log 7.49 opensbi -6.00 -patman 0.00 -patman.checkpatch 8.04 -patman.command 4.74 -patman.commit 3.25 -patman.control 8.14 -patman.cros_subprocess 7.56 -patman.func_test 8.14 -patman.get_maintainer 6.47 -patman.gitutil 5.62 -patman.main 8.23 -patman.patchstream 9.11 -patman.project 6.67 -patman.series 6.16 -patman.settings 5.89 -patman.setup 5.00 -patman.status 8.62 -patman.terminal 7.05 -patman.test_checkpatch 6.81 -patman.test_util 6.89 -patman.tools 4.31 -patman.tout 3.12 powerpc_mpc85xx_bootpg_resetvec -10.00 -rkmux 6.90 -rmboard 7.76 scp -6.00 -section 4.68 +section 4.56 sqfs_common 8.41 test 8.18 test_000_version 7.50 @@ -203,7 +215,7 @@ u_boot_tpl_with_ucode_ptr -20.83 u_boot_ucode 1.52 u_boot_utils 6.94 u_boot_with_ucode_ptr -0.71 -vblock -1.61 +vblock 0.27 vboot_evil 8.95 vboot_forge 9.22 x86_reset16 -15.71 @@ -212,4 +224,3 @@ x86_reset16_tpl -15.71 x86_start16 -15.71 x86_start16_spl -15.71 x86_start16_tpl -15.71 -zynqmp_pm_cfg_obj_convert 6.67 Entries like "score name" rather than "name score" are odd, why did that happen? Also there's at least a few entries where the score decreased, but I assume that's because of code removal. Can you interpret the above and tell me what's going on, and that it makes sense to you at least? And, as a problem, re-running "make pylint" with that base file then fails with awk-spew related to "score name" and not "name score" lines.
Hi Tom, On Thu, 27 Jan 2022 at 10:31, Tom Rini <trini@konsulko.com> wrote: > > On Wed, Jan 26, 2022 at 08:35:43PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/10771 > > > > > > The following changes since commit 6146cd62aedc4849fec66f10ab0aa57f1dc64b8e: > > > > Merge branch '2022-01-24-assorted-updates' (2022-01-25 08:01:43 -0500) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-26jan22 > > > > for you to fetch changes up to 2d2384bbaff0ab84c868b553c74048a5f6acc9e3: > > > > tools: mkimage: Show where signatures/keys are written (2022-01-26 > > 08:50:44 -0700) > > > > I'm going to push back on this slightly. I ran "make pylint" (which > yes, is new) locally, and it complains about needing to update > scripts/pylint.base. Did that, and here's the less than obvious to me > results: > > diff --git a/scripts/pylint.base b/scripts/pylint.base > index cefdc2355506..f5a045a384a9 100644 > --- a/scripts/pylint.base > +++ b/scripts/pylint.base > @@ -1,71 +1,105 @@ > +0.00 patman.checkpatch > +1.43 buildman.test > +1.71 buildman.builder > +10.00 binman.fmap_util > +3.12 rkmux > +3.25 patman.control > +4.15 buildman.board > +4.48 binman.main > +4.50 dtoc.fdt_util > +4.69 patman.tout > +4.74 patman.commit > +4.86 binman.setup > +5.00 binman.state > +5.00 dtoc.src_scan > +5.00 patman.status > +5.62 patman.main > +5.89 patman.setup > +6.16 patman.settings > +6.17 buildman.toolchain > +6.44 concurrencytest > +6.47 patman.gitutil > +6.67 > +6.67 patman.series > +6.70 dtoc.main > +6.71 endian-swap > +6.81 patman.test_util > +6.88 binman.ftest > +6.89 patman.tools > +6.90 rmboard > +6.91 buildman.builderthread > +6.96 dtoc.test_src_scan > +7.05 patman.test_checkpatch > +7.10 binman.image_test > +7.18 buildman.kconfiglib > +7.25 moveconfig > +7.26 dtoc.dtb_platdata > +7.39 buildman.cmdline > +7.45 binman.image > +7.49 buildman.main > +7.56 patman.func_test > +7.76 zynqmp_pm_cfg_obj_convert > +7.78 dtoc.setup > +7.82 buildman.bsettings > +7.90 dtoc.fdt > +7.95 microcode-tool > +8.04 patman.command > +8.10 buildman.func_test > +8.14 patman.cros_subprocess > +8.14 patman.get_maintainer > +8.23 patman.patchstream > +8.32 patman > +8.56 dtoc.test_fdt > +8.62 patman.terminal > +8.91 dtoc.test_dtoc > +9.04 buildman.control > +9.11 patman.project > +9.29 genboardscfg > +9.43 efivar > _testing 0.83 > +_testing 6.09 > atf_bl31 -6.00 > atf_fip 0.29 > -binman.cbfs_util 8.38 > -binman.cbfs_util_test 9.30 > -binman.cmdline 9.09 > -binman.control 4.92 > +binman.bintool 8.59 > +binman.bintool_test 9.87 > +binman.cbfs_util 8.46 > +binman.cbfs_util_test 9.38 > +binman.cmdline 9.03 > +binman.comp_util 6.88 > +binman.control 5.01 > binman.elf 6.73 > binman.elf_test 5.41 > -binman.entry 3.38 > +binman.entry 3.36 > binman.entry_test 5.34 > binman.fdt_test 3.23 > -binman.fip_util 9.86 > -binman.fip_util_test 9.75 > -binman.fmap_util 6.88 > -binman.ftest 7.46 > -binman.image 7.05 > -binman.image_test 4.48 > -binman.main 5.00 > -binman.setup 5.00 > -binman.state 4.15 > +binman.fip_util 9.85 > blob -1.58 > blob_dtb -10.00 > blob_ext -19.09 > -blob_ext_list -0.32 > +blob_ext_list 0.00 > blob_named_by_arg -7.78 > blob_phase -5.00 > -buildman.board 7.82 > -buildman.bsettings 1.71 > -buildman.builder 6.91 > -buildman.builderthread 7.39 > -buildman.cmdline 9.04 > -buildman.control 8.10 > -buildman.func_test 7.18 > -buildman.kconfiglib 7.49 > -buildman.main 1.43 > -buildman.test 6.17 > -buildman.toolchain 6.55 > capsule_defs 5.00 > cbfs -1.44 > +cbfstool 7.83 > collection 2.67 > -concurrencytest 7.26 > conftest -3.29 > conftest 1.88 > conftest 5.13 > conftest 6.56 > cros_ec_rw -6.00 > defs 6.67 > -dtoc.dtb_platdata 7.90 > -dtoc.fdt 4.50 > -dtoc.fdt_util 6.70 > -dtoc.main 7.78 > -dtoc.setup 5.00 > -dtoc.src_scan 8.91 > -dtoc.test_dtoc 8.56 > -dtoc.test_fdt 6.96 > -dtoc.test_src_scan 9.43 > -efivar 6.71 > -endian-swap 9.29 > fdtmap -3.28 > files -7.43 > fill -6.43 > -fit 5.32 > +fiptool 7.62 > +fit 5.26 > fmap -0.29 > fstest_defs 8.33 > fstest_helpers 4.29 > -gbb -0.30 > -genboardscfg 7.95 > +futility 7.39 > +gbb 0.83 > +ifwitool 3.81 > image_header 5.77 > intel_cmc -12.50 > intel_descriptor 4.62 > @@ -75,43 +109,21 @@ intel_fsp -12.50 > intel_fsp_m -12.50 > intel_fsp_s -12.50 > intel_fsp_t -12.50 > -intel_ifwi 2.71 > +intel_ifwi 2.88 > intel_me -12.50 > intel_mrc -10.00 > intel_refcode -10.00 > intel_vbt -12.50 > intel_vga -12.50 > -microcode-tool 7.25 > -mkimage 2.57 > -moveconfig 8.32 > +lz4 6.30 > +lzma_alone 6.97 > +mkimage 2.31 > +mkimage 7.86 > multiplexed_log 7.49 > opensbi -6.00 > -patman 0.00 > -patman.checkpatch 8.04 > -patman.command 4.74 > -patman.commit 3.25 > -patman.control 8.14 > -patman.cros_subprocess 7.56 > -patman.func_test 8.14 > -patman.get_maintainer 6.47 > -patman.gitutil 5.62 > -patman.main 8.23 > -patman.patchstream 9.11 > -patman.project 6.67 > -patman.series 6.16 > -patman.settings 5.89 > -patman.setup 5.00 > -patman.status 8.62 > -patman.terminal 7.05 > -patman.test_checkpatch 6.81 > -patman.test_util 6.89 > -patman.tools 4.31 > -patman.tout 3.12 > powerpc_mpc85xx_bootpg_resetvec -10.00 > -rkmux 6.90 > -rmboard 7.76 > scp -6.00 > -section 4.68 > +section 4.56 > sqfs_common 8.41 > test 8.18 > test_000_version 7.50 > @@ -203,7 +215,7 @@ u_boot_tpl_with_ucode_ptr -20.83 > u_boot_ucode 1.52 > u_boot_utils 6.94 > u_boot_with_ucode_ptr -0.71 > -vblock -1.61 > +vblock 0.27 > vboot_evil 8.95 > vboot_forge 9.22 > x86_reset16 -15.71 > @@ -212,4 +224,3 @@ x86_reset16_tpl -15.71 > x86_start16 -15.71 > x86_start16_spl -15.71 > x86_start16_tpl -15.71 > -zynqmp_pm_cfg_obj_convert 6.67 > > Entries like "score name" rather than "name score" are odd, why did that > happen? Also there's at least a few entries where the score decreased, > but I assume that's because of code removal. Can you interpret the > above and tell me what's going on, and that it makes sense to you at > least? And, as a problem, re-running "make pylint" with that base file > then fails with awk-spew related to "score name" and not "name score" > lines. It's because one of the lines doesn't have a filename and everything gets out of sync: ... 6.51 patman.tools 6.55 buildman.builderthread 6.67 6.67 binman.ftest 6.71 endian-swap ... I'd suggest going ahead with the PR and I can dig into it at the weekend. I was trying to do everything in the makefile but it is possible I was too ambitious and we might need a tool. As you say, this is not part of CI yet. Regards, Simon
On Thu, Jan 27, 2022 at 12:09:29PM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 27 Jan 2022 at 10:31, Tom Rini <trini@konsulko.com> wrote: > > > > On Wed, Jan 26, 2022 at 08:35:43PM -0700, Simon Glass wrote: > > > > > Hi Tom, > > > > > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/10771 > > > > > > > > > The following changes since commit 6146cd62aedc4849fec66f10ab0aa57f1dc64b8e: > > > > > > Merge branch '2022-01-24-assorted-updates' (2022-01-25 08:01:43 -0500) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-26jan22 > > > > > > for you to fetch changes up to 2d2384bbaff0ab84c868b553c74048a5f6acc9e3: > > > > > > tools: mkimage: Show where signatures/keys are written (2022-01-26 > > > 08:50:44 -0700) > > > > > > > I'm going to push back on this slightly. I ran "make pylint" (which > > yes, is new) locally, and it complains about needing to update > > scripts/pylint.base. Did that, and here's the less than obvious to me > > results: > > > > diff --git a/scripts/pylint.base b/scripts/pylint.base > > index cefdc2355506..f5a045a384a9 100644 > > --- a/scripts/pylint.base > > +++ b/scripts/pylint.base > > @@ -1,71 +1,105 @@ > > +0.00 patman.checkpatch > > +1.43 buildman.test > > +1.71 buildman.builder > > +10.00 binman.fmap_util > > +3.12 rkmux > > +3.25 patman.control > > +4.15 buildman.board > > +4.48 binman.main > > +4.50 dtoc.fdt_util > > +4.69 patman.tout > > +4.74 patman.commit > > +4.86 binman.setup > > +5.00 binman.state > > +5.00 dtoc.src_scan > > +5.00 patman.status > > +5.62 patman.main > > +5.89 patman.setup > > +6.16 patman.settings > > +6.17 buildman.toolchain > > +6.44 concurrencytest > > +6.47 patman.gitutil > > +6.67 > > +6.67 patman.series > > +6.70 dtoc.main > > +6.71 endian-swap > > +6.81 patman.test_util > > +6.88 binman.ftest > > +6.89 patman.tools > > +6.90 rmboard > > +6.91 buildman.builderthread > > +6.96 dtoc.test_src_scan > > +7.05 patman.test_checkpatch > > +7.10 binman.image_test > > +7.18 buildman.kconfiglib > > +7.25 moveconfig > > +7.26 dtoc.dtb_platdata > > +7.39 buildman.cmdline > > +7.45 binman.image > > +7.49 buildman.main > > +7.56 patman.func_test > > +7.76 zynqmp_pm_cfg_obj_convert > > +7.78 dtoc.setup > > +7.82 buildman.bsettings > > +7.90 dtoc.fdt > > +7.95 microcode-tool > > +8.04 patman.command > > +8.10 buildman.func_test > > +8.14 patman.cros_subprocess > > +8.14 patman.get_maintainer > > +8.23 patman.patchstream > > +8.32 patman > > +8.56 dtoc.test_fdt > > +8.62 patman.terminal > > +8.91 dtoc.test_dtoc > > +9.04 buildman.control > > +9.11 patman.project > > +9.29 genboardscfg > > +9.43 efivar > > _testing 0.83 > > +_testing 6.09 > > atf_bl31 -6.00 > > atf_fip 0.29 > > -binman.cbfs_util 8.38 > > -binman.cbfs_util_test 9.30 > > -binman.cmdline 9.09 > > -binman.control 4.92 > > +binman.bintool 8.59 > > +binman.bintool_test 9.87 > > +binman.cbfs_util 8.46 > > +binman.cbfs_util_test 9.38 > > +binman.cmdline 9.03 > > +binman.comp_util 6.88 > > +binman.control 5.01 > > binman.elf 6.73 > > binman.elf_test 5.41 > > -binman.entry 3.38 > > +binman.entry 3.36 > > binman.entry_test 5.34 > > binman.fdt_test 3.23 > > -binman.fip_util 9.86 > > -binman.fip_util_test 9.75 > > -binman.fmap_util 6.88 > > -binman.ftest 7.46 > > -binman.image 7.05 > > -binman.image_test 4.48 > > -binman.main 5.00 > > -binman.setup 5.00 > > -binman.state 4.15 > > +binman.fip_util 9.85 > > blob -1.58 > > blob_dtb -10.00 > > blob_ext -19.09 > > -blob_ext_list -0.32 > > +blob_ext_list 0.00 > > blob_named_by_arg -7.78 > > blob_phase -5.00 > > -buildman.board 7.82 > > -buildman.bsettings 1.71 > > -buildman.builder 6.91 > > -buildman.builderthread 7.39 > > -buildman.cmdline 9.04 > > -buildman.control 8.10 > > -buildman.func_test 7.18 > > -buildman.kconfiglib 7.49 > > -buildman.main 1.43 > > -buildman.test 6.17 > > -buildman.toolchain 6.55 > > capsule_defs 5.00 > > cbfs -1.44 > > +cbfstool 7.83 > > collection 2.67 > > -concurrencytest 7.26 > > conftest -3.29 > > conftest 1.88 > > conftest 5.13 > > conftest 6.56 > > cros_ec_rw -6.00 > > defs 6.67 > > -dtoc.dtb_platdata 7.90 > > -dtoc.fdt 4.50 > > -dtoc.fdt_util 6.70 > > -dtoc.main 7.78 > > -dtoc.setup 5.00 > > -dtoc.src_scan 8.91 > > -dtoc.test_dtoc 8.56 > > -dtoc.test_fdt 6.96 > > -dtoc.test_src_scan 9.43 > > -efivar 6.71 > > -endian-swap 9.29 > > fdtmap -3.28 > > files -7.43 > > fill -6.43 > > -fit 5.32 > > +fiptool 7.62 > > +fit 5.26 > > fmap -0.29 > > fstest_defs 8.33 > > fstest_helpers 4.29 > > -gbb -0.30 > > -genboardscfg 7.95 > > +futility 7.39 > > +gbb 0.83 > > +ifwitool 3.81 > > image_header 5.77 > > intel_cmc -12.50 > > intel_descriptor 4.62 > > @@ -75,43 +109,21 @@ intel_fsp -12.50 > > intel_fsp_m -12.50 > > intel_fsp_s -12.50 > > intel_fsp_t -12.50 > > -intel_ifwi 2.71 > > +intel_ifwi 2.88 > > intel_me -12.50 > > intel_mrc -10.00 > > intel_refcode -10.00 > > intel_vbt -12.50 > > intel_vga -12.50 > > -microcode-tool 7.25 > > -mkimage 2.57 > > -moveconfig 8.32 > > +lz4 6.30 > > +lzma_alone 6.97 > > +mkimage 2.31 > > +mkimage 7.86 > > multiplexed_log 7.49 > > opensbi -6.00 > > -patman 0.00 > > -patman.checkpatch 8.04 > > -patman.command 4.74 > > -patman.commit 3.25 > > -patman.control 8.14 > > -patman.cros_subprocess 7.56 > > -patman.func_test 8.14 > > -patman.get_maintainer 6.47 > > -patman.gitutil 5.62 > > -patman.main 8.23 > > -patman.patchstream 9.11 > > -patman.project 6.67 > > -patman.series 6.16 > > -patman.settings 5.89 > > -patman.setup 5.00 > > -patman.status 8.62 > > -patman.terminal 7.05 > > -patman.test_checkpatch 6.81 > > -patman.test_util 6.89 > > -patman.tools 4.31 > > -patman.tout 3.12 > > powerpc_mpc85xx_bootpg_resetvec -10.00 > > -rkmux 6.90 > > -rmboard 7.76 > > scp -6.00 > > -section 4.68 > > +section 4.56 > > sqfs_common 8.41 > > test 8.18 > > test_000_version 7.50 > > @@ -203,7 +215,7 @@ u_boot_tpl_with_ucode_ptr -20.83 > > u_boot_ucode 1.52 > > u_boot_utils 6.94 > > u_boot_with_ucode_ptr -0.71 > > -vblock -1.61 > > +vblock 0.27 > > vboot_evil 8.95 > > vboot_forge 9.22 > > x86_reset16 -15.71 > > @@ -212,4 +224,3 @@ x86_reset16_tpl -15.71 > > x86_start16 -15.71 > > x86_start16_spl -15.71 > > x86_start16_tpl -15.71 > > -zynqmp_pm_cfg_obj_convert 6.67 > > > > Entries like "score name" rather than "name score" are odd, why did that > > happen? Also there's at least a few entries where the score decreased, > > but I assume that's because of code removal. Can you interpret the > > above and tell me what's going on, and that it makes sense to you at > > least? And, as a problem, re-running "make pylint" with that base file > > then fails with awk-spew related to "score name" and not "name score" > > lines. > > > It's because one of the lines doesn't have a filename and everything > gets out of sync: > > ... > 6.51 patman.tools > 6.55 buildman.builderthread > 6.67 > 6.67 binman.ftest > 6.71 endian-swap > ... > > I'd suggest going ahead with the PR and I can dig into it at the > weekend. I was trying to do everything in the makefile but it is > possible I was too ambitious and we might need a tool. > > As you say, this is not part of CI yet. OK. Applied, thanks!