Created attachment 166528 [details] patch The attached patch makes USES=iconv permanent, instead of being only for NLS=on case. 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. I reported this bug to bash-devel ML. So in the future, when bash will fix this, this patch will make a difference.
A commit references this bug: Author: ehaupt Date: Tue Jun 28 07:28:02 UTC 2016 New revision: 417697 URL: https://svnweb.freebsd.org/changeset/ports/417697 Log: [1] Make USES=iconv permanent, instead of being only for NLS=on case. 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 Changes: head/shells/bash/Makefile head/shells/bash/distinfo
Committed, thanks!