Skip to content

Updated documentation generation scripts #810

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion doc/scripts/GenerateDocumentation.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,11 @@ In order to successfully run it, the additional library scripts `settings.sh`
and `utilities.sh` must also be in the same directory as the main script.
In addition, `experiment_settings.sh` overriding the general settings is also
parsed.

> **NOTE** This script uses the deployment system, currently UPS, to discover
> the version of all the repositories to pull in. UPS must be available in
> the system where the script is run, possibly using a container.

There are two possibilities:

* clone the GIT repository altogether, and find the scripts in
Expand All @@ -74,7 +79,7 @@ Note that the script *must* be executed from outside the `icaruscode` GIT
repository, like in the example above.

Normally this will clone or _update_ the repository, and run Doxygen on it.
Note that **"update"**: the script `updateLocalDocumentation.sh` **does** mess
Note the word "update": the script `updateLocalDocumentation.sh` **does** mess
up with the GIT repository, and while there is some provision not to lose data,
if that repository has anything valuable it should be backed up or pushed before
running.
Expand Down Expand Up @@ -133,6 +138,12 @@ web content directory; and before that, the node where this script is ran must
have POSIX (e.g. NFS) access to that directory. Enabling these is matter of
Fermilab service desk requests.

> **NOTE** If running in a container because of the requirements of the previous
> step, that container needs to bind the appropriate directory (the path is set
> by `PublishBaseDir` variable in `settings.sh`) in read/write mode.
> However, neither the publishing nor the version list update currently require
> access to UPS nor a container.

If the directory already exists, the script will refuse to proceed. This is
intentional, to prevent a potentially complex script from unintentionally
deleting information on the web site in case of errors or bugs.
Expand Down
2 changes: 1 addition & 1 deletion doc/scripts/settings.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ declare DefaultBranch='master'
declare RepoDocSubdir='doc'

# where the content is published
declare PublishBaseDir='/web/sites/i/icarus-exp.fnal.gov/htdocs/at_work/software/doc'
declare PublishBaseDir='/pubhosting/sites/i/icarus-exp.fnal.gov/htdocs/at_work/software/doc'

# directory of Doxygen generation metadata, relative to Doxygen output directory
declare MetadataFileRelPath='meta/doxygen'
Expand Down
2 changes: 1 addition & 1 deletion doc/scripts/updateLocalDocumentation.sh
100644 → 100755
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ function UpdateROOTtag() {
ROOTVersionTag="${ROOTVersionTag//.}"

echo "Fetching Doxygen tag for ROOT version ${ROOTVersion}"
UpdateTag 'ROOT.tag' "https://root.cern/doc/v${ROOTVersionTag}/ROOT.tag" "ROOT"
UpdateTag 'ROOT.tag' "https://root.cern/doc/v${ROOTVersionTag}/ROOT.tag.gz" "ROOT"

} # UpdateROOTtag()

Expand Down
2 changes: 1 addition & 1 deletion doc/scripts/utilities.sh
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ function ExperimentSetup() {
[[ -r "$SetupScript" ]] || FATAL 1 "Can't find setup script for experiment '${Experiment}'."

echo "Setting up the UPS area for ${Experiment}..."
source "$SetupScript"
source "$SetupScript" || LASTFATAL "Failed to set up the UPS area."

local -r LatestVersion=$(FindLatestUPS "$ExperimentCodeName")

Expand Down