whyrusleeping / gx

A package management tool
MIT License
1.88k stars 110 forks source link

Fix/windows subtool suffix #154

Closed djdv closed 6 years ago

djdv commented 6 years ago

https://github.com/whyrusleeping/gx/pull/150 does not fix issue https://github.com/whyrusleeping/gx/issues/153 This set does. An explanation on the ||, Windows will accept both / and its native separator, so we need to check for either.

ae64f8aabdc1751565054920e50ffcd4fa74b7b0 is not necessary but I feel like it's more correct since that string varies for each platform, if upstream changes it, we'll fail where we should succeed.

I don't know if there's a nicer way to handle this, criticism welcomed.

Edit: hash changed