Feat: Added nouveau driver option for leagacy NVIDIA GPUs. #1166
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Installs mesa & xf86-video-nouveau for older NVIDIA GPUs, defaults back to nvidia-lts for modern NVIDIA GPUs.
Type of Change
Description
This pull request addresses an issue from #1129 where the Arch Server setup fails on systems with older NVIDIA GPUs (e.g., Optimus configs like GT 220M). The original script installed the 'nvidia-lts' package, which is incompatible with legacy hardware, causing the system to hang indefinitely (according to #1129).
To resolve this, I added a user prompt within the graphics drivers installation section in '~/core/tabs/system-setup/arch/server-setup.sh' that prompts the user to select the type of NVIDA GPU drivers to install.
This change now enables people with older GPUs to install arch onto their machines via the 'Arch Server Setup'.
Testing
To validate the changes, I performed the following tests in a virtual machine environment:
server-setup.shscript in a fresh Arch Linux VM without GPU passthrough. The script completed successfully, and the new NVIDIA-specific logic was correctly skipped as expected.nouveaupath, the logic for the default path (installingnvidia-lts) remains the same and was verified to not be broken by the changes.The core change is the addition of a conditional check, which has been reviewed for correctness.
Impact
Issues / other PRs related
Additional Information
Checklist
cargo xtask docgen).