Bug 527736 – dev-db/postgresql should only use RDEPEND for SELinux policy packages

• Product: Gentoo Linux
• Status: RESOLVED OBSOLETE

[Description Sven Vermeulen (RETIRED) 2014-11-01 10:09:17 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) 2014-11-01 10:10:45 UTC]

Index: postgresql-9.3.5.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/dev-db/postgresql/postgresql-9.3.5.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 postgresql-9.3.5.ebuild
--- postgresql-9.3.5.ebuild     11 Oct 2014 19:35:08 -0000      1.1
+++ postgresql-9.3.5.ebuild     1 Nov 2014 10:10:21 -0000
@@ -40,7 +40,7 @@
        echo -n ${enable_langs}
 }
 
-RDEPEND="
+CDEPEND="
 >=app-admin/eselect-postgresql-1.2.0
 sys-apps/less
 virtual/libintl
@@ -50,15 +50,15 @@
 perl? ( >=dev-lang/perl-5.8 )
 python? ( ${PYTHON_DEPS} )
 readline? ( sys-libs/readline )
-selinux? ( sec-policy/selinux-postgresql )
 ssl? ( >=dev-libs/openssl-0.9.6-r1 )
 tcl? ( >=dev-lang/tcl-8 )
 uuid? ( dev-libs/ossp-uuid )
 xml? ( dev-libs/libxml2 dev-libs/libxslt )
 zlib? ( sys-libs/zlib )
 "
-
-DEPEND="${RDEPEND}
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-postgresql )"
+DEPEND="${CDEPEND}
 !!dev-db/postgresql-docs:${SLOT}
 !!dev-db/postgresql-base:${SLOT}
 !!dev-db/postgresql-server:${SLOT}
Index: postgresql-9.4_beta3.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/dev-db/postgresql/postgresql-9.4_beta3.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 postgresql-9.4_beta3.ebuild
--- postgresql-9.4_beta3.ebuild 26 Oct 2014 02:22:46 -0000      1.1
+++ postgresql-9.4_beta3.ebuild 1 Nov 2014 10:10:21 -0000
@@ -41,7 +41,7 @@
        echo -n ${enable_langs}
 }
 
-RDEPEND="
+CDEPEND="
 >=app-admin/eselect-postgresql-1.2.0
 sys-apps/less
 virtual/libintl
@@ -51,15 +51,15 @@
 perl? ( >=dev-lang/perl-5.8 )
 python? ( ${PYTHON_DEPS} )
 readline? ( sys-libs/readline )
-selinux? ( sec-policy/selinux-postgresql )
 ssl? ( >=dev-libs/openssl-0.9.6-r1 )
 tcl? ( >=dev-lang/tcl-8 )
 uuid? ( dev-libs/ossp-uuid )
 xml? ( dev-libs/libxml2 dev-libs/libxslt )
 zlib? ( sys-libs/zlib )
 "
-
-DEPEND="${RDEPEND}
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-postgresql )"
+DEPEND="${CDEPEND}
 !!dev-db/postgresql-docs:${SLOT}
 !!dev-db/postgresql-base:${SLOT}
 !!dev-db/postgresql-server:${SLOT}
Index: postgresql-9999.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/dev-db/postgresql/postgresql-9999.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 postgresql-9999.ebuild
--- postgresql-9999.ebuild      11 Oct 2014 19:35:08 -0000      1.1
+++ postgresql-9999.ebuild      1 Nov 2014 10:10:21 -0000
@@ -40,7 +40,7 @@
        echo -n ${enable_langs}
 }
 
-RDEPEND="
+CDEPEND="
 >=app-admin/eselect-postgresql-1.2.0
 sys-apps/less
 virtual/libintl
@@ -50,15 +50,15 @@
 perl? ( >=dev-lang/perl-5.8 )
 python? ( ${PYTHON_DEPS} )
 readline? ( sys-libs/readline )
-selinux? ( sec-policy/selinux-postgresql )
 ssl? ( >=dev-libs/openssl-0.9.6-r1 )
 tcl? ( >=dev-lang/tcl-8 )
 uuid? ( dev-libs/ossp-uuid )
 xml? ( dev-libs/libxml2 dev-libs/libxslt )
 zlib? ( sys-libs/zlib )
 "
-
-DEPEND="${RDEPEND}
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-postgresql )"
+DEPEND="${CDEPEND}
 !!dev-db/postgresql-docs:${SLOT}
 !!dev-db/postgresql-base:${SLOT}
 !!dev-db/postgresql-server:${SLOT}

[Comment 2 Sven Vermeulen (RETIRED) 2014-11-02 08:12: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.