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.

Sign up for GitHub

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

Jump to bottom

Add install-nodoc target #153

Closed
heplesser opened this issue Nov 10, 2015 · 3 comments · Fixed by #890
Closed

Add install-nodoc target #153

heplesser opened this issue Nov 10, 2015 · 3 comments · Fixed by #890
Assignees
@jougs
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. ZC: Installation DO NOT USE THIS LABEL ZP: In progess DO NOT USE THIS LABEL

Comments

@heplesser
Copy link
Contributor

Installation of NEST online help takes an irritating amount of time during development work, when one is frequently recompiling and reinstalling. As long as one only changes C++ code or PyNEST Python code, one can use make install-exec to skip installation of documentation, but this also skips installation of SLI libraries, the testsuite and examples. Therefore, we should add an install-nodoc target that installs everything except the online documentation.

See also #152 .
@heplesser heplesser mentioned this issue Nov 10, 2015
Closed
@steffengraber
Copy link
Contributor

See my comment in #152.

@heplesser
Copy link
Contributor Author

@steffengraber Given that your new solution is blazingly fast, I think we can just drop this ticket to keep complexity within bounds. This issue should be closed when #157 is merged.

@heplesser
Copy link
Contributor Author

After some more experience, I think we still need this feature. Building and installing the documentation under macOS can be slow, in particular when recompiling often.

@heplesser heplesser added ZC: Installation DO NOT USE THIS LABEL I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) ZP: In progess DO NOT USE THIS LABEL S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. labels Jun 29, 2017
jougs added a commit to jougs/nest-simulator that referenced this issue Feb 6, 2018
@jougs
Add install-nodoc target as requested in nest#153
d28e4b0
@jougs jougs mentioned this issue Feb 6, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jougs jougs

Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. ZC: Installation DO NOT USE THIS LABEL ZP: In progess DO NOT USE THIS LABEL
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add 'install-nodoc' target for make jougs/nest-simulator
3 participants
@heplesser @steffengraber @jougs