Horms wrote:
> On Tue, Jan 18, 2005 at 07:48:38PM +0900, Kenshi Muto wrote:
> > Package: kernel-patch-debian-2.6.10
> > Severity: normal
> > Version: 2.6.10-3
> >
> > When I tried to build kernel 2.6.10 from debian source, I got following
> > message.
> >
> > /usr/src/kernel-patches/all/2.6.10/apply/debian 2.6.10-3
> > /usr/src/kernel-patches/all/2.6.10/apply/debian: 160: Syntax error: Missing
> > '))'
> >
> > I'm using dash as sh.
> > I think patch-code should use #!/bin/bash or be fixed for plain bourne
> > shell.
>
> Agreed, could you take a moment to see if the
> attached patch resolves your problem.
[snip]
> apply_patch() {
> - patch=$(find_patch $home/$1)
> + patch=`find_patch $home/$1`
FWIW, $( ... ) is POSIX-required and preferred syntax for ` ... `.
[snip]
> @@ -139,8 +139,7 @@
> die "Upstream $target_up doesn't match $upstream!"
> # We don't have that version out yet!
> elif [ ! -n "$target_rev" ] || ( [ "$target_rev" != "$target" ] && [
> $target_rev -gt $revision ] ); then
> - year=$(($(date +%Y) + 1))
> - die "Can't patch to nonexistent revision $target_rev (wait until $year)"
> + die "Can't patch to nonexistent revision $target_rev"
> fi
The shell arithmetic is the only bashism here.
Thiemo
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
