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 CMakePresets.json #4362

Closed
wants to merge 2 commits into from
Closed

Conversation

dinomight
Copy link
Contributor

Just starting off with the entries for Visual Studio 2022 to make it simpler to work in Visual Studio environment.

Just starting off with the entries for Visual Studio 2022 to make it
simpler to work in Visual Studio environment.
@dinomight
Copy link
Contributor Author

I just wanted to start a conversation on this as this is what I needed to do so I could check the named arg bug out on VS. There are several ways to tweak this, but I went with some of the default stuff VS would do. Let me know if you'd like to see this setup differently. If I can get a working GCC environment, I can also work on some configs for the Linux side and possibly create entries for all the build configs that are present in the PR check process.

@vitaut
Copy link
Contributor

vitaut commented Feb 26, 2025

Thanks for the PR but I don't see much value in providing specific presets that will likely be irrelevant to most users.

@vitaut vitaut closed this Feb 26, 2025
@dinomight
Copy link
Contributor Author

Thanks for the PR but I don't see much value in providing specific presets that will likely be irrelevant to most users.

What do you mean specific presets? These are mirroring the Github actions configurations and rely on whatever CXX is in the environment to select the compiler. It helps to more easily create the build environments for folks and helps for trying to replicate a PR action failure. I don't think I put anything in here that is specific to my environment and presets are relied upon by tools like Visual Studio/CLion to setup the build environment and reduce the amount of command line arguments for command line users. Essentially, this should be interesting to every user.

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