Re: [Quilt-dev] Quilt 0.43 has been released! [SERIOUS BUG]

[Peter Williams]

Andreas Gruenbacher wrote:
> On Tuesday 14 February 2006 04:25, Peter Williams wrote:
>
> > The problem arises when pushing a patch that has errors in it (due to
> > changes in the previous patches in the series) and needs the -f flag to
> > force the push.  What's happening is that the reverse of the errors is
> > being applied to the "pre patch" file in the .pc directory.  Then when
> > you pop this patch it returns the file to a state with the reverse of
> > the errors applied to it.
>
>
> Found and fixed. It's a missed rollback_patch on one of the two branches of 
> the code that checks if a patch can be reverse applied.

That was quick.

> This case apparently 
> doesn't trigger as easily as it seems, or else we would have found it sooner. 
> Still quite bad.
>
> Shall we wait until the translations are up-to-date again, or release 0.44 
> immediately?
>
>
> > I'm having trouble understanding how quilt could be dumb enough to do
> > this as surely the "pre patch" file in the .pc directory should be just
> > a copy of the file before the patch is applied.
>
>
> Hey, it's just a bug.

:-)

> > This bug can completely hose a set of patches if the user doesn't notice
> > it very early and do something about it.  The work around is to revert
> > to version 0.42 of quilt.
>
>
> You should have sent your gquilt announcement to the quilt-dev list as well. 
> Thank you for summarizing the changes.

Sorry.  They were mainly due to your change from "-p <patch>" to "-P 
<patch>" for some commands.  The other issue was a change to the return 
value and error message when "quilt top" was used in a directory without 
any quilt data.  So no real changes to gquilt as seen by the user just 
implementation changes.

BTW the --version function made it possible to make gquilt still work 
with versions earlier than 0.43.

Thanks
Peter
--
Peter Williams                                   address@hidden

"Learning, n. The kind of ignorance distinguishing the studious."
 -- Ambrose Bierce