Fixed --convert-to-git-lfs command

[AntonOvseenko]

1. File .gitattributes created only in the root folder. Previously it created in all folders with lfs files.
2. Generating .gitattributes file. All file extensions written in a new line.
   Before:
   *.{zip,mp4} filter=lfs diff=lfs merge=lfs -text
   After:
   *.zip filter=lfs diff=lfs merge=lfs -text
   *.mp4 filter=lfs diff=-lfs merge=lfs -text

[dakotahawkins]

I think this will work for the example, but not for other valid patterns.

E.g., I have some things specified with extension wildcards, and other things specified explicitly, so I'm trying to run it like:

--convert-to-git-lfs '{*.lib,*.blah,Exactly/this/file,Special/files/under/here/**/*.special}'

Obviously I don't want it to remove all the occurrences of '*.' and then add one to the front of everything.

[dakotahawkins]

This may work better (haven't tested) at least for my particular case:

val globRegexMatcher = """([^{]*)\{?([^}]*)\}?(.*)""".r  
val globRegexMatcher(prefix, group, postfix) = lfsGlobExpression

val gitAttributes = group.split(",").map(prefix.concat(_).concat(postfix).concat(" filter=lfs diff=lfs merge=lfs -text"))

Anything before a '{' is the prefix, anything after a '}' is the postfix, and everything in between is assumed to be a comma-separated list.

I think if it's not using this syntax it should just spit out what you put in, but that might not work since in that case the whole string should be in prefix. This should be close though.

[AntonOvseenko]

@dakotahawkins Thank you! Your fix really works. But there is a problem. Correctly handled only this particular case: '{*.lib,*.blah,filename.extension}'. I could not find the path to the file in the commit tree. I only have the file name.

[dakotahawkins]

Were you able to see what it does if you call it without the brace syntax? Like:

--convert-to-git-lfs '*.lib'

There may need to be a check that if group is empty (or if it doesn't contain commas, what does split do?) you just add one thing to gitAttributes that's prefix.concat(group).concat(postfix) or something like that.

[AntonOvseenko]

Yes, I test it.I checked --convert-to-git-lfs: '*.lib', '*.{lib,png}', '{*.lib,*.png}', '{*.lib,*.png,filename.extension}'. All cases worked correctly.

[bicschneider]

is this fixed or abandonned ?

[AntonOvseenko]

Good question ) Need to check, 4 years have passed and I have not used this tool anymore