Skip to content

update blt with new sphinx feature #1610

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

Open
chapman39 wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

update blt with new sphinx feature #1610

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
847a630
update blt with new sphinx feature
chapman39 May 29, 2026
4144027
Merge branch 'develop' into task/chapman39/update-blt
chapman39 Jun 2, 2026
3167f4e
update blt
chapman39 Jun 3, 2026
27ab44e
Merge branch 'task/chapman39/update-blt' of github.com:LLNL/smith int…
chapman39 Jun 3, 2026
2cde679
api changes
chapman39 Jun 3, 2026
6cdf019
display jobs in addition to pipeline for hubcast
chapman39 Jun 3, 2026
6e1a3a2
update blt final
chapman39 Jun 3, 2026
7f18e64
use smith_add_executable
chapman39 Jun 3, 2026
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
2 changes: 1 addition & 1 deletion .github/hubcast.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Repo:
# Set to [pipeline] for overall pipeline status only
# Set to [jobs] for individual job statuses only
# Set to [pipeline, jobs] to report both
check_types: [pipeline]
check_types: [pipeline, jobs]

# Optional: delete branches from destination when source PR is closed (default: true)
delete_closed: true
Expand Down
2 changes: 1 addition & 1 deletion cmake/blt
Open in desktop
Submodule blt updated 411 files
3 changes: 1 addition & 2 deletions src/docs/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,9 @@
# SPDX-License-Identifier: (BSD-3-Clause)

if (SPHINX_FOUND)
blt_add_sphinx_target(smith_docs)
blt_add_sphinx_target(TARGET smith_docs)
endif()

if (DOXYGEN_FOUND)
add_subdirectory(doxygen)
endif()

19 changes: 1 addition & 18 deletions src/docs/doxygen/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,23 +4,7 @@
#
# SPDX-License-Identifier: (BSD-3-Clause)

find_package(Doxygen COMPONENTS dot)

if(DOXYGEN_DOT_FOUND)
Copy link
Copy Markdown
Collaborator Author

@chapman39 chapman39 Jun 3, 2026

Choose a reason for hiding this comment

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

@tupek2 let me know why this was needed and we can look into re-adding this. blt handles setting up doxygen. and im not sure about the pdflatex thing.

set(DOXYGEN_HAVE_DOT "YES")
else()
set(DOXYGEN_HAVE_DOT "NO")
endif()

# Check for pdflatex for latex generation
find_program(PDFLATEX_EXECUTABLE pdflatex)
if(PDFLATEX_EXECUTABLE)
set(DOXYGEN_GENERATE_LATEX "YES")
else()
set(DOXYGEN_GENERATE_LATEX "NO")
endif()

blt_add_doxygen_target( smith_doxygen )
blt_add_doxygen_target(TARGET smith_doxygen)

# Required directory for doxygen to generate output to
add_custom_target( build_doxygen_output_dir
Expand All @@ -29,4 +13,3 @@ add_custom_target( build_doxygen_output_dir
)

add_dependencies( smith_doxygen build_doxygen_output_dir )

Loading