darcs

Patch 284 Failing test for issue1877: pull --xml-output should n...

Title Failing test for issue1877: pull --xml-output should n...
Superseder Nosy List kowey, lele
Related Issues
Status accepted Assigned To
Milestone

Created on 2010-06-20.09:28:34 by lele, last changed 2011-05-10.19:05:52 by darcswatch.

Files
File name Status Uploaded Type Edit Remove
failing-test-for-issue1877_-pull-__xml_output-should-not-emit-any-verbosity.dpatch lele, 2010-06-20.09:28:34 text/x-darcs-patch
unnamed lele, 2010-06-20.09:28:34
See mailing list archives for discussion on individual patches.
Messages
msg11492 (view) Author: lele Date: 2010-06-20.09:28:34
1 patch for repository http://darcs.net:

Sun Jun 20 11:31:36 CEST 2010  lele@nautilus.homeip.net
  * Failing test for issue1877: pull --xml-output should not emit any verbosity
Attachments
msg11507 (view) Author: kowey Date: 2010-06-21.17:15:02
It's always a big help when users contribute with test cases.
Thanks Lele!

This is on its way to being applied.

Failing test for issue1877: pull --xml-output should not emit any verbosity
---------------------------------------------------------------------------
> +# This does the right thing... at least until the second command is executed
> +not darcs pull --dry-run --xml-output 2>&1 ../R | grep "Would pull"
> +
> +# This does not, never
> +not darcs pull --summary --dry-run --xml-output 2>&1 ../R | grep "Would pull"
> +
> +# From now on, this fails too!
> +not darcs pull --dry-run --xml-output 2>&1 ../R | grep "Would pull"

As I mentioned on IRC, this is perhaps due to the defaultrepo being set.

I'll bet this is actually something you could help us fix, but I've
requested that Florent have a look in the meantime.  Hopefully in the
long term, we'll think of a good way to keep XML output intact, maybe
by printing log messages with the hiddenPrinter, or developing some
sort of logical log printer.

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
PGP Key ID: 08AC04F9
msg11508 (view) Author: kowey Date: 2010-06-21.17:42:21
Hmm, DarcsWatch isn't tracking this patch either.
Also, Lele, the convention is "Accept issue1877:" (although right now,
this has no effect).  Anyway, patch was applied.  Thanks!
msg11510 (view) Author: darcswatch Date: 2010-06-21.18:06:20
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-ff2abb0823ef67ea4df39e1614ea1d00a815f8bf
msg14140 (view) Author: darcswatch Date: 2011-05-10.19:05:52
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/reviewed.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-ff2abb0823ef67ea4df39e1614ea1d00a815f8bf
History
Date User Action Args
2010-06-20 09:28:34lelecreate
2010-06-21 17:15:03koweysetnosy: + kowey
messages: + msg11507
2010-06-21 17:42:21koweysetstatus: needs-review -> accepted
messages: + msg11508
2010-06-21 18:06:20darcswatchsetmessages: + msg11510
2011-05-10 19:05:52darcswatchsetmessages: + msg14140