Skip to content
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

allow running from any path #124

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dieterbe
Copy link

this allows the bash files to live anywhere.
we could actually even get rid of the "running locally" case, and always use the XDG config directory, which seems more clean to me.

how does this look and sound?

@Dieterbe Dieterbe force-pushed the support-any-installation-path branch from 2225a77 to 9eda55b Compare November 21, 2021 20:55
@CyberShadow
Copy link
Owner

Unfortunately I think this will break aconfmgr packages, at least in the way that it is currently packaged (the main script in /usr/bin and the rest elsewhere).

Perhaps it would make more sense to instead change the if condition and the first part of the if body respectively?

we could actually even get rid of the "running locally" case, and always use the XDG config directory, which seems more clean to me.

I think using the configuration from the current directory makes more sense in some circumstances, like when setting up a new system (you would clone aconfmgr and then your configuration).

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.

2 participants