| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6d48bd21ac Tutorial: Clarify the expected change to target_include_directories
4d56db038d Tutorial: Make `INSTALL_DESTINATION` consistent with installed destination
da22b269b2 Tutorial: Add missing `$` in generator expression
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !9120
|
| |
| |
| |
| |
| | |
The expectation is to modify the existing `target_include_directories`,
not add add a new `target_include_directories` command.
|
| | |
|
| | |
|
|/
|
|
|
|
|
| |
- TODO 6 has to be Link SqrtLibrary
- TODO 7 has to be Link MathFunctions
Signed-off-by: Bhavith C <bhavithc.acharya@gmail.com>
|
|
|
|
|
|
|
| |
* Fix typo in Step 2
* Fix incorrect code referenced in Step 3, TODO 7. Update comments in
MathFunctions/CMakeLists.txt to unique strings.
* Resolve inconsistencies between steps in MathFunctions/CMakeLists.txt
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
b7704e8818 Help/guide/importing-exporting: Update line numbers in prose
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !8691
|
| |
| |
| |
| |
| |
| | |
These were missed in commit aef057d0e8 (Help/guide: fix
MathFunctionsTargets.cmake path in importing-exporting, 2023-05-26,
v3.27.0-rc1~39^2) when a line was added to the example code.
|
| | |
|
| | |
|
| | |
|
|/ |
|
|
|
|
|
| |
The solution for step 7 todo 5 did not include all of the code changes.
Modify it to include the needed `#endif` code change.
|
|
|
|
|
|
|
| |
It is using ${CMAKE_INSTALL_LIBDIR} and an extra "/MathFunctions" path
suffix since these commits:
- e58a76de51 Help/guide: fix importing-exporting CMake install destinations
- 26cf661b64 Help/guide: use GNUInstallDirs in importing-exporting example
|
|
|
|
|
| |
During the training, we talk about INTERFACE libraries just before Step3
so it makes more sense to do the exercise related to them in Step3
|
|\
| |
| |
| |
| |
| |
| |
| | |
e1f2b35723 Tutorial: Refactor MakeTable commands into MakeTable.cmake
Acked-by: Kitware Robot <kwrobot@kitware.com>
Tested-by: buildbot <buildbot@kitware.com>
Merge-request: !8249
|
| | |
|
|/ |
|
|
|
|
|
|
|
| |
Propagate the refactor in Step 10 MathFunctions through all of the
steps. Use MathFunctions/MathFunctions.cxx instead of Tutorial.cxx
to determine which sqrt library is called. Adds .h files which
correspond to their .cxx files by name.
|
|
|
|
|
|
| |
The previous Catch2 v2.13.4 does not compile on some platforms [1].
[1] https://github.com/catchorg/Catch2/issues/2326
|
| |
|
|
|
|
|
| |
The solution in Step 11 sets POSITION_INDEPENDENT_CODE to
`${BUILD_SHARED_LIBS}`, rather than simply to "True" in all cases.
|
|
|
|
|
| |
The link in Step 6 (to "Testing Support") clearly refers to Exercise 2.
Fix the location to which the link jumps.
|
|
|
|
|
| |
If you look at the solution for Step 6, you will see `0.0001` rather
than `0.00001` used for the last test case. Fix the typo.
|
| |
|
| |
|
|
|
|
|
| |
When mention execitable names use Sphinx semantic role `:program`
instead of ``executable`` or **executable**.
|
|\
| |
| |
| |
| |
| |
| |
| | |
f9ef15f3dc Tutorial: Update step 7 style
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7787
|
| | |
|
|/
|
|
|
|
| |
Since commit 6e58a01f4e (Tutorial: Update step 6 style, 2022-10-06) the
`literalinclude` included an extra comment from step 6 instructions.
Fix the include bounds to remove the extra text.
|
|\
| |
| |
| |
| |
| |
| |
| | |
6e58a01f4e Tutorial: Update step 6 style
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7756
|
| | |
|
| |
| |
| |
| |
| |
| | |
In commit 80f5d28813 (Tutorial: Update step 2 style, 2022-07-25,
v3.25.0-rc1~226^2) we replaced some uses of `USE_MYMATH` with `MY_MATH`.
Restore the former name for consistency with the rest of the tutorial.
|
| | |
|
|/
|
|
| |
`CMAKE_CXX_REQUIRED_STANDARD` => `CMAKE_CXX_STANDARD_REQUIRED`
|
|\
| |
| |
| |
| |
| |
| |
| | |
92edbce342 Tutorial: Update step 5 style
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7728
|
| | |
|
|/
|
|
|
|
| |
Define the entry point to each mode as an option for the `cmake`
program, but reference the options for that mode as part of stand-in
`cmake-<mode>` programs.
|
|\
| |
| |
| |
| |
| |
| |
| | |
37fceb6fd3 Tutorial: Update step 4 style
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7646
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
Augment vague mentions of "Transitive Usage Requirements" with links to
a detailed explanation.
Fixes: #23799
|
|/
|
|
| |
Fixes: #23918
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| | |
ccba87b05b Tutorial: Move step 10 to step 4
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7506
|
| |
| |
| |
| | |
Shifts steps 4-9 to 5-10.
|