Skip to content

Commit

Permalink
Add News and Remove Debug Print
Browse files Browse the repository at this point in the history
  • Loading branch information
iguessthislldo committed Feb 29, 2024
1 parent 99c215d commit 2dc2560
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/news.d/cmake-cxx-std.rst
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.. news-prs: 4481
.. news-start-section: Fixes
.. news-start-section: Platform Support and Dependencies
.. news-start-section: Building with CMake
- Fixed :ghissue:`building with CMake and Apple Clang on macOS <4478>` without setting ``-DCMAKE_CXX_STANDARD=14`` or using ACE 6 by setting :cmake:var:`OPENDDS_CXX_STD` when ACE 7 is used.
.. news-end-section
Expand Down
7 changes: 7 additions & 0 deletions docs/news.d/cmake-include-base.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
.. news-prs: 4489
.. news-start-section: Fixes
.. news-start-section: CMake Config Package
- Made :cmake:func:`opendds_target_sources(INCLUDE_BASE)` work correctly in more cases, specifically involving generating an export header.
.. news-end-section
.. news-end-section
1 change: 0 additions & 1 deletion tests/cmake/idl_compiler_tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ function(assert_generated_output file)
set(expect_prefix_path "${expect_dir_path}/${filename_no_ext}")
foreach(var failed dir_path file_path prefix_path)
set(val "${${var}}")
# message("${var} ${val}")
set(expect "${expect_${var}}")
if(NOT val STREQUAL expect)
message(SEND_ERROR
Expand Down

0 comments on commit 2dc2560

Please sign in to comment.