
Hi,
I was just about to apply a DR by David. First I ran
$ arc patch D390
which put me on a feature branch. Then I reworded the commit with
$ git commit --amend
and then I tried to land it with
$ arc land
Luckily I also passed --hold...
It even asked me if I want to land the commit although I am not the
author:
This branch has revision 'D390: Reorder GHC.List; fix performance
regressions' but you are not the author. Land this revision by dfeuer?
[y/N] y
But it still put me in the author field:
$ git show HEAD
commit a387c359b99667ce55016de3816b5d9873a155db
Author: Joachim Breitner

Austin: can you report this bug with the Phabricator people please.
For committers: don't use 'arc land' to land patches from others!
If this has happened to you in the past, could you please let me know. I am
soon updating the contributors list on
https://ghc.haskell.org/trac/ghc/wiki/TeamGHC, and would like it to be
complete. These are the patches that I'm aware of that were misattributed
this year:
D600: Brian McKenna -> Erik de Castro Lopo
D971: Bernard Desmyter -> Thomas Miedema
D974: Zejun Wu -> Simon Marlow
#11209: John Paul Adrian Glaubitz -> Sergei Trofimovich
Thanks
On Wed, Oct 29, 2014 at 8:21 AM, Joachim Breitner
Hi,
I was just about to apply a DR by David. First I ran $ arc patch D390 which put me on a feature branch. Then I reworded the commit with $ git commit --amend and then I tried to land it with $ arc land Luckily I also passed --hold...
It even asked me if I want to land the commit although I am not the author:
This branch has revision 'D390: Reorder GHC.List; fix performance regressions' but you are not the author. Land this revision by dfeuer? [y/N] y
But it still put me in the author field:
$ git show HEAD commit a387c359b99667ce55016de3816b5d9873a155db Author: Joachim Breitner
Date: Wed Oct 29 08:17:05 2014 +0100 Reorder GHC.List; fix performance regressions
Summary: Rearrange some oddly placed code.
(and undid my commit message changes....)
Did I do something wrong?
Anyways, it said something along the lines of Cleaning up feature branch... (Use `git checkout -b 'arcpatch-D390' 'a3ceb0e7f44d53f728fb8bd0dfb4c97297818029'` if you want it back.) so with $ git commit --amend -C a3ceb0e7f44d53f728fb8bd0dfb4c97297818029 I was able to get the edited commit message with the right author back.
Greetings, Joachim
-- Joachim “nomeata” Breitner mail@joachim-breitner.de • http://www.joachim-breitner.de/ Jabber: nomeata@joachim-breitner.de • GPG-Key: 0xF0FBF51F Debian Developer: nomeata@debian.org
_______________________________________________ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs

Hi, Am Freitag, den 18.12.2015, 19:08 +0100 schrieb Thomas Miedema:
D600: Brian McKenna -> Erik de Castro Lopo D971: Bernard Desmyter -> Thomas Miedema D974: Zejun Wu -> Simon Marlow #11209: John Paul Adrian Glaubitz -> Sergei Trofimovich
add D322 David Feuer → Joachim Breitner Greetings, Joachim -- Joachim “nomeata” Breitner mail@joachim-breitner.de • http://www.joachim-breitner.de/ Jabber: nomeata@joachim-breitner.de • GPG-Key: 0xF0FBF51F Debian Developer: nomeata@debian.org
participants (2)
-
Joachim Breitner
-
Thomas Miedema