| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| | |
|
| |
|
|
|
|
|
| |
The corresponding `pre-commit` configuration is coming in the
separate MR.
Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Run the `clang-format.bash` script to update all our C and C++ code to a
new style defined by `.clang-format`, now with "east const" enforcement.
Use `clang-format` version 18.
* If you reached this commit for a line in `git blame`, re-run the blame
operation starting at the parent of this commit to see older history
for the content.
* See the parent commit for instructions to rebase a change across this
style transition commit.
Issue: #26123
|
| |
|
|
| |
Exercise 3 requires editing `TutorialConfig.h.in`.
|
| |
|
|
|
|
|
| |
Restore a comment added by commit f9ef15f3dc (Tutorial: Update step 7
style, 2022-10-07, v3.26.0-rc1~482^2) that was accidentally removed by
commit 8ddf32196c (Tutorial: Refactor MathFunctions code, 2023-02-10,
v3.27.0-rc1~441^2).
|
| |
|
|
| |
Fix and improve wording in the "Adding Export Configuration" step.
|
| |
|
|
| |
In addition to CPACK_SOURCE_GENERATOR also set CPACK_GENERATOR.
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
The solution for step 7 todo 5 did not include all of the code changes.
Modify it to include the needed `#endif` code change.
|
| |
|
|
|
| |
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
|
| | |
|
| |
|
|
|
|
|
| |
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 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`
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | | |
|