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

Nikolaus Schulz Nikolaus Schulz <microschulz@web.de>, 298017@bugs.debian.org
Tue, 8 Mar 2005 18:33:36 +0100


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.

Regards,
Nikolaus