darcs

Patch 598 resolve issue2065: filter and give warning for any mal...

Title resolve issue2065: filter and give warning for any mal...
Superseder Nosy List owst
Related Issues
Status accepted Assigned To
Milestone 2.8.0

Created on 2011-04-23.02:10:22 by owst, last changed 2012-04-20.19:46:27 by mndrix.

Files
File name Status Uploaded Type Edit Remove
resolve-issue2065_-filter-and-give-warning-for-any-malformed-boring-regexs_.dpatch owst, 2011-04-23.02:10:22 application/x-darcs-patch
unnamed owst, 2011-04-23.02:10:22 text/x-darcs-patch
unnamed owst, 2011-04-23.02:10:22
See mailing list archives for discussion on individual patches.
Messages
msg13956 (view) Author: owst Date: 2011-04-23.02:10:22
1 patch for repository http://darcs.net/screened:

This patch works, but has been observed to cause segfaults in the testsuite,
due to a bug in the regex-posix library, as reported here: https://sourceforge.net/tracker/?func=detail&aid=3291702&group_id=163236&atid=826943

Sat Apr 23 01:28:22 BST 2011  Owen Stephens <darcs@owenstephens.co.uk>
  * resolve issue2065: filter and give warning for any malformed boring regexs.
Attachments
msg14006 (view) Author: ganesh Date: 2011-05-08.17:46:05
I'm reluctantly sticking this in "in-discussion" for now, because I don't 
really want to introduce segfaults into our test suite, even if they 
aren't our fault.
msg15579 (view) Author: mndrix Date: 2012-04-20.17:28:23
Setting status back to in-discussion.  It was needs-review, but this patch 
is not in the screened repository and I don't see any discussion about 
gh's segfault concerns.
msg15582 (view) Author: mndrix Date: 2012-04-20.19:46:27
Owen pointed out that the segfault problem has already been resolved.  
Accepting.

To push this patch elsewhere, push "Patch 598 dependencies" which bundles 
both necessary patches together.
History
Date User Action Args
2011-04-23 02:10:22owstcreate
2011-05-08 17:46:06ganeshsetstatus: needs-screening -> in-discussion
messages: + msg14006
2012-01-20 06:23:35ganeshsetstatus: in-discussion -> needs-review
2012-03-08 16:14:12owstsetmilestone: 2.8.0
2012-04-20 17:28:23mndrixsetstatus: needs-review -> in-discussion
messages: + msg15579
2012-04-20 19:46:27mndrixsetstatus: in-discussion -> accepted
messages: + msg15582