darcs

Patch 366 Accept issue1926: amend-record ignores --index option.

Title Accept issue1926: amend-record ignores --index option.
Superseder Nosy List iago.abal
Related Issues
Status accepted Assigned To
Milestone

Created on 2010-08-25.17:03:07 by iago.abal1, last changed 2010-08-26.16:32:40 by kowey.

Files
File name Status Uploaded Type Edit Remove
AcceptIssue1926 iago.abal1, 2010-08-25.17:03:07 application/octet-stream
unnamed iago.abal1, 2010-08-25.17:03:07 text/html
See mailing list archives for discussion on individual patches.
Messages
msg12300 (view) Author: iago Date: 2010-08-25.17:03:07
-- 
Iago Abal Rivas
Attachments
msg12316 (view) Author: kowey Date: 2010-08-26.16:23:45
Thanks for the test!

> +## Test for issueNNNN - <SYNOPSIS: WHAT IS THE BUG?  THIS SYNOPSIS
> +## SHOULD BE ONE OR TWO SENTENCES.>
> +##
> +## Copyright (C) YEAR  AUTHOR

You forgot to customise this; I'll be pushing a follow up patch tweaking
the title and assigning you the copyright, which I think is safe to
assume you do not object to.

Also a good chance for us in the Review Team to remember the "trivial
things should not block patch application" principle, which I think I
tend to forget in my nitpickiness :-) [of course, there's also a sort of
"raise the bar" principle where you start to expect certain things from
longtime contributors, but that's a different story!]

> +echo 'Example content 1.' > f
> +darcs record -lam 'Add file f'
> +echo 'Example content 2.' > g
> +darcs record -lam 'Add file g'
> +darcs put ../S                  # S as a copy of R
> +echo "y" | darcs amend-record --index=2 -m 'A new file f'   # Since --index is ignored this command works in
> +                                                            # interactive mode, and the amended patch is the one
> +                                                            # with index 1.

I tried to improve this by using amend-record --all but Petr
pointed out that this applies to the primitive patches only
(we call them "changes" for end users).

Curious, I did a little test on darcs rollback to see how it
handles --all (since it's another command that interactively
steps through both named and primitive patches), and in that
context, all applies to both.

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
For a faster response, try +44 (0)1273 64 2905 or
xmpp:kowey@jabber.fr (Jabber or Google Talk only)
msg12317 (view) Author: kowey Date: 2010-08-26.16:32:40
Heh, darcswatch wasn't tracking this (probably because it was using some 
other mechanism to detect darcs patches, like searching for "darcs patch: 
" in the subject line or a .dpatch attachment).  Duct tape everywhere.
History
Date User Action Args
2010-08-25 17:03:07iago.abal1create
2010-08-25 17:06:54koweysetnosy: + iago.abal, - iago.abal1
2010-08-26 16:23:45koweysetmessages: + msg12316
2010-08-26 16:32:40koweysetstatus: needs-review -> accepted
messages: + msg12317