Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 296087 - asterisk-1.6.1.11.ebuild.patch fixing permissions and rights in /etc/asterisk
Summary: asterisk-1.6.1.11.ebuild.patch fixing permissions and rights in /etc/asterisk
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Tony Vroon (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-07 15:51 UTC by Patryk Rzadzinski
Modified: 2009-12-19 18:59 UTC (History)
1 user (show)

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

Attachments
asterisk-1.6.1.11.ebuild patch (asterisk-1.6.1.11.ebuild.patch,489 bytes, patch)
2009-12-07 15:52 UTC, Patryk Rzadzinski 		Details | Diff
update for the patch before to use fperms and fowners (asterisk-1.6.1.11.ebuild.fperms.patch,594 bytes, patch)
2009-12-07 17:11 UTC, Patryk Rzadzinski 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Patryk Rzadzinski 2009-12-07 15:51:22 UTC 
As discussed with mr Chainsaw, please see this little patch for the ebuild which just adds setting the ownership of asterisk and permissions of 660 to the /etc/asterisk config directory.

Reproducible: Always
Comment 1 Patryk Rzadzinski 2009-12-07 15:52:56 UTC 
Created attachment 212352 [details, diff]
asterisk-1.6.1.11.ebuild patch
Comment 2 Dawid Węgliński (RETIRED) gentoo-dev 2009-12-07 16:02:17 UTC 
Why not fperms/fowners?
Comment 3 Patryk Rzadzinski 2009-12-07 17:11:03 UTC 
Created attachment 212355 [details, diff]
update for the patch before to use fperms and fowners
Comment 4 Patryk Rzadzinski 2009-12-07 17:11:34 UTC 
Updated with fperms/fowners.
Comment 5 Patryk Rzadzinski 2009-12-08 10:53:22 UTC 
Actually, the ebuild had an attempt to do that - which can now be removed:

        ebegin "Fixing up permissions"
                chown -R asterisk:asterisk "${ROOT}"var/log/asterisk
                chmod -R u=rwX,g=rX,o=     "${ROOT}"var/log/asterisk

                chown asterisk:asterisk "${ROOT}"etc/asterisk/
                chown asterisk:asterisk "${ROOT}"etc/asterisk/*.adsi
                chown asterisk:asterisk "${ROOT}"etc/asterisk/extensions.ael
                chmod u=rwX,g=rwX,o= "${ROOT}"etc/asterisk/
                chmod u=rwX,g=rwX,o= "${ROOT}"etc/asterisk/*.adsi
                chmod u=rwX,g=rwX,o= "${ROOT}"etc/asterisk/extensions.ael
        eend $?

This code in pkg_postinst() can now be removed from the ebuild.

Patryk
Comment 6 Tony Vroon (RETIRED) gentoo-dev 2009-12-19 18:59:44 UTC 
+*asterisk-1.6.1.12 (19 Dec 2009)
+
+  19 Dec 2009; <chainsaw@gentoo.org>
+  +files/1.6.1/asterisk-1.6.1.12-longer-sip-cid.patch,
+  +files/1.6.1/asterisk.initd2, +asterisk-1.6.1.12.ebuild,
+  +files/1.6.1/asterisk-1.6.1.12-parallel-make.patch:
+  Version bump. Improved permission handling in ebuild & initscript by
+  Patryk Rzadzinski, closes bugs #294601 & #296087. Set ASTLDFLAGS
+  environment variable to enforce LDFLAGS respect. Additional parallel make
+  fixes.