Re: kbuild: mksysmap: Build regression for next-20240604

From: Jon Hunter
Date: Wed Jun 05 2024 - 12:19:58 EST



On 05/06/2024 14:02, Masahiro Yamada wrote:
On Wed, Jun 5, 2024 at 9:56 PM Kent Overstreet
<kent.overstreet@xxxxxxxxx> wrote:

On Wed, Jun 05, 2024 at 09:14:23AM +0100, Jon Hunter wrote:
Hi Kent,

Since next-20240604 I noticed a build regression in our farm builders
and I am seeing the following error ...

/usr/bin/env: invalid option -- 'S'
Try '/usr/bin/env --help' for more information.

These builders have an older version of 'env' that do not support the
'-S' argument and so your change [0] is breaking the for these
machines. I did not see your patch on any list and found it on your
bcachefs tree [1]. So not clear if this has been reviewed and if this
is trying to fix some related to Masahiro change [2]?

It is also not clear what the minimum version of 'env' is supported
for building the linux kernel, but these builders have been building
the kernel fine for years.

Thanks
Jon

[0] https://evilpiepirate.org/git/bcachefs.git/commit/?h=for-next&id=973eca8db5570dd0c3f2b3190867138cc446eb3b
[1] https://evilpiepirate.org/git/bcachefs.git/log/?h=for-next
[2] https://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git/commit/?h=kbuild&id=b18b047002b7d3b19d9fb905c1bd2a214016c153

I'll drop it, but we need a real fix then, my CI builders are broken
without it...

Proper fixes are always good so that everyone's CI builders work :-)

I will apply Richard's patch instead and send a pull request for -rc3.

https://lore.kernel.org/linux-kbuild/CAK7LNASx_iSWZ14W5dpTOM87kB8A-Paq8fM_3nKH5tdfJQ7+ZQ@xxxxxxxxxxxxxx/T/#mfa8b89c64a6a14a764338e9398d40be9bc48cdf1

Works for me!

Thanks
Jon

--
nvpublic