[arch-projects] [devtools] [PATCH 6/9] lib/common.sh: Add a lock_close function.

[Luke Shumaker]

At Sun, 11 May 2014 11:49:57 -0400,
Dave Reisner wrote:
> 
> On Sun, May 11, 2014 at 12:16:33AM -0400, Luke Shumaker wrote:
> > At Sat, 10 May 2014 22:40:02 -0400,
> > Dave Reisner wrote:
> > > > +	eval "exec $fd>&-"
> > > 
> > > eval isn't needed here, you can use:
> > > 
> > >   exec {fd}>&-
> > 
> > Whoa!  I should have taken a closer look at the Bash 4.1 CHANGES
> > file.  That's cool new bit of syntax.  I suppose neither of us is
> > concerned about supporting Bash 4.0 or earlier?
> 
> pacman adopted bash4 quite some time ago, and tends to be a little on
> the conservative side. I don't see any problem with devtools adopting
> bash4 features, since they're expected to be run on an up to date Arch
> install.

That syntax wasn't in 4.0, it's new to 4.1.  But yeah, I agree.