ports/shells/bash, branch 2016Q3

MFH: r418029

2016-07-04T16:07:09Z

Fix build on FreeBSD 9 WITHOUT=NLS. Sponsored by: Absolight Approved by: ports-secteam (build fix blanket)

Remove stray ICONV patch that breaks unicode support if NLS is disabled.

2016-06-28T11:56:03Z

PR: 210654 Submitted by: yuri@rawbw.com

[1] Make USES=iconv permanent, instead of being only for NLS=on case.

2016-06-28T07:28:01Z

The reason is that NLS is related to message translations to different languages. And bash has the general feature that \uNNNN is translated into the unicode character using iconv. This is unicode support, and should have nothing to do with translations. bash also currently has a bug that --disable-nls causes it to fail to find iconv. This bug has been reported upstream to bash-devel ML. In the future, when bash will fix this, this will make a difference. Note that this uses iconv from libc. [2] Update to 4.3.46 PR: 206903 [1], 210620 [2] Submitted by: yuri@rawbw.com [1], pkubaj@anongoth.pl [2] Discussed with: adamw

Remove NLS, DOCS, EXAMPLES and IPV6 from OPTIONS_DEFAULT, they are enabled by default anyway and don't need to be listed

2016-05-23T20:35:01Z

Approved by: portmgr blanket

- Fix trailing whitespace in pkg-descrs, categories [p-x]*

2016-05-19T10:53:05Z

Approved by: portmgr blanket

The latest patch (42) has been slightly adjusted. See the following diff:

2015-12-06T11:45:53Z

--- diff begins here --- --- /usr/distfiles/bash.old/bash43-042 2015-08-13 21:41:06.000000000 +0200 +++ /usr/distfiles/bash/bash43-042 2015-12-01 22:57:39.000000000 +0100 @@ -27,7 +27,7 @@ + lex_rwlen = 0; } } -*** ../bash-4.3-patched/parse.y 2015-05-18 19:27:05.000000000 -0400 +*** ../bash-4.3-patched/y.tab.c 2015-05-18 19:27:05.000000000 -0400 --- y.tab.c 2015-06-29 10:59:27.000000000 -0400 *************** *** 6021,6024 **** --- diff ends here --- Notified by: many

Update to 4.3.42

2015-08-14T14:10:19Z

bash: Export symbols for "enable -f"

2015-07-30T18:50:18Z

PR: 201959 Reviewed by: ehaupt Approved by: markj (mentor) Sponsored by: EMC / Isilon Storage Division Differential Revision: https://reviews.freebsd.org/D3231

In addition to the previous commit also define USE_MKSTEMP=1 for a different

2015-07-10T08:11:23Z

code section. Notified by: naddy

Scripts like https://github.com/henricj/scripts/blob/master/stir.sh often

2015-07-09T05:47:32Z

generate a bunch of, cannot make pipe for process substitution: File exists errors. The problem lies with colliding pipe names. The code in sh_mktmpname() in lib/sh/tmpfile.c uses a combination of things like the current time, the PID, and an incrementing counter. Since the child PIDs tend to be sequential (barring kern.randompid=1), there are collisions. Fix this problem without rewriting upstream code by defining USE_MKTEMP=1 to use the mktemp(3) code of bash. FreeBSD's mktemp() is not nearly as bad as the bash default and isn't brain-damaged like some platforms (which is likely why the bash code tries to do it's own thing). In FreeBSD, "mktemp(3)" it uses arc4random to pick one of 62 symbols for each "X". Submitted by: Henric Jungheim