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

Missing dependency: wget, git #33

Closed
MykalMachon opened this issue Apr 12, 2024 · 3 comments · Fixed by #35
Closed

Missing dependency: wget, git #33

MykalMachon opened this issue Apr 12, 2024 · 3 comments · Fixed by #35

Comments

@MykalMachon
Copy link

MykalMachon commented Apr 12, 2024

Summary of issue

I'm setting up a test Islandora site for evaluation on a RHEL 8 virtual machine. I was following the install guide, had installed the listed required tools, and ran the installer. A few prompts in, I received an error: environment: line 115: wget: command not found. This implies that you need wget on your system to run the installer. wget is not in the listed requirements in the README.md.

Does it make sense to list wget as an additional dependency in the README.md? I know it's a pretty standard tool but it's not bundled with every distro by default (RHEL 8 does not include it, for example). edit: git is also like this, it is required by this script but not listed anywhere.

I am more than happy to make a PR as this is a really simple change was just hoping for some input from the maintainers before doing so 😄

@MykalMachon MykalMachon changed the title Missing dependency: wget Missing dependency: wget, git Apr 15, 2024
@joecorall
Copy link
Contributor

Thanks for the report @MykalMachon! I just opened a PR to get these changes in.

@MykalMachon
Copy link
Author

Thank you @joecorall !

@joecorall
Copy link
Contributor

Sure thing. And pull requests are always welcome!

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 a pull request may close this issue.

2 participants