Patch 995: Report an error if --disable-patch-index fails - Darcs bug tracker

Title	Report an error if --disable-patch-index fails
Superseder		Nosy List	bsrkaditya
Related Issues
Status	accepted	Assigned To
Milestone

Created on 2012-12-24.02:13:50 by bsrkaditya, last changed 2012-12-29.22:45:31 by darcswatch.

Files
File name	Status	Uploaded	Type	Edit	Remove
be-a-bit-more-robust-when-deleting-the-patch-index.dpatch		ganesh, 2012-12-29.00:26:25	application/x-darcs-patch
error.dpatch	dead	bsrkaditya, 2012-12-24.02:13:50	application/x-darcs-patch
patch-preview.txt		ganesh, 2012-12-28.22:17:59	text/x-darcs-patch
patch-preview.txt		ganesh, 2012-12-29.00:26:25	text/x-darcs-patch
report-an-error-if-__disable_patch_index-fails.dpatch		ganesh, 2012-12-28.22:17:59	application/x-darcs-patch
unnamed		ganesh, 2012-12-28.22:17:59
unnamed		ganesh, 2012-12-29.00:26:25

See mailing list archives for discussion on individual patches.

Messages
msg16480 (view)	Author: bsrkaditya	Date: 2012-12-24.02:13:50
Mark discovered (in issue2271) that --patch-index-disable reports success even if it failed. This patch corrects this problem. Attachments error.dpatch
msg16496 (view)	Author: ganesh	Date: 2012-12-28.22:17:59
The original attachment has got wrapped in HTML somehow so doesn't work properly. 1 patch for repository darcs-unstable@darcs.net:screened: Thu Dec 20 06:20:26 GMT 2012 bsrkaditya@gmail.com * Report an error if --disable-patch-index fails Attachments patch-preview.txt report-an-error-if-__disable_patch_index-fails.dpatch unnamed
msg16497 (view)	Author: darcswatch	Date: 2012-12-28.22:25:19
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/screened. This message was brought to you by DarcsWatch http://darcswatch.nomeata.de/repo_http:__darcs.net_screened.html#bundle-c8cf09b1e46dcc9c12fb1f13a3242dbfcc6e175f
msg16501 (view)	Author: ganesh	Date: 2012-12-29.00:26:25
The previous patch breaks a bunch of tests because it fails even if the patch index doesn't exist in the first place. 1 patch for repository darcs-unstable@darcs.net:screened: Fri Dec 28 23:58:42 GMT 2012 Ganesh Sittampalam <ganesh@earth.li> * be a bit more robust when deleting the patch index Attachments be-a-bit-more-robust-when-deleting-the-patch-index.dpatch patch-preview.txt unnamed
msg16506 (view)	Author: owst	Date: 2012-12-29.22:32:53
This, with Ganesh's follow up look sensible, and the tests pass now. Accepting.
msg16509 (view)	Author: darcswatch	Date: 2012-12-29.22:45:31
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-c8cf09b1e46dcc9c12fb1f13a3242dbfcc6e175f

History
Date	User	Action	Args
2012-12-24 02:13:50	bsrkaditya	create
2012-12-28 22:17:59	ganesh	set	files: + patch-preview.txt, report-an-error-if-__disable_patch_index-fails.dpatch, unnamed messages: + msg16496
2012-12-28 22:21:01	ganesh	set	status: needs-screening -> needs-review
2012-12-28 22:25:19	darcswatch	set	messages: + msg16497
2012-12-29 00:26:25	ganesh	set	files: + patch-preview.txt, be-a-bit-more-robust-when-deleting-the-patch-index.dpatch, unnamed messages: + msg16501
2012-12-29 22:32:53	owst	set	status: needs-review -> accepted messages: + msg16506
2012-12-29 22:45:31	darcswatch	set	messages: + msg16509