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

Re: quilt, cdbs, dpatch, but is there even simpler ?



Hi,

On Wed, Mar 21, 2007 at 10:40:26AM +0000, Jörg Sommer wrote:
> Hi Charles,
> 
> Charles Plessy <charles-debian-nospam@plessy.org> 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



Reply to: