[MERGE] per-user file properties
Jelmer Vernooij
jelmer at samba.org
Tue May 6 13:21:59 BST 2008
Hi Ian,
Am Dienstag, den 06.05.2008, 16:39 +1000 schrieb Ian Clatworthy:
> This patch provides the plumbing for finding and reading per-user file
> properties. This feature is a building block for EOL support. However,
> it has been explicitly designed to be a generic capability and ought to
> be suitable for configuring other interesting things like custom diff
> and merge algorithms.
>
> Properties are defined in the BAZAAR_HOME/.bzrproperties file using an
> ini file syntax where the section headings are patterns and the section
> contents are the properties. Patterns use the same syntax as that used
> by .bzrignore files. Here is an example:
I haven't done any code review but a quick comment nonetheless;
I'd rather see a different filename than $BAZAAR_HOME/.bzrproperties.
Using a dot file doesn't make sense in this case since it is
inconsistent with the names of all other files in BAZAAR_HOME and it
makes it hard to discover the file. Also, I think the word 'file' should
be in the name somewhere to make clear what sort of properties these
are.
What about something like $BAZAAR_HOME/fileproperties ?
Cheers,
Jelmer
--
Jelmer Vernooij <jelmer at samba.org> - http://samba.org/~jelmer/
Jabber: jelmer at jabber.fsfe.org
More information about the bazaar
mailing list