git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: BUG: [Cosmetic] Commiting a gerrit ChangeId before the commit hook was installed

From
SStrainu <strainu10@gmail.com>
Date
Jan 17, 2014, 21:10 UTC
Message-ID
<CAC9meR+78_EvoqHNcuPyDKOZDjSsUngziGDa2aV1QDtOb8eOXw@mail.gmail.com>
In-Reply-To
<20140117204132.GB18964@google.com>
2014/1/17 Jonathan Nieder <jrnieder@gmail.com>:
Show 20 quoted lines
> Hi,
>
> Strainu wrote:
>
>> strainu@emily:~/core> git review -f
>> Creating a git remote called "gerrit" that maps to:
>>         ssh://strainu@gerrit.wikimedia.org:29418/pywikibot/core.git
>> Your change was committed before the commit hook was installed.
>> Amending the commit to add a gerrit change id.
>>
>> At this point I ended the transaction, as I was confused by the last
>> message: I was afraid the ChangeId would have changed, causing the
>> patch to be attached to another review.
>>
>> I think git should not show this message if the change description
>> already has a change id
>
> This message doesn't come from git.  It comes from the git-review
> tool (in git_review/cmd.py), so cc-ing the authors in case they
> have thoughts on that.
Thanks for clarifying that. I'll log a bug on launchpad then.
Strainu
Previous: Jonathan NiederNext: Jerry Xinyu Zhao
Message 3 of 5 in “BUG: [Cosmetic] Commiting a gerrit ChangeId before the commit hook was installed”
  1. StrainuJan 17, 2014
  2. Jonathan NiederJan 17, 2014
  3. StrainuJan 17, 2014
  4. Jerry Xinyu ZhaoJan 17, 2014
  5. StrainuJan 17, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.