fix: do not hard depend on sudo being installed

[plurpio]

Type of Change

• New feature
• Bug fix
• Documentation update
• Refactoring
• Hotfix
• Security patch
• UI/UX improvement

Description

the checkCommandRequirements function under checkEnv function in common-script.sh hard-depended on sudo, although doas could be picked as an $ESCALATION_TOOL.

Testing

I activated acouple of the scripts from with the UI, there was no error for not having sudo on my sudo-less system anymore.

Impact

Allows doas to properly be used as an escalation tool now.

Issues / other PRs related

• Similar to Add Multi-Escalation support #193 which introduced doas support.

Additional Information

Checklist

• My code adheres to the coding and style guidelines of the project.
• 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.
• My changes generate no errors/warnings/merge conflicts.

[plurpio]

It was not removed, just reordered.

[nnyyxxxx]

It was not removed, just reordered.

oops, was really tired when i typed that lol

[cartercanedy]

if we're starting to broach the topic of advanced privilege escalation tools, you should also look at refactoring this:

linutil/core/tabs/common-script.sh

Lines 91 to 107 in 466f875

	checkSuperUser() {
	## Check SuperUser Group
	SUPERUSERGROUP='wheel sudo root'
	for sug in ${SUPERUSERGROUP}; do
	if groups | grep -q "${sug}"; then
	SUGROUP=${sug}
	printf "%b\n" "${CYAN}Super user group ${SUGROUP}${RC}"
	break
	fi
	done
	## Check if member of the sudo group.
	if ! groups | grep -q "${SUGROUP}"; then
	printf "%b\n" "${RED}You need to be a member of the sudo group to run me!${RC}"
	exit 1
	fi
	}

doas might be configured to use wheel group over sudo group for privilege checks

[plurpio]

Both doas and sudo have commands for checking the current user's permission to run commands as root.

• sudo --validate
• doas -C /etc/doas.conf echo # requires a command and configuration file to be passed

[plurpio]

btw both of these need to be interactive so they aren't really an option.