[PATCH 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 8ee248703aa1..c1fc746d6019 100644
--- a/eclass/rpm.eclass
+++ b/eclass/rpm.eclass
@@ -22,8 +22,8 @@ inherit estack
 # @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
 # - 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= 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.