Hi,
On Wed, Mar 21, 2007 at 10:40:26AM +0000, Jörg Sommer wrote:
> Hi Charles,
>
> Charles Plessy <[EMAIL PROTECTED]> wrote:
> > Le Tue, Mar 20, 2007 at 11:31:06AM -0700, Brandon Philips a écrit :
> >> > >
> >> > > I moved to debian/patches with dpatch. Is this a reasonable solution?
> >> >
> >> > use what you like. I usually find quilt simpler, but really, I care
> >> > much about you beeing comfortable with it than me. You may want to
> >> > read[0].
> >>
> >> Wow, quilt is much easier to use. The new maintainer guide recommended
> >> dpatch but it sort of sucked, good thing I asked :)
> >
> > For the moment I use dpatch, but is is a slight work overhead since it
> > is needed to convert patches to dpatches,
>
> Are you sure, this is necessary? AFAIK the comments and shell commands
> are optional.
Comments are optional but the first line is needed now as:
#! /bin/sh /usr/share/dpatch/dpatch-run
I had the same impression as yours.
I guess changing following 2 lines should fix situation.
line 416: test -x ${patch} || chmod +x ${patch}
line 434: if eval ${patch} -patch ${wd} ${redir} ${stamp}.new 2>&1; then
I guess drop 416 and run dpatch-run in 434.
Probably, deapply needs fix too.
I do not know this is caused by some design decision or not. (CCing to current
active maintaner)
Osamu
