Re: [RFC PATCH 0/4] pathspec labels [WAS: submodule groups]

From: Junio C Hamano <gitster@pobox.com>
Date: May 15, 2016, 19:33 UTC
Message-ID: <xmqqfutj5d73.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <xmqq60uf6v5d.fsf@gitster.mtv.corp.google.com>

Junio C Hamano <gitster@pobox.com> writes:

Show 9 quoted lines

> Duy Nguyen <pclouds@gmail.com> writes:
>
>> Instead of putting everything in under the same attribute name
>> "label", make one attribute per label? Would this work?
>>
>> *.[ch] c-group code-group
>
> The attribute subsystem expects that there will not be unbounded
> large number of attributes, so this is not a good direction to go.

Having said that, I do not mind too much if it turns out that it is necessary to use an unbounded set of random attributes to solve a specific problem, if the use case is good.

But even then, in order to avoid confusion and name clashes, I'd prefer to see more like

	*.[ch] group-c group-code

that is matched by

	git cmd ':(group:c code)

i.e. reserve a single prefix that is not and will not be used for other purposes.