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

Bug#291107: kernel-patch-debian-2.6.9: bashism in apply/debian file



On Jan/19, Horms wrote:

> > Package: kernel-patch-debian-2.6.9
> > Version: 2.6.9-5
> > Severity: normal
> > 
> > 	The file apply/debian (/usr/src/kernel-patches/all/2.6.9/apply/debian
> > 	in my system) has a bashism in line 160:
> Wow, nobody notices this for months then two in one day.

	"It never rains but pours", wasn't it what they said? :-) I think the
problem is that most people is using bash as /bin/sh, so they never realized
about the problem.

> I just made a fix for this and sent it to #291039. Could you
> please test out the attached patch and see if it works for you.

	Patches fine, but then ...

/usr/src/kernel-patches/all/2.6.9/apply/debian: 160: Syntax error: word unexpected (expecting "do")

	It just lacks a ";" in that line, after the last backquote:

	for base in `(cd $home/series/ && ls -d *) | sort -rnt- -k 2` do

	I'd attach a patch, but it seemed a bit silly for such a little fix :-)

-- 
Roberto Suarez Soto					Alfa21 Outsourcing
    robe@alfa21.com				     http://www.alfa21.com



Reply to: