On Sun, 2007-04-15 at 16:44 -0700, Junio C Hamano wrote: > > This change means there is no way to enable the automatic heuristics for a > > specific pattern once it has been disable for a more generic pattern. Would it > > make sense to make the attributes more than simply boolean? > > I do not think that is a problem in practice. Do not set > something to "false" explicitly with a generic pattern, if you > might want to override it. I also don't think it's a problem, but I think it would generally be a good idea to have values for attributes. So you can say crlf=yes|no|auto diff=yes|no|my-xml-diff|... merge=3way|... In the yes/no case we could keep the existing syntax on just add the attribute=othervalue for those that need more than a boolean decision. -- best regards Ray - 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