darcs

Patch 1062 Resolve issue2163: new option for amend, select author...

Title Resolve issue2163: new option for amend, select author...
Superseder Nosy List radoslav.dorcik
Related Issues darcs amend-record -A should default to my author id
View: 2163
Status accepted Assigned To
Milestone

Created on 2013-03-02.21:23:55 by radoslav.dorcik, last changed 2013-06-08.20:11:48 by gh.

Files
File name Status Uploaded Type Edit Remove
haddocks-for-functions-that-look-for-user-e_mail.dpatch gh, 2013-06-03.17:20:37 application/x-darcs-patch
patch-preview.txt radoslav.dorcik, 2013-03-02.21:23:54 text/x-darcs-patch
patch-preview.txt radoslav.dorcik, 2013-03-02.21:29:35 text/x-darcs-patch
patch-preview.txt gh, 2013-06-03.17:20:37 text/x-darcs-patch
resolve-issue2163_-new-option-for-amend_-select-author-for-patch-stealing_.dpatch dead radoslav.dorcik, 2013-03-02.21:23:54 application/x-darcs-patch
resolve-issue2163_-new-option-for-amend_-select-author-for-patch-stealing_.dpatch radoslav.dorcik, 2013-03-02.21:29:35 application/x-darcs-patch
unnamed radoslav.dorcik, 2013-03-02.21:23:54
unnamed radoslav.dorcik, 2013-03-02.21:29:35
unnamed gh, 2013-06-03.17:20:37
See mailing list archives for discussion on individual patches.
Messages
msg16738 (view) Author: radoslav.dorcik Date: 2013-03-02.21:23:54
1 patch for repository http://darcs.net/:

Sat Mar  2 22:05:49 UTC 2013  Radoslav Dorcik <dixiecko@gmail.com>
  * Resolve issue2163: new option for amend, select author for patch stealing.
  Feature request suggested the different implementation: I wish there
  was a way for darcs to pick my default id (content of either
  _darcs/author or .darcs/author as my id. I'd expect darcs am -A
  without argument to use that.
  
  But it makes impossible express both the author selection and also
  amended element: darcs -A somefile. For that reason, the new parameter
  is introduced --select-author. It can be used instead of -A for author
  selection. The author selection always offers the prompt, hence it is
  possible enter the other author than stored.
Attachments
msg16739 (view) Author: radoslav.dorcik Date: 2013-03-02.21:29:35
1 patch for repository http://darcs.net/:

Sat Mar  2 22:29:00 UTC 2013  Radoslav Dorcik <dixiecko@gmail.com>
  * Resolve issue2163: new option for amend, select author for patch stealing.
  Feature request suggested the different implementation: I wish there
  was a way for darcs to pick my default id (content of either
  _darcs/author or .darcs/author as my id. I'd expect darcs am -A
  without argument to use that.
  
  But it makes impossible express both the author selection and also
  amended element: darcs -A somefile. For that reason, the new parameter
  is introduced --select-author. It can be used instead of -A for author
  selection. The author selection always offers the prompt, hence it is
  possible enter the other author than stored.
Attachments
msg16740 (view) Author: dixiecko Date: 2013-03-02.21:32:15
The previous patch was obsoleted because I overlooked one warning:
missing getContent pattern match for new flag (SelectAuthor).
msg16823 (view) Author: gh Date: 2013-06-03.17:20:37
I'm adding this haddock patch that 'fixes' a couple of comments in
Darcs.UI.Arguments. The patch seems fine to me, if nobody objects I'm
going to accept it in a couple of days. It does add a new feature
( darcs amend --select-author ), which seems to work correctly.


1 patch for repository http://darcs.net:

Mon Jun  3 12:22:18 ART 2013  Guillaume Hoffmann <guillaumh@gmail.com>
  * haddocks for functions that look for user e-mail
Attachments
msg16833 (view) Author: gh Date: 2013-06-08.20:11:48
Done.
History
Date User Action Args
2013-03-02 21:23:55radoslav.dorcikcreate
2013-03-02 21:24:23dixieckosetissues: + darcs amend-record -A should default to my author id
2013-03-02 21:29:35radoslav.dorciksetfiles: + patch-preview.txt, resolve-issue2163_-new-option-for-amend_-select-author-for-patch-stealing_.dpatch, unnamed
messages: + msg16739
2013-03-02 21:32:15dixieckosetmessages: + msg16740
2013-06-03 15:08:47ghsetstatus: needs-screening -> needs-review
2013-06-03 17:20:38ghsetfiles: + patch-preview.txt, haddocks-for-functions-that-look-for-user-e_mail.dpatch, unnamed
messages: + msg16823
2013-06-08 20:11:48ghsetstatus: needs-review -> accepted
messages: + msg16833