darcs

Patch 674 Accept issue2100: darcs add should move on from first ...

Title Accept issue2100: darcs add should move on from first ...
Superseder Nosy List kowey
Related Issues
Status accepted Assigned To kowey
Milestone

Created on 2011-09-07.17:12:38 by kowey, last changed 2012-05-12.19:33:34 by gh.

Files
File name Status Uploaded Type Edit Remove
accept-issue2100_-darcs-add-should-move-on-from-first-failure_.dpatch kowey, 2011-09-07.17:12:38 application/x-darcs-patch
make-exit-code-of-darcs-add-non_zero-if-any-adds-fail.dpatch gh, 2012-03-22.18:03:07 application/x-darcs-patch
patch-preview.txt gh, 2012-03-22.18:03:07 text/x-darcs-patch
unnamed kowey, 2011-09-07.17:12:38 text/x-darcs-patch
unnamed kowey, 2011-09-07.17:12:38
unnamed gh, 2012-03-22.18:03:07
See mailing list archives for discussion on individual patches.
Messages
msg14723 (view) Author: kowey Date: 2011-09-07.17:12:38
1 patch for repository http://darcs.net:

Note: not screened because I'm not even sure this is a sensible thing to want
(see issue2100; go ahead and screen if I'm just being silly)

Wed Sep  7 17:59:15 BST 2011  Eric Kow <kowey@darcs.net>
  * Accept issue2100: darcs add should move on from first failure.
Attachments
msg15380 (view) Author: gh Date: 2012-03-22.18:03:07
Changing the (failing test) so that it verifies that exit code
be non-zero as soon as one add fails.
(see http://bugs.darcs.net/issue2100)

1 patch for repository http://darcs.net:

Thu Mar 22 14:52:38 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * make exit code of darcs add non-zero if any adds fail
Attachments
msg15385 (view) Author: mndrix Date: 2012-03-23.21:00:20
gh's exit code patch looks good.

> hunk ./tests/failing-issue2100-add-failures.sh
> +not darcs add d/non-existent d/f
> +darcs whatsnew > log2 2>&1
> +grep "./d/non-existent does not exist" log2

This tests for whatsnew giving an error after a failed add.  As I 
understand issue2100, we want add to give the error.  Eric, did you mean 
to capture and grep "darcs add" output?
msg15676 (view) Author: gh Date: 2012-05-12.19:33:34
mndrix's comment is correct.

I've just self-accepeted a followup patch that fixes the test script
accordingly:

 hunk ./tests/failing-issue2100-add-failures.sh 34
-not darcs add d/non-existent d/f
+not darcs add d/non-existent d/f  > log1 2>&1
hunk ./tests/failing-issue2100-add-failures.sh 36
-grep "./d/non-existent does not exist" log2
+grep "./d/non-existent does not exist" log1

Accepting all of this. (Patches for test scripts can be self-accepted
according to http://wiki.darcs.net/Development/PatchReview, for those
who wonder.)
History
Date User Action Args
2011-09-07 17:12:38koweycreate
2011-12-27 19:57:46ganeshsetstatus: needs-screening -> needs-review
2012-03-22 18:03:07ghsetfiles: + patch-preview.txt, make-exit-code-of-darcs-add-non_zero-if-any-adds-fail.dpatch, unnamed
messages: + msg15380
2012-03-23 21:00:20mndrixsetstatus: needs-review -> followup-requested
assignedto: kowey
messages: + msg15385
2012-05-12 19:33:34ghsetstatus: followup-requested -> accepted
messages: + msg15676