[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#480939: check for brace bashism in debhelper config files



Raphael Geissert wrote:
> > It seems that some people may decide that using brace expansion (ie,
> > docs/{foo,bar}) in a debian/package.docs file is a good idea. Despite it
> > not being documented anywhere, not being an intended syntax, working
> > only by accident in certian configurations, and of course, FTBFS if
> > /bin/sh is not bash.
> 
> Hmm, not quite right.
> From Dh_lib:
> >                 if (defined $globdir && ! compat(2)) {
> >                         for (map { glob "$globdir/$_" } split) {
> >                                 s#^$globdir/##;
> >                                 push @line, $_;
> >                         }
> >                 }
> 
> $ cat foo
> use strict;
> use warnings;
> use Data::Dumper;
> 
> print Dumper(glob('{a,b,c,d}.txt'));
> 
> $ perl foo
> $VAR1 = 'a.txt';
> $VAR2 = 'b.txt';
> $VAR3 = 'c.txt';
> $VAR4 = 'd.txt';
> 
> So, debhelper does indeed provide /bin/sh-independent brace expansion as long 
> as the compat level is four or above ;).

It doesn't matter if bash or perl is providing this, it's still
completly undocumented and works only in some situations due to a detail
of implementation.
 
-- 
see shy jo

Attachment: signature.asc
Description: Digital signature


Reply to: