PacketFence - BTS - PacketFence
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001674PacketFencepackagingpublic2013-07-29 15:522015-02-18 11:20
Reportermzagrabe 
Assigned Tolmunro 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionwon't fix 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
fixed in git revision
fixed in mtn revision
Summary0001674: debian postinst should install sudo commands to /etc/sudoers.d/packetfence, not /etc/sudoers
DescriptionGreetings,

sudo snippets for Debian can be placed in /etc/sudoers.d. It is the logical place and makes more sense than mangling /etc/sudoers.

The debian/packetfence.postinst should reference a file like:

/etc/sudoers.d/packetfence. The attached patch fixes this.

Thanks for packetfence.

-mz
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch packetfence_sudo.patch (2,055) 2013-07-29 15:52
https://www.packetfence.org/bugs/file_download.php?file_id=181&type=bug
Issue History
Date ModifiedUsernameFieldChange
2013-07-29 15:52mzagrabeNew Issue
2013-07-29 15:52mzagrabeFile Added: packetfence_sudo.patch
2013-07-31 19:32fdurandNote Added: 0003353
2015-02-18 11:20lmunroNote Added: 0003957
2015-02-18 11:20lmunroStatusnew => closed
2015-02-18 11:20lmunroAssigned To => lmunro
2015-02-18 11:20lmunroResolutionopen => won't fix

Notes
(0003353)
fdurand   
2013-07-31 19:32   
Hello,
it was there because on squeeze /etc/sudoers does not include /etc/sudoers.d.
So now we can place the sudo file inside this directory but i think we can do it by another way.

Like in rules file:

mkdir -p debian/packetfence/etc/sudoers.d
install -m 440 -o root -g root debian/packetfence.sudoers debian/packetfence/etc/sudoers.d/packetfence

Where packetfence.sudoers contain all the stuff for sudo and when you remove packetfence then this file will be remove too.

Regards
Fabrice
(0003957)
lmunro   
2015-02-18 11:20   
Moving to github issues.

Will be scheduled for inclusion in 5.0