DDMAL / vida.js

Verovio Image Display Assistant
8 stars 4 forks source link

Strike old copies of verovio toolkit from repo history #24

Open ahwitz opened 9 years ago

ahankinson commented 9 years ago

Are the binaries too big?

ahwitz commented 9 years ago

Yup. I don't think it's worth having three copies in each commit spread throughout this and the VerovioEditor repo when they're all 2-7MB each and only one is actually used. It might also not be worth the surgery required to complete this task, but we'll see.

ahankinson commented 9 years ago

Pretty easy, if not so elegant (basically you're rewriting your commit history, copying old commits to a new commit).

ahwitz commented 9 years ago

I've done something similar before many times, the issue is that it may not be as simple as one git filter-branch since the version numbers change commit to commit.

ahankinson commented 9 years ago

Regex?

ahwitz commented 9 years ago

I want to make sure I'm keeping the correct copy in the repo for each commit, and I'm not sure how consistent my stupidity was. Once I have time to do it, we'll see - this is rather low on my priority list at the moment.