darcs

Issue 186 "darcs push --dry-run --set-default some-repo" changes default repo

Title "darcs push --dry-run --set-default some-repo" changes default repo
Priority bug Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, kowey, marnix, thorkilnaur, tommy
Assigned To kowey
Topics

Created on 2006-06-10.06:36:10 by marnix, last changed 2009-08-27.14:10:52 by admin.

Messages
msg703 (view) Author: marnix Date: 2006-06-10.06:36:07
When doing "darcs push --dry-run" with "--set-default", I was expecting no
changes at all.  As it turns out, the 'set default' action *is* performed, in
other words, if a repository is given explicitly, this repository becomes the
default one.  This seems like a bug to me.  But tastes might differ...

Reproduction scenario, where /tmp/repo1 is the current defaultrepo (and where
there are no changes at all between the current repo, /tmp/repo1, and /tmp/repo2):

$ darcs push --dry-run
Pushing to "/tmp/repo1"...
No recorded local changes to push!
$ darcs push --dry-run --set-default /tmp/repo2
No recorded local changes to push!
$ darcs push --dry-run
Pushing to "/tmp/repo2"...
No recorded local changes to push!
$
msg1899 (view) Author: kowey Date: 2007-07-23.06:37:48
Resolved in unstable with the following patch.  Thanks!

Tue Jul 17 12:16:07 CEST 2007  Eric Kow <eric.kow@loria.fr>
  * Do not set defaultrepo if --dry-run (issue186).
History
Date User Action Args
2006-06-10 06:36:10marnixcreate
2007-07-17 10:18:50koweysetnosy: + kowey, beschmi
2007-07-23 06:37:49koweysetstatus: unread -> resolved-in-unstable
messages: + msg1899
2007-07-31 17:56:59koweysetstatus: resolved-in-unstable -> resolved-in-stable
2008-09-16 21:30:15adminsetstatus: resolved-in-stable -> resolved
nosy: + dagit
2009-08-06 17:51:28adminsetnosy: + markstos, jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, simon, thorkilnaur, - droundy, marnix
2009-08-06 20:50:10adminsetnosy: - beschmi
2009-08-10 21:51:59adminsetnosy: + marnix, - markstos, darcs-devel, zooko, jast, Serware, mornfall
2009-08-10 23:54:24adminsetnosy: - dagit
2009-08-25 18:02:50adminsetnosy: + darcs-devel, - simon
2009-08-27 14:10:52adminsetnosy: marnix, tommy, kowey, darcs-devel, thorkilnaur, dmitry.kurochkin