'make install' should require the caller to invoke sudo, not attempt to do so itself. #36

Open
opened 2026-07-07 17:14:47 +01:00 by danny · 1 comment
danny commented 2026-07-07 17:14:47 +01:00 (Migrated from gitlab.keyop.co.uk)

We should remove use of 'sudo' inside the Makefile, as the caller may not be able to use sudo without passwords and the result will be counter-intuitive. Instead add some guard checks that we already have the correct privileges when privileged targets are called. We will also need to ensure the environment is sane, since 'sudo' can modify several env vars including PATH.

We should remove use of 'sudo' inside the Makefile, as the caller may not be able to use sudo without passwords and the result will be counter-intuitive. Instead add some guard checks that we already have the correct privileges when privileged targets are called. We will also need to ensure the environment is sane, since 'sudo' can modify several env vars including `PATH`.
danny commented 2026-07-07 17:16:16 +01:00 (Migrated from gitlab.keyop.co.uk)

changed the description

changed the description
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
keyop-go/nfq_forwarder#36
No description provided.