-
Notifications
You must be signed in to change notification settings - Fork 4
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
need for submodules is not explained in README #197
Comments
From the installation manual.
This automatically installs the submodules. Can you specify what is lacking about not explicitly stating that there are submodules? It is not needed if the listed installation instructions are used. |
I cloned the repo, because it was clear I was going to have to fix something, and ran setup.py. I also, I think, tried to run pip passing the directory. I definitely ended up with a "can't find Makefile.unx" error which was quite mysterious. I think instructions should help people building modified versions from local repos, not just building from unmodified github. I don't understand your reluctance to just straightforwardly state the requirements. Having that missing seems like an unnecessary barrier to use and contributions. I look at a readme to be for people using the source. It seems entirely fine to put installation instructions in INSTALL.md and have only a line or two in readme to say "see INSTALL.md for installation". |
Clear, I could add a small development chapter in the documentation which would state to use "git clone --recursive" etc and also provide a link to the ISA-L list of build requirements. I could link that from the README.
That's not an entirely accurate rendition of my position. I do not want to clutter the landing page with very detailed information that is irrelevant to most users. Having said that, your suggestion to provide detailed instructions linked from the README also works for me. I will put this on the to-do list. |
Great, thanks. I do understand the githubby landing page notion, even though I come from an earlier time :-) |
A recursive clone is now recommended on the README. |
I got hints from other PRs, but the need for submodules is not documented. This is with today's git develop.
The text was updated successfully, but these errors were encountered: