Task #9522 (closed)
Opened 7 years ago
Closed 7 years ago
Clean up Sphinx Makefile
| Reported by: | bpindelski | Owned by: | rleigh |
|---|---|---|---|
| Priority: | minor | Milestone: | OMERO-4.4.4 |
| Component: | Documentation | Version: | n.a. |
| Keywords: | sphinx | Cc: | sbesson |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
Currently a lot of logic is in the 'html' make target.
Do we split the Unix/Windows? installation files into separate folders? If so, Makefile will be simpler.
Change History (9)
comment:1 Changed 7 years ago by bpindelski
- Keywords sphinx added
comment:2 Changed 7 years ago by rleigh
comment:3 Changed 7 years ago by sbesson
Blazej, with https://github.com/openmicroscopy/ome-documentation/pull/19 being merged, should we close this ticket?
comment:4 Changed 7 years ago by bpindelski
The proposed changes only fix part of the problem. The gist was - do we want to use tags (windows/unix) in the build? I guess this can be shelved for now, as not much time left to reorganize files.
comment:6 Changed 7 years ago by rleigh
That's fine. I'm a little unclear on what the alternative to the current logic would be though. It looks like the copying is necessary so long as the files are referenced through a toctree; include might allow the same sources to be used in-place, but at the expense of merging all the windows and unix files into single windows and unix install pages.
comment:7 Changed 7 years ago by jmoore
Ok, I still don't know what there is to actually do. If there's something specific, let's outline it and either finish or push. If not, we can close and come back to this when the time comes.
comment:8 Changed 7 years ago by bpindelski
Closing for now. Makefile looks clean for now. Once we figure out how to work with includes and maybe remove the toctree, we will look again at how the Windows/Unix? pages are split.
comment:9 Changed 7 years ago by bpindelski
- Resolution set to fixed
- Status changed from new to closed
See proposed fix here:
https://github.com/openmicroscopy/ome-documentation/pull/19