Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 356133 - Featurerequest: media-tv/gentoo-vdr-scripts provide sudoers.d for shutdown
Summary: Featurerequest: media-tv/gentoo-vdr-scripts provide sudoers.d for shutdown
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Gentoo VDR Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-22 21:18 UTC by WOLfgang Schricker
Modified: 2012-02-24 04:49 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
pre-start-50-shutdown.sh.sudoers.patch (pre-start-50-shutdown.sh.sudoers.patch,773 bytes, patch)
2011-02-22 21:21 UTC, WOLfgang Schricker
Details | Diff
alternate patch to pre-start-50-shutdown.sh (gentoo-vdr-scripts-0.4.9_sudoers-warning.patch,1.11 KB, patch)
2011-11-02 20:36 UTC, Lucian Muresan
Details | Diff
ebuild patch sudoers warning and epatch_user (gentoo-vdr-scripts-0.4.9.ebuild_sudoers+epatch_user.diff,1.36 KB, patch)
2011-11-02 20:38 UTC, Lucian Muresan
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description WOLfgang Schricker 2011-02-22 21:18:50 UTC
For some time reads 'app-admin/sudo' drop-in files from /etc/sudoers.d. My patch provides a file /etc/sudoers.d/vdr - see attachment...
Comment 1 WOLfgang Schricker 2011-02-22 21:21:26 UTC
Created attachment 263563 [details, diff]
pre-start-50-shutdown.sh.sudoers.patch
Comment 2 Joerg Bornkessel (RETIRED) gentoo-dev 2011-02-26 13:11:42 UTC
Never i will touch the stable 0.4.6 for featurerequests!

changed/fixed the "Summary" line
maybe it is a candidat for next version of gentoo-vdr-scripts,
need more info for the /etc/soudoers.d handling,
looks like gentoo isn't support this from default,
how should be the (default)content from the /etc/sudoers.d/vdr file then?
Comment 3 Lucian Muresan 2011-11-02 20:31:48 UTC
(In reply to comment #2)
> Never i will touch the stable 0.4.6 for featurerequests!
> 
> changed/fixed the "Summary" line
> maybe it is a candidat for next version of gentoo-vdr-scripts,
> need more info for the /etc/soudoers.d handling,
> looks like gentoo isn't support this from default,
> how should be the (default)content from the /etc/sudoers.d/vdr file then?

Just like this is enough, actually what the gentoo-vdr-scripts add in /etc/sudoers normally:

vdr ALL=NOPASSWD:/usr/share/vdr/bin/vdrshutdown-really.sh

I just stumbled upon this bug now when we are at gentoo-vdr-scripts-0.4.9 already and wanted only to get rid of the message from the ebuild and pre-start-50-shutdown.sh. My patch does basically the same in pre-start-50-shutdown.sh, the point is, if the relevant entry is present anywhere in a file read by sudo, the shutdown scripts will just work.
Comment 4 Lucian Muresan 2011-11-02 20:36:19 UTC
Created attachment 291549 [details, diff]
alternate patch to pre-start-50-shutdown.sh

This variant does not demand that the file under /etc/sudoers.d is necessarily called "vdr", it could be called anything as long as it contains the magic entry.
Comment 5 Lucian Muresan 2011-11-02 20:38:43 UTC
Created attachment 291551 [details, diff]
ebuild patch sudoers warning and epatch_user

Joerg, please consider also adapting the warning in the ebuild, and also calling epatch_user like in this patch, in a future version.
Comment 6 Joerg Bornkessel (RETIRED) gentoo-dev 2012-02-24 04:49:19 UTC
added to

gentoo-vdr-scripts-0.5.1

entry in /etc/sudoers is depricated

emerge --config gentoo-vdr-scripts

needed!