Skip to content

remove regular gcc install on nvdriver template#377

Merged
ArangoGutierrez merged 1 commit intoNVIDIA:mainfrom
ArangoGutierrez:devel/v0212
May 30, 2025
Merged

remove regular gcc install on nvdriver template#377
ArangoGutierrez merged 1 commit intoNVIDIA:mainfrom
ArangoGutierrez:devel/v0212

Conversation

@ArangoGutierrez
Copy link
Collaborator

No description provided.

@ArangoGutierrez ArangoGutierrez self-assigned this May 30, 2025
@ArangoGutierrez ArangoGutierrez requested review from Copilot and removed request for cdesiniotis, elezar and tariq1890 May 30, 2025 08:38
Copy link

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.

Pull Request Overview

This PR removes the installation of the regular gcc package from the nv-driver template and relocates the installation of make to a different package group. Key changes include:

  • Removing "gcc make" from the linux-headers installation command.
  • Adding "make" to the subsequent package installation command alongside pkg-config.
  • Ensuring that gcc is only installed via gcc-12 later in the script.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
pkg/provisioner/templates/nv-driver_test.go Removed gcc and moved make installation to a different package group.
pkg/provisioner/templates/nv-driver.go Removed gcc and moved make installation to a different package group.

@ArangoGutierrez ArangoGutierrez changed the title remove regular gcc install on nvdriver temaplte remove regular gcc install on nvdriver template May 30, 2025
@ArangoGutierrez ArangoGutierrez force-pushed the devel/v0212 branch 2 times, most recently from eef8914 to bccaeeb Compare May 30, 2025 09:39
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
@ArangoGutierrez ArangoGutierrez merged commit f9958c5 into NVIDIA:main May 30, 2025
15 of 16 checks passed
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