[MERGE] line-endings support: part 1 of 2: versioned properties

Alexander Belchenko bialix at ukr.net
Mon Apr 14 20:31:10 BST 2008


Neil Martinsen-Burrell пишет:
> Alexander Belchenko wrote:
>> Neil Martinsen-Burrell пишет:
>>
>>> In fact, we could leverage some of the code, as well as some
>>> mental complexity by making the possible patterns in prop-set the same
>>> as the
>>> patterns in ignore.
>>
>> Currently my code assume that filemasks should always start with '*'.
>> It's makes
>> the implementation much simpler.
> 
> It certainly would make it much simpler, but I think at an added mental 
> cost for every user of patterns.  One would have to remember that 
> patterns in the properties files are restricted to a (very) limited 
> subset of the possible patterns that can be used in other parts of 
> Bazaar.  You are only using the requirement that the pattern start with 
> '*' to disambiguate between file-ids and glob patterns.  There must be 
> another way of doing this that is not so restrictive.  Once that is 
> done, you current implementation should work exactly as is.

Accepting only filemasks in the form '*.ext' is deliberate decision.
Because if I will allow to using something like:

dir/subdir/*.c

Then I'll get potential problems with renames/moves. And I don't want to have this pain.

No, I think restriction for *.ext is right here.



More information about the bazaar mailing list