Cask-repair won't work if Taps remote url are not in HTTP (ie. force ssh via gitconfig) about tiny-scripts HOT 8 CLOSED

Thanks a lot ! Your script is really useful and saves a lot of time when I want to push an upgrade to a casks !

from tiny-scripts.

Done.

from tiny-scripts.

That's working perfectly, thanks a lot !

from tiny-scripts.

Non-https remotes historically caused a lot of trouble, so I’m not very inclined to allow those. Yes, some people might be stopped from using the script, but not enough to be worth the hassle.

Edit cask-repair on your machine and comment out line 396. That alone should fix it for your case. Please try it out and report back.

from tiny-scripts.

It works fines once commented, thanks !

Would a command line option be something to consider then ? I don't really see myself having to do the modification each time the script is updated but aliasing cask-repair to cask-repair -u is pretty much fine.

I can understand that you do not wish to allow non-https remotes by default though I don't really see how they might cause problem.

from tiny-scripts.

Would a command line option be something to consider then ?

Not really, that’d be a confusing option.

I can understand that you do not wish to allow non-https remotes by default though I don't really see how they might cause problem.

This isn’t hypothetical. They did cause problems. That said, they did so before the script revamp, so I may have fixed those issues inadvertently. I’ll add [email protected]:caskroom/ to that check in the script. If it causes problems in the future, I’ll deal with them then.

from tiny-scripts.

@vitorgalvao Just checked the code and tested. It is not working properly as ssh remote are [email protected]:caskroom and not [email protected]/caskroom.

from tiny-scripts.

Should be fixed now.

from tiny-scripts.