Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-16 Thread Jim Meyering
Jim Meyering [EMAIL PROTECTED] wrote:
 I discovered that Solaris 11's /bin/sh exhibits the following
 surprising behavior:

 $ /bin/sh -c 'umask 22; (umask 0); umask'
 

 That would cause test failures in coreutils, because gl_POSIX_SHELL
 accepted /bin/sh.  The following change is enough to make
 gl_POSIX_SHELL reject that shell and instead use /bin/bash, which
 is also provided:

   posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)
   * m4/posix-shell.m4 (gl_POSIX_SHELL): Reject a shell that
   lets a umask modification leak out of a subshell.  Otherwise,
   Solaris 11's /bin/sh would be accepted and thus cause unwarranted
   failures in coreutils test suite.

I've pushed that change.




Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-16 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

According to Jim Meyering on 9/16/2008 3:58 AM:
 Jim Meyering [EMAIL PROTECTED] wrote:
 I discovered that Solaris 11's /bin/sh exhibits the following
 surprising behavior:

 $ /bin/sh -c 'umask 22; (umask 0); umask'
 

We should also document this in the autoconf manual.

- --
Don't work too hard, make some time for fun as well!

Eric Blake [EMAIL PROTECTED]
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkjPqjIACgkQ84KuGfSFAYAaQgCePsvgEpaBmjj8yzWy12pTG8vo
wQYAoKqLVm7+8CpKVoTX9/5CiwBlh4a1
=4APv
-END PGP SIGNATURE-




Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-16 Thread Peter O'Gorman
Eric Blake wrote:
 According to Jim Meyering on 9/16/2008 3:58 AM:
 Jim Meyering [EMAIL PROTECTED] wrote:
 I discovered that Solaris 11's /bin/sh exhibits the following
 surprising behavior:

 $ /bin/sh -c 'umask 22; (umask 0); umask'
 
 
 We should also document this in the autoconf manual.
 

Has it been reported? Seems like the kind of thing that could be fixed
before Solaris 11 is released.

http://defect.opensolaris.org/

Peter
-- 
Peter O'Gorman
http://pogma.com




Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-16 Thread Jim Meyering
Peter O'Gorman [EMAIL PROTECTED] wrote:

 Eric Blake wrote:
 According to Jim Meyering on 9/16/2008 3:58 AM:
 Jim Meyering [EMAIL PROTECTED] wrote:
 I discovered that Solaris 11's /bin/sh exhibits the following
 surprising behavior:

 $ /bin/sh -c 'umask 22; (umask 0); umask'
 

 We should also document this in the autoconf manual.


 Has it been reported? Seems like the kind of thing that could be fixed
 before Solaris 11 is released.

 http://defect.opensolaris.org/

Thanks for the link.  I've just filed a report:

http://defect.opensolaris.org/bz/show_bug.cgi?id=3450




Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-16 Thread Peter O'Gorman
Jim Meyering wrote:
 Peter O'Gorman [EMAIL PROTECTED] wrote:
 
 Eric Blake wrote:
 According to Jim Meyering on 9/16/2008 3:58 AM:
 Jim Meyering [EMAIL PROTECTED] wrote:
 I discovered that Solaris 11's /bin/sh exhibits the following
 surprising behavior:

 $ /bin/sh -c 'umask 22; (umask 0); umask'
 
 We should also document this in the autoconf manual.

 Has it been reported? Seems like the kind of thing that could be fixed
 before Solaris 11 is released.

 http://defect.opensolaris.org/
 
 Thanks for the link.  I've just filed a report:
 
 http://defect.opensolaris.org/bz/show_bug.cgi?id=3450

Ugh, I should have tried it before telling you to report it:
% uname -a
SunOS XX 5.11 snv_91 i86pc i386 i86pc
% /bin/sh -c 'umask 22; (umask 0); umask'
0022

Looks like this bug is already fixed somewhere between builds 86 and 91.

Peter
-- 
Peter O'Gorman
http://pogma.com




Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-13 Thread Bruno Haible
Hi Jim,

 I discovered that Solaris 11's /bin/sh ...

What is Solaris 11? Do you have any references? Neither Wikipedia [1] nor
the Sun documentation [2] mention such a release.

Bruno

[1] http://en.wikipedia.org/wiki/Solaris_(operating_system)
[2] http://docs.sun.com/app/docs/prod/solaris





Re: [PATCH] posix-shell.m4: reject Solaris 11's /bin/sh (due to umask)

2008-09-13 Thread Jim Meyering
Bruno Haible [EMAIL PROTECTED] wrote:
 I discovered that Solaris 11's /bin/sh ...

 What is Solaris 11? Do you have any references? Neither Wikipedia [1] nor
 the Sun documentation [2] mention such a release.

Hi Bruno,
Opensolaris calls itself SunOS 5.11.