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

Unique WARN_LOGFILE for documentation builds #1589

Closed
skliper opened this issue May 28, 2021 · 0 comments · Fixed by #1612
Closed

Unique WARN_LOGFILE for documentation builds #1589

skliper opened this issue May 28, 2021 · 0 comments · Fixed by #1612
Assignees
@jphickey
Labels
docs This change only affects documentation.
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented May 28, 2021

Is your feature request related to a problem? Please describe.
WARN_LOGFILE is set the same for both cfe documentation builds since it's set in cfe-common.doxyfile.in. Potential conflict during a parallel build.

Describe the solution you'd like
Set unique warning file names for usersguide and doc build.

Describe alternatives you've considered
None

Additional context
#1378 would also deconflict, but would deconflict CI artifacts (no longer would need to rename in related Action). This was mentioned in #389, but updated to make specific to the undocumented warnings.

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added the docs This change only affects documentation. label May 28, 2021
@skliper skliper added this to the 7.0.0 milestone May 28, 2021
@skliper skliper mentioned this issue May 28, 2021
Open
jphickey added a commit to jphickey/cFE that referenced this issue Jun 8, 2021
@jphickey
Fix nasa#1589, Separate warning file for doc builds
24ddaf7 
Use a unique file name for the doxygen warnings, instead
of "warnings.log", so the mutiple documentation targets
will not overwrite the other log file.

Use a prefixed name so that if the log file is uploaded
to the build artifacts, it will be identifiable.

Detail Design warnings in: detaildesign-warnings.log
Users Guide warnings in: cfe-usersguide-warnings.log
@jphickey jphickey mentioned this issue Jun 8, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
docs This change only affects documentation.
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #1378, 1589, doxygen cleanup jphickey/cFE
2 participants
@jphickey @skliper