Skip to content

Store the executable in /usr/local/bin #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 21, 2023
Merged

Conversation

bowlofeggs
Copy link

/usr/bin is not intended to be used for locally installed software.
/usr/local/bin is a more appropriate place to default to storing
locally installed scripts, as per FHS[0].

[0] https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s09.html

/usr/bin is not intended to be used for locally installed software.
/usr/local/bin is a more appropriate place to default to storing
locally installed scripts, as per FHS[0].

[0] https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s09.html
@bowlofeggs
Copy link
Author

FTR, the old location in /etc/openvpn/scripts was also suitable as per FHS. What was the reason it was moved out of there?

@bowlofeggs
Copy link
Author

Note that the .conf file installed by the Makefile is still installed to a strange place, even with this patch. /usr/bin and /usr/local/bin are intended for executables. This was less strange in the former location at /etc/openvpn/scripts.

@srisch
Copy link

srisch commented Jun 10, 2022

@bowlofeggs looks like it was moved in this PR #95 - with reference to #94 which doesn't seem to be an actual issue with the paths. So it also doesn't make sense why it was moved.

@tomeon
Copy link
Collaborator

tomeon commented Jul 19, 2023

[L]ooks like it was moved in this PR #95 - with reference to #94 which doesn't seem to be an actual issue with the paths. So it also doesn't make sense why it was moved.

I suspect the script path change mentioned in #94 was a downstream change, possibly having to do with the AUR package. This AUR package commit introduced the switch to /usr/bin and is dated 2021-04-27; #94 was opened 2022-02-03.


[T]he old location in /etc/openvpn/scripts was also suitable as per FHS. What was the reason it was moved out of there?

Suspect it's a secondary(/tertiary/quaternary/...) effect of Arch Linux packaging policy, but (though it's not the actual reason the script was moved) there's also this comment from #96:

That's right, because the script is stored in /etc it's considered a settings file and changes/fixes won't get applied. :-(


Note that the .conf file installed by the Makefile is still installed to a strange place, even with this patch. /usr/bin and /usr/local/bin are intended for executables. This was less strange in the former location at /etc/openvpn/scripts.

Good point. Perhaps somewhere under /usr/share would be a good choice?

@bowlofeggs
Copy link
Author

There could be an /etc/update-systemd-resolved/ folder, which could be a suitable place for it since you probably don't want to claim paths in the /etc/openvpn namespace. System-wide configuration should generally be in /etc.

@tomeon tomeon merged commit cfdc3c4 into jonathanio:master Jul 21, 2023
@tomeon tomeon mentioned this pull request Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants