Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527752 - mail-client/thunderbird should only use RDEPEND for SELinux policy package dependency
Summary: mail-client/thunderbird should only use RDEPEND for SELinux policy package de...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527698
  Show dependency tree
 
Reported: 2014-11-01 10:38 UTC by Sven Vermeulen (RETIRED)
Modified: 2014-11-02 08:32 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 10:38:32 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 10:38:48 UTC
Index: thunderbird-31.2.0-r1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/mail-client/thunderbird/thunderbird-31.2.0-r1.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 thunderbird-31.2.0-r1.ebuild
--- thunderbird-31.2.0-r1.ebuild        20 Oct 2014 17:53:11 -0000      1.1
+++ thunderbird-31.2.0-r1.ebuild        1 Nov 2014 10:38:40 -0000
@@ -59,10 +59,9 @@
 
 ASM_DEPEND=">=dev-lang/yasm-1.1"
 
-RDEPEND="
+CDEPEND="
        >=dev-libs/nss-3.16.5
        >=dev-libs/nspr-4.10.6
-       selinux? ( sec-policy/selinux-thunderbird )
        !x11-plugins/enigmail
        crypt?  ( || (
                ( >=app-crypt/gnupg-2.0
@@ -73,8 +72,9 @@
                )
                =app-crypt/gnupg-1.4*
        ) )"
-
-DEPEND="${RDEPEND}
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-thunderbird )"
+DEPEND="${CDEPEND}
        amd64? ( ${ASM_DEPEND}
                virtual/opengl )
        x86? ( ${ASM_DEPEND}
Comment 2 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-02 08:32:21 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.