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

Cmake updates #1295

Merged
merged 3 commits into from
May 16, 2024
Merged

Cmake updates #1295

merged 3 commits into from
May 16, 2024

Conversation

mathstuf
Copy link
Contributor

@mathstuf mathstuf commented May 7, 2024

Copy link
Member

@mattgodbolt mattgodbolt left a comment

Choose a reason for hiding this comment

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

Thanks! I'm sorry we don't have good documentation here, but the symlink thing is purely a workaround for the site's one "cmake" used for building programs. Can you please remove the other symlinks? And the ninja here seems unrelated and unused - if it's necessary can you link it somewhere or explain why it's needed? thanks!

bin/yaml/tools.yaml Outdated Show resolved Hide resolved
bin/yaml/tools.yaml Outdated Show resolved Hide resolved
bin/yaml/tools.yaml Outdated Show resolved Hide resolved
bin/yaml/tools.yaml Outdated Show resolved Hide resolved
This allows for stable CMake configurations to get patch updates.
This includes support for `import std`.
Copy link
Member

@mattgodbolt mattgodbolt left a comment

Choose a reason for hiding this comment

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

Thank you!

@mattgodbolt mattgodbolt merged commit 9ea67b7 into compiler-explorer:main May 16, 2024
3 checks passed
@mathstuf mathstuf deleted the cmake-updates branch May 18, 2024 12:13
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.

None yet

2 participants