Skip to content

Doc to use CMakePresets and integration in CLion#3052

Draft
JasonMarechal25 wants to merge 11 commits intodevelopfrom
doc/cmake_presets
Draft

Doc to use CMakePresets and integration in CLion#3052
JasonMarechal25 wants to merge 11 commits intodevelopfrom
doc/cmake_presets

Conversation

@JasonMarechal25
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
Signed-off-by: Jason Marechal <jason.marechal+github@gmail.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 8, 2025

Comment thread src/CMakeUserPresetsExample.json Outdated
"CMAKE_CXX_FLAGS": "-fclang-abi-compat=17",
"CMAKE_INSTALL_PREFIX": "/tmp/deps/install_simulator_debug_docker",
"CODE_COVERAGE": "OFF"
//"CMAKE_CXX_INCLUDE_WHAT_YOU_USE": "include-what-you-use"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems to be invalid JSON format (github highlights it as such)

Comment thread src/CMakeUserPresetsExample.json Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@flomnes flomnes left a comment

Choose a reason for hiding this comment

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

It would be nice to make presets for our CI builds, so we can reproduce CI builds more easily


```sh
cd docker/clang
docker build -t antares/clang:latest .
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I couldn't find this image on dockerhub, does it exist ?

### 2.4. Drawbacks and advantages

- **Advantage:** You keep your work and dependencies persistent, and can use your favorite editor on the host.
- **Drawback:** File permissions and line endings may differ between host and container.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, that's a major pain point. Last time I tried all files were owned by root on the host machine

Comment on lines +121 to +122
> **Note:** CLion requires the build directory to be inside the source folder. While it is cleaner to keep build output
> outside (e.g., `/tmp/build`), you must ensure the build directory is within the source tree for full IDE integration.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do symbolic links work ? e.g Antares_Simulator/build -> /tmp/build

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

CMakePresets.json ?

@JasonMarechal25 JasonMarechal25 marked this pull request as draft November 24, 2025 09:07
@flomnes flomnes added the stale Pull request has been inactive for an extended period label Apr 23, 2026
@flomnes
Copy link
Copy Markdown
Member

flomnes commented Apr 23, 2026

Hey @JasonMarechal25, is this still relevant?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L stale Pull request has been inactive for an extended period

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants