linux-kernel - Re: [PATCH] scripts/ver

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]

Date:	Mon, 24 Aug 2009 12:19:46 +0800
From:	Amerigo Wang <xiyou.wangcong@...il.com>
To:	Christian Kujau <lists@...dbynature.de>
Cc:	Amerigo Wang <xiyou.wangcong@...il.com>,
	LKML <linux-kernel@...r.kernel.org>,
	torvalds@...ux-foundation.org, bunk@...nel.org
Subject: Re: [PATCH] scripts/ver_linux

On Sun, Aug 23, 2009 at 08:52:05PM -0700, Christian Kujau wrote:
>On Sun, 16 Aug 2009 at 22:54, Christian Kujau wrote:
>> On Mon, 17 Aug 2009 at 13:47, Amerigo Wang wrote:
>> > Hmm, so probably you are also using a Red Hat distribution?
>> > AFAIK, only some Fedora and RHEL don't have /sbin and /usr/sbin in $PATH.
>> 
>> No, Debian Linux - but as a mere user I never have sbin directories in my 
>> PATH. And since it is a headless system, I removed the console-* packages, 
>> that's why I did not have the "loadkeys" command installed. No big deal, 
>> but the ver_linux output looked weird....
>
>Hm, looking at the changelog of scripts/ver_linux again I see:
>
> > commit 48a61569bb5396415c5dad0e81e1cfeb87c0aca3
> > Author: Adrian Bunk <bunk@...nel.org>
> > Date:   Wed Jul 23 22:50:45 2008 +0300
> > 
> >  kbuild: scripts/ver_linux: don't set PATH
> >
> >  It would have saved both a bug submitter and me a few hours if
> >  scripts/ver_linux had picked the same gcc as the build.
> >
> >  Since I can't see any reason why it fiddles with PATH at all this 
> > patch therefore removes the PATH setting.
>
>I'm still a bit confused, Amerigo, why you thought not having /sbin
>directories in PATH was distribution specific - I don't recall any 
>distribution where mere users had /sbin:/usr/sbin in their PATH.


Try archlinux.


>I was tempted to specify the whole path in ver_linux, but that'll break 
>for sure. 
>
>Adrian, what do you think - this patch puts the sbin paths *after*
>PATH, so that should not lead to such conflicts your commit once fixed, 
>right?

I believe so.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/