[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: stable coreutils-8.1 today, fingers crossed
From: |
Jim Meyering |
Subject: |
Re: stable coreutils-8.1 today, fingers crossed |
Date: |
Thu, 19 Nov 2009 11:13:09 +0100 |
Pádraig Brady wrote:
> Jim Meyering wrote:
>>>From ce9ff50623ad010fb52c3a4bf18a232eb875137c Mon Sep 17 00:00:00 2001
>> From: Jim Meyering <address@hidden>
>> Date: Thu, 19 Nov 2009 10:13:22 +0100
>> Subject: [PATCH] tests: avoid spurious failures due to insecure directory in
>> PATH
>>
>> These tests perform no PATH search, and used to simply delete PATH from
>> the environment. However, that is not portable, as seen on Cygwin,
>> where cygwin.dll must be resolvable via PATH when starting a sub-shell.
>> With commit 0cc04241, we took the alternate approach of untaining the
>
> s/untaining/untainting/
Thanks! Fixed.
> Off the top of my head the following might point
> out dodgy $PATH entries to the reporters:
>
> echo $PATH | tr : '\n' | sed -n '/^\//!p'
> find -H $(
> (
> echo $PATH | tr : '\n' | xargs -n1 readlink -f
> echo $PATH | tr : '\n'
> ) |
> sed -n ':s; p; s#/\{0,\}[^/]\{1,\}$##; /./b s' |
> sort -u
> ) -maxdepth 0 -perm -o=w
>
> Hmm, as we're adding /abs/path/to/coreutils/src to the start
> of the $PATH, then I'm guessing that this issue is caused
> by running from /tmp which is often o+w
> Maybe we need to just remove the coreutils/src from
> the $PATH for these tests?
I can attest that building from a subdirectory of /tmp (which has o+t set)
causes no problem. I do that all the time and it never fails. It's only
if your path contains a directory that is o+w, or if any path-dir *ancestor*
is o+w but without o+t.
- stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Pádraig Brady, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Andreas Schwab, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Bauke Jan Douma, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Gilles Espinasse, 2009/11/18
- Re: stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/19
- Re: stable coreutils-8.1 today, fingers crossed, Pádraig Brady, 2009/11/19
- Re: stable coreutils-8.1 today, fingers crossed,
Jim Meyering <=
- Re: stable coreutils-8.1 today, fingers crossed, Gilles Espinasse, 2009/11/21
- Re: stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/21
- Re: stable coreutils-8.1 today, fingers crossed, Gilles Espinasse, 2009/11/21
- Re: stable coreutils-8.1 today, fingers crossed, Gilles Espinasse, 2009/11/22
- Re: stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/22
- Re: stable coreutils-8.1 today, fingers crossed, Alan Curry, 2009/11/22
- Re: stable coreutils-8.1 today, fingers crossed, Jim Meyering, 2009/11/22
- permissions of files in dist tarball (was: stable coreutils-8.1 today, fingers crossed), Ralf Wildenhues, 2009/11/24
- Re: permissions of files in dist tarball, Jim Meyering, 2009/11/25
- Re: permissions of files in dist tarball, Ralf Wildenhues, 2009/11/27