5 Aug
2013
5 Aug
'13
10:23 a.m.
On Mon, Aug 05, 2013 at 11:30:09AM +0800, lolilolicon wrote:
Allan McRae <allan@archlinux.org> wrote:
On 04/08/13 02:26, Dave Reisner wrote:
+ if [[ $1 = ?(_[[:alpha:]])+([[:alnum:]_])?(+)=* ]]; then
This matches "1=2" which then bails on the export. Should it be:
+(_[[:alpha:]])?([[:alnum:]_])?(+)=*
There must be a typo, I think it should be this:
[_[:alpha]]*([_[:alnum:]])?(+)
which also allows setting the special shell variable $_ but maybe it's fine.
Yeah, this is the correct solution. It's fine if people try to set things they aren't supposed to -- makepkg shouldn't actively be in the game of trying to discourage it.