darcs

Issue 457 Check --match syntax before (slow) commuting

Title Check --match syntax before (slow) commuting
Priority wishlist Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, kowey, markstos, thorkilnaur, tommy
Assigned To
Topics ProbablyEasy

Created on 2007-05-12.20:35:10 by tommy, last changed 2009-08-27.14:02:07 by admin.

Messages
msg1693 (view) Author: tommy Date: 2007-05-12.20:34:59
I did:

  darcs pull --match 'author "No Close Quote' ../many-commutations

Darcs commuted for ten minutes and then exited with:

  darcs: Invalid --match pattern 'author "No Close Quote'.
      "match" (line 1, column 23):
      unexpected end of input
      expecting "\\" or "\""

I wish darcs would check command syntax (and fail) before doing
all the commutations.

This is with both current stable (1.0.9rc2 + some patches) and
current unstable.
msg2301 (view) Author: markstos Date: 2008-01-01.22:04:47
I agree this change would be a nice improvement. 

With novice Haskell skill, I was unable to figure out how to refactor things for
a fix.
msg2865 (view) Author: kowey Date: 2008-01-29.15:59:16
I think my patch implements this.

Mon Jan 28 14:48:40 GMT 2008  Eric Kow <E.Y.Kow@brighton.ac.uk>
  * [issue457] Check match syntax early.
History
Date User Action Args
2007-05-12 20:35:10tommycreate
2007-08-03 07:08:58koweysettopic: + ProbablyEasy
nosy: droundy, tommy, beschmi, kowey
title: Bad --match syntax not detected before (slow) commuting -> Check --match syntax before (slow) commuting
2008-01-01 22:04:55markstossetstatus: unread -> unknown
nosy: + markstos
messages: + msg2301
2008-01-29 15:59:17koweysetstatus: unknown -> resolved-in-unstable
nosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg2865
2008-08-05 23:28:47koweysetstatus: resolved-in-unstable -> resolved
nosy: + dagit
2009-08-06 17:42:01adminsetnosy: + jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, simon, thorkilnaur, - droundy
2009-08-06 20:39:07adminsetnosy: - beschmi
2009-08-10 22:04:11adminsetnosy: - darcs-devel, zooko, jast, Serware, mornfall
2009-08-11 00:00:32adminsetnosy: - dagit
2009-08-25 17:55:36adminsetnosy: + darcs-devel, - simon
2009-08-27 14:02:07adminsetnosy: tommy, kowey, markstos, darcs-devel, thorkilnaur, dmitry.kurochkin