svn commit: r351864 - head/shells/bash

[Jung-uk Kim]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 2014-04-23 13:06:35 -0400, Wolfgang Jenkner wrote:
> On Wed, Apr 23 2014, Emanuel Haupt wrote:
> 
>> "Adam Weinberger" <adamw at adamw.org> wrote:
>>> Also I'm confused by the statement "bash requires fdescfs(5)
>>> mounted on /dev/fd", as it runs just fine without it. Perhaps
>>> it should be changed to: bash will support a '/dev/fd/N' syntax
>>> if fdescfs(5) is mounted on /dev/fd
>> 
>> Adam, this seems more accurate. Please feel free to commit.
> 
> This is not quite true either, but kib@ perfectly explained the
> awkward situation in
> 
> http://permalink.gmane.org/gmane.os.freebsd.devel.amd64/15636
> 
> Also (if you look at what started the thread) the main problem is
> not the /dev/fd/* syntax per se, but the process substitution
> feature, as mat@ already observed.
> 
> So I'd suggest something like
> 
> You should mount fdescfs(5) on /dev/fd if you are installing a
> precompiled package from an official pkg repository or, more 
> generally, if bash was compiled with fdescfs(5) being mounted.
> 
> The reason is that bash implements process substitution by means of
> the file-descriptor file system if that was mounted at build time.
> 
> For details, please see `Process Substitution' in the bash(1)
> manual.

The attached patch should force configure script to use /dev/fd.

Jung-uk Kim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (FreeBSD)

iQEcBAEBAgAGBQJTV/o4AAoJEHyflib82/FGVZsH/0LS52li6o/qoQCp1uayylSZ
OtYEFzJKiF9yo0iCl472SNHzLHB9VyFlaiKFNsRivlYWz8EtVlzoSA5erQ5X+8Dg
Krbw6MFifgUeojAeD6MPb7K3ewsHNhcfOc0UmEoGsXOyvvd0b25ICpg5VAXIHl7f
pUj+zw5Pn//fdT+aaXclGjUN8CD+FwYO6yanRCjByhqGDqGNo4xjgCbAA3wTnMAC
4CUV36JNRYW8W1hoHdIhC+yrUzgxg+Tdb8vbvGA8BTVtfUzzvuuCnsKO7fbIMuep
bQTNJxOeyQrHLY+lfHpEk7YNFuGajbpuBkRNkfX0L9JOhBPh5Cya83JCeJ2+y+s=
=otzk
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bash.diff
Type: text/x-patch
Size: 420 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20140423/738ab635/attachment.bin>