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

Re: [PATCH] tests: do not use "slave branch" nomenclature

From
Michal Suchánek <msuchanek@suse.de>
Date
Jun 19, 2020, 09:20 UTC
Message-ID
<20200619092035.GQ21462@kitsune.suse.cz>
In-Reply-To
<xmqqwo44nt6m.fsf@gitster.c.googlers.com>
On Thu, Jun 18, 2020 at 11:08:01AM -0700, Junio C Hamano wrote:
Show 21 quoted lines
> Jeff King <peff@peff.net> writes:
> 
> > On Thu, Jun 18, 2020 at 06:38:43PM +0200, Paolo Bonzini wrote:
> >
> >> From: Paolo Bonzini <pbonzini@redhat.com>
> >> 
> >> Git does not have slave branches and has never had.  Independent
> >> of any future change to the naming of branches, remove the sole
> >> appearance of the term.
> >
> > I think this is a sensible change, though note that something simpler
> > was proposed recently:
> >
> >   https://lore.kernel.org/git/20200610165441.iktvzuwz44sbytfg@chatter.i7.local/
> >
> > and the review suggested using a name that is even more meaningful to
> > the test case (so we not just remove the unwanted names, but make the
> > test easier to follow).
> 
> Thanks for spotting that both versions share the same issue.  We'd
> just need a single one that is fixed up ;-).
How about s/slave/feature/

This is about simulating some development happening on the new branch so this name looks appropriate.

Thanks
Michal
Previous: Junio C HamanoNext: Paolo Bonzini
Message 4 of 5 in “tests: do not use "slave branch" nomenclature”
  1. tests: do not use "slave branch" nomenclaturePaolo Bonzini, Jun 18, 2020
  2. Jeff KingJun 18, 2020
  3. Junio C HamanoJun 18, 2020
  4. Michal SuchánekJun 19, 2020
  5. Paolo BonziniJun 19, 2020

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.