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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <365d7243-812c-4d65-b91b-a9843f038319@nvidia.com>
Date: Wed, 5 Jun 2024 17:14:42 +0100
From: Jon Hunter <jonathanh@...dia.com>
To: Masahiro Yamada <masahiroy@...nel.org>,
 Kent Overstreet <kent.overstreet@...ux.dev>
Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
 linux-kbuild@...r.kernel.org
Subject: Re: kbuild: mksysmap: Build regression for next-20240604


On 05/06/2024 14:02, Masahiro Yamada wrote:
> On Wed, Jun 5, 2024 at 9:56 PM Kent Overstreet
> <kent.overstreet@...ux.dev> 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@mail.gmail.com/T/#mfa8b89c64a6a14a764338e9398d40be9bc48cdf1

Works for me!

Thanks
Jon

-- 
nvpublic