MycroftAI / snapcraft-mycroft-core

This project is for building mycroft-core snaps
MIT License
13 stars 11 forks source link

the sudo check function raises an exception in snap confinement #27

Open NoamDev opened 3 years ago

NoamDev commented 3 years ago

Describe the bug The _check_sudo function call 'sudo -n true' to check whether root access is available.

To Reproduce Install the mycroft snap. run mycroft. while installing skills the error will be raised.

Expected behavior In snap confinement it should return False anyhow, without running 'sudo -n true'

Actual behavior A PermissionDenied('sudo') error is raised. stopping skills from being installed.

See also MycroftAI/pako#12

krisgesling commented 3 years ago

Excellent, thanks for this - we'll get the fix pushed through to the Snap build as quick as we can.