public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Zack Weinberg <zack@owlfolio.org>
To: Xi Ruoyao <xry111@xry111.site>, libc-alpha@sourceware.org
Subject: Re: [PATCH] Add config.log to .gitignore
Date: Wed, 22 Mar 2023 16:14:07 -0400	[thread overview]
Message-ID: <d8001275-13fb-938f-5027-bc62d8e06978@owlfolio.org> (raw)
In-Reply-To: <5ba7d99d44e6bea1a0159a5f2b421995cb1f95f2.camel@xry111.site>

On 2023-03-22 12:26 PM, Xi Ruoyao wrote:
> On Wed, 2023-03-22 at 12:01 -0400, Zack Weinberg via Libc-alpha wrote:
>> On 2023-03-21 10:10 AM, 荣涛 via Libc-alpha wrote:
>>>> How the config.log file could ever be here?
>>> You are right, sorry, my mistake.
>>
>> I think the patch is desirable anyway.  If someone makes a mistake [...]
> 
> I guess you mean "git add .", not "git commit -a".  The latter won't add
> config.log even it's there.

Yes, probably...

> I'll not object if the others think this patch can prevent a human error
> from "git add .", but I just want to say "don't use `git add .`, unless,
> maybe, you've visually inspected the output of `git status`".

Yes, that should be the advice for contributors, but mechanical 
backstops against human error are a good idea regardless.

I actually got into an argument with one of my coworkers once, over 
exactly what "git add" does, that only ended when we discovered he had 
become so habituated to typing "git add ." that he didn't even realize 
that that was what he was doing.

zw

      reply	other threads:[~2023-03-22 20:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-21 12:24 Rong Tao
2023-03-21 12:38 ` Xi Ruoyao
2023-03-21 14:10   ` =?gb18030?B?yNnMzg==?=
2023-03-22 16:01     ` Zack Weinberg
2023-03-22 16:26       ` Xi Ruoyao
2023-03-22 20:14         ` Zack Weinberg [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d8001275-13fb-938f-5027-bc62d8e06978@owlfolio.org \
    --to=zack@owlfolio.org \
    --cc=libc-alpha@sourceware.org \
    --cc=xry111@xry111.site \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).