search

zyedidia / eget

Easily install prebuilt binaries from GitHub.
MIT License
954 stars 39 forks source link

Remove some platform-specific suffixes #47

Closed nikolay closed 2 years ago

nikolay commented 2 years ago

I tried to install https://github.com/editorconfig-checker/editorconfig-checker and it worked, but the binary carries some suffixes, which could be stripped off automatically: ec-darwin-amd64, i.e. the proper executable must be just ec.

st0le commented 2 years ago

I have this issue but I think it warrants a discussion whether it's an eget issue. The owner of the repo needs to ensure that the binary is named what is expected.

That said, perhaps eget can have a flag that can "attempt" to rename files post extraction. Best effort ofcourse.

dufferzafar commented 2 years ago

Yeah, attempting auto-rename would be a challenge, as there isn't a set naming convention.

Having a manual flag for renaming sounds good though, I currently have to eget & then mv

zyedidia commented 2 years ago

There is the --to flag. For example eget editorconfig-checker/editorconfig-checker --to ec.

nikolay commented 2 years ago

@zyedidia I thought --to always required a directory though, but now, after you clarified it, it makes sense. Thanks!