Using bazaar (baz)

[Jérôme Marant]

Quoting Rob Browning
> > Looks good. I guess you apply all patches first, then unapply them al=
l,
> > right?
>
> The autofiles dpatch is (must be) listed last in 00list, so yes,
> dpatch-edit-patch will apply all of the other patches to the temp tree
> first.

Alright.

...
> > BTW, I did not understand about this exit 230. It looks that it abort=
s
> > dpatch and thus does not generate the patch. Isn't exit 0 expected
> > instead?
>
> The "set -e" and the initial trap is a trick to make sure that if the
> command sequence fails for any reason, it'll exit with 230 rather than
> with the exit code of the failed command.  This is so that
> dpatch-edit-patch will know something went wrong.  I'm not sure why
> dpatch-edit-patch doesn't consider any failure code a failure, but it
> doesn't.  The final "trap - EXIT" clears the EXIT trap after we're
> finished so we can exit normally.  Let me know if I didn't explain
> that clearly enough.

It is pretty clear. I'm not used to trap, but after reading the manpage,
I find it quite usefull. Thanks.

> > After all, if you think it'll prevent us from mistakes, you can
> > go ahead.
>
> Well, adding that should make it nearly impossible to end up with an
> autofiles.dpatch that isn't "in sync" with all of the dpatches, but
> practically speaking, it would also mean that every time you change or
> add *any* dpatch, you'll have to run "debian/rules autofiles-sync".
> Would that be too aggravating?  I'm not sure myself.

OK. Well, let's see if we do too many mistakes first :-) We'll add it
add it later if necessary.

Cheers,


--
Jérôme Marant