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

Re: [PATCH] tile: support GENERIC_KERNEL_THREAD and GENERIC_KERNEL_EXECVE

From
CMCatalin Marinas <catalin.marinas@arm.com>
Date
Oct 23, 2012, 21:09 UTC
Message-ID
<CAHkRjk6x9ToVzY7jv1ZxPt57F6agcH7SfHZpZNpHC3QP3PZp3Q@mail.gmail.com>
In-Reply-To
<20121023205119.GA27729@sigill.intra.peff.net>
On 23 October 2012 21:51, Jeff King <peff@peff.net> wrote:
Show 21 quoted lines
> On Tue, Oct 23, 2012 at 10:47:28PM +0200, Thomas Gleixner wrote:
>
>> I agree that this is a common issue. Acked-by/Reviewed-by mails come
>> in after the fact that the patch has been committed to an immutable
>> (i.e no-rebase mode) branch or if the change in question already hit
>> Linus tree.
>>
>> Still it would be nice to have a recording of that in the git tree
>> itself.
>>
>> Something like: "git --attach SHA1 <comment>" would be appreciated!
>
> It is spelled:
>
>   git notes add -m <comment> SHA1
>
> The resulting notes are stored in a separate revision-controlled branch
> and can be pushed and pulled like regular refs. Note, though, that the
> default refspecs do not yet include refs/notes, so you'd have to add
> them manually. The workflows around notes are not very mature yet, so if
> you start using them, feedback would be appreciated.

What would be nice is that notes are pushed/pulled automatically with standard git push/fetch/pull commands. Usually git walks the DAG starting with the pulled commit or tag and following the parents. With notes, the reference is reversed, the note pointing to the commit and not the other way around. So handling this automatically in Git would be really useful.

The other feature I'd like is that notes are automatically folded in the log during git rebase (maybe similar to the squash option). If you rebase, you lose all the notes (though this depends on the workflow, it may not be needed with published branches).

-- 
Catalin
Previous: Jeff KingNext: Jeff King
Message 3 of 14 in “Re: [PATCH] tile: support GENERIC_KERNEL_THREAD and GENERIC_KERNEL_EXECVE”
  1. Thomas GleixnerOct 23, 2012
  2. Jeff KingOct 23, 2012
  3. Catalin MarinasOct 23, 2012
  4. Jeff KingOct 23, 2012
  5. Catalin MarinasOct 24, 2012
  6. Thomas GleixnerOct 23, 2012
  7. Jeff KingOct 23, 2012
  8. Marc GauthierOct 23, 2012
  9. Jeff KingOct 23, 2012
  10. Johannes SixtOct 24, 2012
  11. Linus TorvaldsOct 24, 2012
  12. Al ViroOct 24, 2012
  13. Linus TorvaldsOct 24, 2012
  14. Ingo MolnarOct 24, 2012

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.