[darcs-users] correcting typos with amend-record
Michael Hendricks
michael at ndrix.org
Sat Mar 31 11:42:43 UTC 2012
On Sat, Mar 31, 2012 at 12:00 AM, Eric Kow <eric.kow at gmail.com> wrote:
>
> Perhaps best folded into the darcs rebase command hierarchy,
> eg darcs rebase amend (deep amend record)?
>
Thanks for the feedback. It's a good opportunity to mention some thoughts
I omitted from the first email. I didn't suggest a rebase subcommand
because
- a rebase command doesn't yet exist in standard Darcs
- users already use amend-record to reword patch descriptions
- I don't think darcs rebase command, as implemented, is a good way
forward for darcs (I've been compiling feedback after using the branch for
the last couple weeks and may send that later today)
- it seems odd to reword a patch with amend-record when there are no
dependencies but use rebase when there are. The user shouldn't have to
think about patch dependencies (that's why they're using Darcs :-)
That last point may be the most meaningful to me. Git has two commands for
this (git commit --amend vs git rebase -i, depending on whether the commit
you're rewording is the most recent). I've spent many hours trying to
explain the distinction to frustrated developers. We would all have been
happier if I could have said "Run `darcs amend --edit --none` and follow
the prompts, it'll do just what you want"
--
Michael
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osuosl.org/pipermail/darcs-users/attachments/20120331/48ca56c8/attachment.html>
More information about the darcs-users
mailing list