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

configure: remove dependency on which #6

Closed
wants to merge 2 commits into from
Closed

Conversation

orbea
Copy link
Contributor

@orbea orbea commented Nov 10, 2023

I am unsure if you are interested, but this avoids the dependency on which and my exists function has a test suite fwiw.

https://notabug.org/orbea/exists

@orbea
Copy link
Contributor Author

orbea commented Nov 10, 2023

Added a commit to move the fail() function earlier to be set before it might be used.

@juur
Copy link
Owner

juur commented Nov 11, 2023

removing which means going for a portable file, or a portable app, which it isn't either. However, I think i've at least made configure portable now and used env/command as a vague check in file existing.

@juur juur closed this Nov 11, 2023
@orbea
Copy link
Contributor Author

orbea commented Nov 11, 2023

My goal was to remove a build dependency and while command -v probably works in most cases I am going to leave this more in depth explanation on the problems if you are interested.

https://unix.stackexchange.com/a/85250

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