[PATCH v3 5/5] rpm.eclass: make RPM_COMPRESS_TYPE mandatory in EAPI 9

Eli Schwartz <[email protected]>
Newsgroups gmane.linux.gentoo.devel
Message-ID <[email protected]>
Without setting it, there are cases where rpm doesn't work, and cases
where rpm2targz doesn't work. So we eqawarn if not set. For backwards
compat it isn't required... but we can fix that in new EAPI, so do so!

Signed-off-by: Eli Schwartz <[email protected]>
---
 eclass/rpm.eclass | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/eclass/rpm.eclass b/eclass/rpm.eclass
index 6a096aba78bd..920f11262bdd 100644
--- a/eclass/rpm.eclass
+++ b/eclass/rpm.eclass
@@ -22,8 +22,8 @@ inherit estack toolchain-funcs
 # @DEFAULT_UNSET
 # @DESCRIPTION:
 # Comma-separated list of app-arch/rpm compression formats. If set,
-# app-arch/rpm will be allowed as a BDEPEND to unpack distfiles. Supported
-# types:
+# app-arch/rpm will be allowed as a BDEPEND to unpack distfiles. Must be set
+# for EAPI 9. Supported types:
 #
 # - none (rpm is supported but distfile is uncompressed or builtin zlib)
 #
@@ -37,6 +37,9 @@ inherit estack toolchain-funcs
 # - zstd (.zst)
 #
 # - "" (empty -- the ebuild hasn't been updated to resolve deprecations)
+if [[ ${EAPI} != [78] && -z ${RPM_COMPRESS_TYPE} ]]; then
+	die "RPM_COMPRESS_TYPE= must be defined starting with EAPI 9"
+fi
 
 _rpm_set_globals() {
 	local rpmdep= rpmuse= rpm2tar="true" t= types=()
-- 
2.52.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.