Use XDG base directory spec

[KaiWaldrant]

change viash install directory

Close #92

[github-actions]

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-12-04 09:22 UTC

[Grifs]

Giving users instructions to mess with their /usr/... folders is risky in my opinion and in general considered bad practice to do so (the fiddling, not per se the instructing).

• This requires the user to have root access
• This folder is generally managed by package managers, and while I don't think we'll likely override anything, we may still do something odd? [this thought might be wrong/outdated]
• This is very distro dependant, ie. the expected folder might not be /usr/local/bin [this thought might be wrong/outdated]
• The more correct way would be to user /opt, but maybe there is a distro that also doesn't follow that topology? [I guess this is more for bigger packages, not single executables]

tl;dr it's not my prefered way, but it's a personal opinion. However, the user does need to have root access.

[Grifs]

Wasn't the reason the original command used curl that macOS doesn't necessarily have wget but was sure to have curl?

[rcannood]

Giving users instructions to mess with their /usr/... folders is risky in my opinion and in general considered bad practice to do so (the fiddling, not per se the instructing).

I agree, though I suggested installing it in /usr/local/bin because $HOME/bin and $HOME/.local/bin are not on the PATH out of the box. Then again, we could just have separate installation instructions for Mac OS X.

[rcannood]

Oh, I know, I know!

set +e; IFS=":"; for dir in $PATH; do cp viash $dir; done

?

[Grifs]

I merged main in the PR, with all 3 options moving viash to /usr/local/bin, but with using curl for macOS. However I tweaked the wording ever so slightly.

[rcannood]

LGTM! Thanks!