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

Define compiler env vars for CMake in dev containers. #576

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

alliepiper
Copy link
Collaborator

Description

closes #575

Adds compilers to the devcontainer environments.

@alliepiper alliepiper requested review from a team as code owners October 17, 2023 15:18
@alliepiper alliepiper requested review from jarmak-nv and miscco and removed request for a team October 17, 2023 15:18
@copy-pr-bot
Copy link

copy-pr-bot bot commented Oct 17, 2023

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Collaborator

@gevtushenko gevtushenko left a comment

Choose a reason for hiding this comment

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

Build through the CMake extension works now, thanks! One minor note is that it's using make as generator. Any chance we could set CMAKE_GENERATOR to Ninja in devcontainers?

@alliepiper
Copy link
Collaborator Author

Build through the CMake extension works now, thanks! One minor note is that it's using make as generator. Any chance we could set CMAKE_GENERATOR to Ninja in devcontainers?

Strange...the presets are hardcoded to use ninja currently. I'll look into this.

@gevtushenko
Copy link
Collaborator

My bad, Ninja is used indeed!

@alliepiper alliepiper merged commit dd2abc4 into NVIDIA:main Oct 18, 2023
464 checks passed
@alliepiper alliepiper deleted the devcontainer-cmake-env-vars branch October 18, 2023 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[BUG]: Using CMake presets in a devcontainer does not properly configure compilers.
3 participants