From owner-svn-ports-all@FreeBSD.ORG Wed Apr 23 17:36:57 2014 Return-Path: Delivered-To: svn-ports-all@FreeBSD.org Received: from hammer.pct.niksun.com (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by hub.freebsd.org (Postfix) with ESMTP id 0C8F7C26; Wed, 23 Apr 2014 17:36:56 +0000 (UTC) Message-ID: <5357FA38.10200@FreeBSD.org> Date: Wed, 23 Apr 2014 13:36:56 -0400 From: Jung-uk Kim User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Wolfgang Jenkner , Emanuel Haupt Subject: Re: svn commit: r351864 - head/shells/bash References: <5356B13D.3010600@gmx.de> <3C27DBBC-628F-48F7-9EC4-82C7BA559D85@adamw.org> <20140423102806.daa72bd1fec855d882d47163@critical.ch> <85ha5kgdu8.fsf@iznogoud.viz> In-Reply-To: <85ha5kgdu8.fsf@iznogoud.viz> X-Enigmail-Version: 1.6 Content-Type: multipart/mixed; boundary="------------080308030503020804040701" Cc: olli hauer , svn-ports-all@FreeBSD.org, ports-committers@freebsd.org, Emanuel Haupt , Adam Weinberger X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Apr 2014 17:36:57 -0000 This is a multi-part message in MIME format. --------------080308030503020804040701 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit -----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" 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----- --------------080308030503020804040701 Content-Type: text/x-patch; name="bash.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="bash.diff" Index: shells/bash/Makefile =================================================================== --- shells/bash/Makefile (revision 351960) +++ shells/bash/Makefile (working copy) @@ -51,6 +51,7 @@ PORTDOCS= FAQ INTRO CHANGES COMPAT NEWS POSIX RBA CONFIGURE_ARGS+= --without-bash-malloc \ --disable-rpath \ --enable-disabled-builtins +CONFIGURE_ENV= bash_cv_dev_fd=standard .include --------------080308030503020804040701--