Skip to content
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.

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

Add support for CMake presets #603

Merged
merged 8 commits into from
May 28, 2024

Conversation

uilianries
Copy link
Contributor

The CMake presets is an useful feature to avoid passing a bunch of parameters when building the project, instead, only need the preset name.

As result, we can simplify the CI by a lot, running only few commands and not managing folders anymore.

Plus, I was able to run Ninja in Windows build, which is faster than regular Visual Studio generator.

TODO: Update documentation in order to explain how to use CMake preset with faker-cxx

Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Copy link
Owner

@cieslarmichal cieslarmichal left a comment

Choose a reason for hiding this comment

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

Wow great! Did not know about these :)

@cieslarmichal cieslarmichal merged commit 3aa05ba into cieslarmichal:main May 28, 2024
6 checks passed
00thirdeye00 pushed a commit to 00thirdeye00/faker-cxx that referenced this pull request Jun 20, 2024
* Use CMake presets

Signed-off-by: Uilian Ries <[email protected]>

* Build debug

Signed-off-by: Uilian Ries <[email protected]>

* Add msvc toolset

Signed-off-by: Uilian Ries <[email protected]>

* remove explicit cl.exe

Signed-off-by: Uilian Ries <[email protected]>

* Use msvc generator

Signed-off-by: Uilian Ries <[email protected]>

* configure cl in path

Signed-off-by: Uilian Ries <[email protected]>

* fix preset name

Signed-off-by: Uilian Ries <[email protected]>

* fix preset name

Signed-off-by: Uilian Ries <[email protected]>

---------

Signed-off-by: Uilian Ries <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants