Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527796 - net-misc/asterisk should only use RDEPEND for SELinux policy package dependency
Summary: net-misc/asterisk should only use RDEPEND for SELinux policy package dependency
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Tony Vroon (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527698
  Show dependency tree
 
Reported: 2014-11-01 11:30 UTC by Sven Vermeulen (RETIRED)
Modified: 2014-11-02 09:02 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Vermeulen (RETIRED) gentoo-dev 2014-11-01 11:30:54 UTC
Please remove the "selinux? ( sec-policy/selinux-* )" dependency in the DEPEND variable (not RDEPEND). Take this into account with your next package bump - if after 6 months there have been no need for other package upgrades, we will be bumping the package(s) ourselves with just this change.

For more information, see our tracker Bug #527698 [1]. The requested change will be posted as a separate comment/patch. Note that if the package also depends on libselinux, libselinux itself should still remain in DEPEND and RDEPEND.

[1] https://bugs.gentoo.org/show_bug.cgi?id=527698


Reproducible: Always
Comment 1 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-01 11:31:37 UTC
Index: asterisk-12.6.1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-misc/asterisk/asterisk-12.6.1.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 asterisk-12.6.1.ebuild
--- asterisk-12.6.1.ebuild      21 Oct 2014 09:41:28 -0000      1.1
+++ asterisk-12.6.1.ebuild      1 Nov 2014 11:30:19 -0000
@@ -30,7 +30,7 @@
 EPATCH_SUFFIX="patch"
 PATCHES=( "${WORKDIR}/asterisk-patchset" )
 
-RDEPEND="dev-db/sqlite:3
+CDEPEND="dev-db/sqlite:3
        dev-libs/popt
        dev-libs/jansson
        dev-libs/libxml2
@@ -63,21 +63,21 @@
        portaudio? ( media-libs/portaudio )
        postgres? ( dev-db/postgresql-base )
        radius? ( net-dialup/radiusclient-ng )
-       selinux? ( sec-policy/selinux-asterisk )
        snmp? ( net-analyzer/net-snmp )
        span? ( media-libs/spandsp )
        speex? ( media-libs/speex )
        srtp? ( net-libs/libsrtp )
        vorbis? ( media-libs/libvorbis )"
 
-DEPEND="${RDEPEND}
+DEPEND="${CDEPEND}
        !net-libs/openh323
        !net-libs/pjsip
        voicemail_storage_imap? ( virtual/imap-c-client )
        virtual/pkgconfig
 "
 
-RDEPEND="${RDEPEND}
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-asterisk )
        syslog? ( virtual/logger )"
 
 PDEPEND="net-misc/asterisk-core-sounds
Comment 2 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-02 09:02:42 UTC
After checking with a few colleague developers it seems that this change does not need a revision bump after all. As such, I will make the necessary changes to the packages myself.

My apologies for the noise.