Opened 9 years ago
Last modified 7 years ago
#555 closed defect
build_sphinx.py needs cleanup — at Version 3
Reported by: | butler | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | SasView 4.1.1 |
Component: | SasView | Keywords: | |
Cc: | Work Package: | SasView Documentation |
Description (last modified by butler)
The build module should properly cleanout everything before rebuilding. Also if a destination directory does not exist it should be created. Currently there are several instances of checking for existence but failing to create if not true — the script just goes on happily to next if statement leaving no message of why the end product cannot build properly. It should also exclude irrelevant files from the document scrapping (such as setup.exe and setup.mac). Finally, the current build script for 4.0 copies down the pre-assembled .rst model help files from sasmodels package so that one must make sure to *first* build the sasmodels docs before building the SasView docs in order for integrated documentation to be built correctly. In the future this should not be a required step every time one builds the SasView docs.
There may be other things as well which should be added to this ticket as they become known - note that sasmodel package specific issues should be reported in separate tickets.
Change History (3)
comment:1 Changed 9 years ago by butler
- Description modified (diff)
comment:2 Changed 9 years ago by smk78
comment:3 Changed 9 years ago by butler
- Description modified (diff)
I think this ticket is referring to a number of tests I put in whilst trying to integrate the sasmodels docs. Trying to figure out paths was a nightmare!
Yes the build is ugly now. But the relevant build folders were/are cleaned out before the build, and I haven't seen any builds failing because of a missing folder. Do you have an example???
I've no idea why setup_exe and setup_mac are being touched.
But as #554 says there needs to a more strategic overview of how things are arranged and sorting out build_sphinx will be part of that.
So I kinda think this ticket is redundant?