quilt-dev
[Top][All Lists]
Advanced

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

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


From: Peter Williams
Subject: Re: [Quilt-dev] Quilt 0.43 has been released! [SERIOUS BUG]
Date: Tue, 14 Feb 2006 15:36:53 +1100
User-agent: Mozilla Thunderbird 1.0.7-1.1.fc4 (X11/20050929)

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]