Re: [PATCH 3/8] archive: add a failure test wrt .gitattributes misreading

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Junio C Hamano schrieb:
> René Scharfe <rene.scharfe@xxxxxxxxxxxxxx> writes:
> 
>> Hmm, "fix" implies that something is broken without this option, which
>> is not necessarily the case.
> 
> "What is recorded in the tree is unsuitable for the purpose of this
> particular export, so we are xxxing using yyy", is the intended use case,
> and I think the verb "fix" is a good fit there in "xxx" part.  "Tweak"
> would also work well.
> 
> Your suggestion is to say "attributes from the work tree" in place of
> "yyy", which also is good.

> I got the impression that people tend to prefer the operation named after
> what it does than how it does it, but I can go either way.

That's a good rule to follow in general.  "Fix" sounds to me as if git's
default behaviour is wrong and needs an option to make it work
correctly, though.  It's too broad a term.

I think the fixing, tweaking or even breaking will be done by the user,
when editing the attribute files in the work tree.  git then only
applies or evaluates these attributes, creating the archive using a
different source of meta data.  "--worktree-attributes" captures this
nicely.

René
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]