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

Re: [PATCH] build: add default configuration

From
DADavid Aguilar <davvid-re5jqeeqqe8avxtiumwx3w@public.gmane.org>
Date
Sep 21, 2013, 06:33 UTC
Message-ID
<749d06d0-3e17-49bf-8299-86c99c0dd9f5@email.android.com>
In-Reply-To
<CAMP44s14hn5qvG_88e_7FDx9ExSSu8DsVFUa5uhU-aOzSFy4MQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
Felipe Contreras <felipe.contreras-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
Show 5 quoted lines
>I know 'git ci' is perfectly fine shortcut to 'git commit'.
>
>Either way, it doesn't matter. Even if we agree that /etc/gitconfig.d
>is what we want, or we add an /usr/share/git/config, Junio is not
>going to apply any patch, even if it's what most users want.
Please stop making personal attacks that add nothing to your argument. No one cares.  Let it be.
Let's move this in a more constructive direction then, no?
How about working on documenting the new aliases and add a knob to the Makefile so that we can choose whether or not to install the stock config?
I'm not trying to fight this patch -- the idea is nice. Most users and distros probably won't change stock aliases, so your energy may be better spent getting consensus on what the stock aliases could be. 

Would it not be better to have these aliases, plus/minus one or two, then none at all? ... Yes I know about .rpmsave files. For rpm, it'll refuse to upgrade Git since this new file will conflict with an existing package. That's easier to deal with because the config package can then be independently modified to install its file to eg git.d/foo.conf in the directory include example. That would then allow the upgrade, and at no point did the intended config ever get lost.

Puppet users, for example, may end up with rpmsave turds on their systems, though. When you are managing lots of machines this can be very annoying -- that's why I mentioned it. Don't bother arguing this point any further. It's boring. ... In summary -- makefile knob, please, and at least mention the stock aliases somewhere in the docs so that the users can know to read /etc/gitconfig if they want to know more. Who knows, maybe it will get applied, but it definitively won't if all you do is whine about it.

-- 
David

-- 
You received this message because you are subscribed to the Google Groups "Git for human beings" group.
To unsubscribe from this group and stop receiving emails from it, send an email to git-users+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/groups/opt_out.
Previous: Felipe ContrerasNext: Felipe Contreras
Message 10 of 11 in “build: add default configuration”
  1. build: add default configurationFelipe Contreras, Sep 17, 2013
  2. David AguilarSep 18, 2013
  3. Felipe ContrerasSep 18, 2013
  4. David AguilarSep 19, 2013
  5. Felipe ContrerasSep 19, 2013
  6. David AguilarSep 20, 2013
  7. Felipe ContrerasSep 20, 2013
  8. David AguilarSep 21, 2013
  9. Felipe ContrerasSep 21, 2013
  10. David AguilarSep 21, 2013
  11. Felipe ContrerasSep 21, 2013

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.