[Parted-maintainers] Bug#298017: parted: bashisms in *.dpatch sh scripts

[Sven Luther]

On Tue, Mar 08, 2005 at 06:33:36PM +0100, Nikolaus Schulz wrote:
> Sven Luther wrote:
> > On Fri, Mar 04, 2005 at 03:04:30AM +0100, Nikolaus Schulz wrote:
> > > the patches in debian/patches are sh scripts. 
> > > Running "checkbashisms debian/patches/*.dpatch" detects this:
> > > 
> > > amiga.dpatch line 8: '. $(dirname $0)/DPATCH'
> > > chs.dpatch line 8: '. $(dirname $0)/DPATCH'
> 
> <more snipped>
> 
> > So what ? bash is part of base or build-essential, so it should be no problem.
> > And since this is only used during package build ...
> 
> If you think it's a minor bug, feel free to downgrade it.
> 
> > Now, if you have a non-bashism workaround for the aboves, they are welcome.
> > maybe just doing . ./DPATCH ?
> 
> Just use backticks instead of the "$()" syntax. I'm not sure if the
> dpatch scripts are in the current directory when they're called. I doubt
> it.

. debian/patches/DPATCH 

would do ? 

Friendly,

Sven Luther