darcs

Issue 1924 obliterate -o/-O is too unstable for 2.5 release

Title obliterate -o/-O is too unstable for 2.5 release
Priority bug Status resolved
Milestone 2.5.0 Resolved in 2.5.0
Superseder Nosy List dmitry.kurochkin, tux_rocker
Assigned To tux_rocker
Topics

Created on 2010-08-15.19:04:10 by tux_rocker, last changed 2010-09-03.16:21:30 by mornfall.

Messages
msg12194 (view) Author: tux_rocker Date: 2010-08-15.19:04:09
The -o and -O options must be removed from the 2.5 release, because they
are too buggy to be fixed in the short term.

As mornfall said it:

unfortunately trying to reproduce issue1921, I ran in couple more
(fairly serious) bugs about darcs unpull -o where it produces busted
bundles. We may consider disabling it for the release.

Basically, passing any noninteractive matchers to unpull -o will make it
produce a partial and definitely wrong context, which then makes darcs
apply explode in a fancy firework. I have observed commutation failures
and bogus conflictors at least.

This is of course fixable, but I am not sure we want to delay the
release a week or more for this (i.e. I won't have time to fix this
sooner, so unless someone else steps up, I suggest just yanking the
-o/-O options from unpull on the release branch).
msg12197 (view) Author: tux_rocker Date: 2010-08-15.20:14:10
The following patch updated issue issue1924 with status=resolved;resolvedin=2.5.0 CURRENT

* resolve issue1924: remove output options of obliterate 
Ignore-this: b85544067119c5c80db2ef449aea8456
msg12204 (view) Author: tux_rocker Date: 2010-08-15.21:11:58
The following patch updated issue issue1924 with status=resolved;resolvedin=2.8.0 HEAD

* resolve issue1924: remove output options of obliterate 
Ignore-this: b85544067119c5c80db2ef449aea8456
msg12205 (view) Author: kowey Date: 2010-08-15.21:16:39
Hah, our infrastructure's pretty stupid isn't it?  (We pushed this to
HEAD after we pushed it to branch-2.5, so that's what we get).

Note that I've rolled this back in HEAD.  
Tsk. Unfortunately, I wasn't paying enough attention to keep the patch
log under 72 chars.
msg12441 (view) Author: mornfall Date: 2010-09-03.16:21:29
This is resolved, dear tracker. At least for all I can tell. Dear Issue 
Manager will hopefully clean up after me if I messed up, thanks!
History
Date User Action Args
2010-08-15 19:04:10tux_rockercreate
2010-08-15 20:14:10tux_rockersetstatus: has-patch -> resolved
messages: + msg12197
resolvedin: 2.5.0
2010-08-15 21:11:59tux_rockersetmessages: + msg12204
resolvedin: 2.5.0 -> 2.8.0
2010-08-15 21:16:40koweysetstatus: resolved -> unknown
messages: + msg12205
resolvedin: 2.8.0 -> 2.5.0
2010-09-03 16:21:30mornfallsetstatus: unknown -> resolved
messages: + msg12441