Skip to content

Manual build#890

Merged
ZedThree merged 5 commits intonextfrom
manual_build
Feb 26, 2018
Merged

Manual build#890
ZedThree merged 5 commits intonextfrom
manual_build

Conversation

@dschwoerer
Copy link
Contributor

Note that the PYTHONPATH needs to be set that something like .. automodule:: works (and that the documentation is generated for $BOUT_TOP rather than some installed version)

bendudson
bendudson previously approved these changes Feb 24, 2018
manual/Makefile Outdated

sphinx-man:
PYTHONPATH=$(BOUT_TOP)/tools/pylib:$$PYTHONPATH $(sphinx-build) -b man sphinx/ man/
@echo "Documentation available in `pwd`/html/"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: html -> man

As TEXDIR is not decleared, this will cd into your home directory and
delete (among others) all pdfs!
@dschwoerer
Copy link
Contributor Author

I found this really ugly and dangerous bug - be careful with cleaning!

@dschwoerer
Copy link
Contributor Author

Was introduced by 90aa604 18 days ago ...
I have backups - so no harm done ...
Should be merged ASAP - if preferred I can make a new PR just for this ...

@ZedThree
Copy link
Member

Will be merged as soon as the tests pass 👍

@ZedThree ZedThree merged commit e932553 into next Feb 26, 2018
@ZedThree ZedThree deleted the manual_build branch February 26, 2018 17:19
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.

4 participants