[PR #2180] [CLOSED] Change sudo -v to sudo -n true for autoinstall to resolve #2177 #3526

New issue

Closed

opened 2026-02-27 03:54:49 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 03:54:49 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/GameServerManagers/LinuxGSM/pull/2180
Author: @Bourne-ID
Created: 2/5/2019
Status: ❌ Closed

Base: develop ← Head: develop

📝 Commits (8)

a187361 Fix Ctrl+C closing wrong session
545d3eb Upstream Merge
c5480f1 Change sudo -v to sudo -n true
0a932e5 Change sudo -v to sudo -n true
7ec16f0 Change sudo -v to sudo -n true
1c2c4da Change sudo -v to sudo -n true for autoinstall
898382a fix(autoinstall): sudo no longer prompts
f09d628 Merge remote-tracking branch 'origin/develop' into develop

📊 Changes

2 files changed (+16 additions, -4 deletions)

View changed files

📝 lgsm/functions/check_deps.sh (+15 -3)
📝 lgsm/functions/check_permissions.sh (+1 -1)

📄 Description

Description

sudo -v prompts when checking for sudo access even with NoPrompt set in sudoers, likewise it will prompt when in auto-install mode. See #2177 for more information as this pull request will ensure no prompt is displayed for Auto-Install, but will prompt if a user has the password set up for sudo in normal mode.

Fixes #2177

Type of change

Bug fix (non breaking change which fixes an issue)
New feature (non breaking change which adds functionality)
Refactor (non breaking change restructures existing code)
This change requires a documentation update

Checklist

My code follows the style guidelines of this project
This pull request links to an issue
I have performed a self review of my own code
I have commented my code, particularly in hard to understand areas
I have made corresponding changes to the documentation

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/GameServerManagers/LinuxGSM/pull/2180 **Author:** [@Bourne-ID](https://github.com/Bourne-ID) **Created:** 2/5/2019 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `develop` --- ### 📝 Commits (8) - [`a187361`](https://github.com/GameServerManagers/LinuxGSM/commit/a187361c8f014285da40af58733e07bc516a7088) Fix Ctrl+C closing wrong session - [`545d3eb`](https://github.com/GameServerManagers/LinuxGSM/commit/545d3eb1196a81969f45fd8e55c6b219ab48bb44) Upstream Merge - [`c5480f1`](https://github.com/GameServerManagers/LinuxGSM/commit/c5480f188622cd5098c5ab9ad337df41914e8c4a) Change sudo -v to sudo -n true - [`0a932e5`](https://github.com/GameServerManagers/LinuxGSM/commit/0a932e55643ea8fb0205de408d93369fce0b22c4) Change sudo -v to sudo -n true - [`7ec16f0`](https://github.com/GameServerManagers/LinuxGSM/commit/7ec16f0b5e7496591a443efea27f1ad23109fa8c) Change sudo -v to sudo -n true - [`1c2c4da`](https://github.com/GameServerManagers/LinuxGSM/commit/1c2c4daa04434d406e85854838e8419df55bc38b) Change sudo -v to sudo -n true for autoinstall - [`898382a`](https://github.com/GameServerManagers/LinuxGSM/commit/898382a6c677481e0d6bddc52d39bc6ae888b7ec) fix(autoinstall): sudo no longer prompts - [`f09d628`](https://github.com/GameServerManagers/LinuxGSM/commit/f09d62832efc701eecb6b596941feced5d037015) Merge remote-tracking branch 'origin/develop' into develop ### 📊 Changes **2 files changed** (+16 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `lgsm/functions/check_deps.sh` (+15 -3) 📝 `lgsm/functions/check_permissions.sh` (+1 -1) </details> ### 📄 Description # Description sudo -v prompts when checking for sudo access even with NoPrompt set in sudoers, likewise it will prompt when in auto-install mode. See #2177 for more information as this pull request will ensure no prompt is displayed for Auto-Install, but will prompt if a user has the password set up for sudo in normal mode. Fixes #2177 ## Type of change * [X] Bug fix (non breaking change which fixes an issue) * [ ] New feature (non breaking change which adds functionality) * [ ] Refactor (non breaking change restructures existing code) * [ ] This change requires a documentation update ## Checklist * [X] My code follows the style guidelines of this project * [X] This pull request links to an issue * [X] I have performed a self review of my own code * [X] I have commented my code, particularly in hard to understand areas * [ ] I have made corresponding changes to the documentation --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>