Source: libcrcutil
Source-Version: 1.0-5.4
Severity: normal
Tags: patch

Hi!

The current explicit list of Architectures is not easy to maintain,
and its purpose is not immediately obvious when looking at the field.
While comments in debian/control might improve that, it still needs
to be adapted any time a new arch gets added or removed.

As it stands, the current list contains a least one invalid entry for
any-x32, which does not exist, as the x32 arch is matched by any-amd64
instead. This means that the package could get rejected by dak, as that
checks for valid architectures in the .dsc Package-List field generated
from the Architecture fields (although it does not seem to be rejecting
it currently).

Attached a patch, that should improve this.

Thanks,
Guillem
From 8580ca45b55bf09b5c1b4c80c9a1960d7f5a8ac0 Mon Sep 17 00:00:00 2001
From: Guillem Jover <[email protected]>
Date: Sun, 7 Sep 2025 05:05:57 +0200
Subject: [PATCH] Switch to use a declarative build depends on arch being
 little-endian

The current explicit list of Architectures is not easy to maintain,
and its purpose is not immediately obvious when looking at the field.
While a comment in debian/control might improve that, it still needs
to be adapted any time a new arch gets added or removed.

As it stands, the current list contains a least one invalid entry for
any-x32, which does not exist, as the x32 arch is matched by any-amd64
instead. This means that the package might get rejected by dak, as that
checks for valid architectures in the .dsc Package-List field generated
from the Architecture fields.
---
 debian/control | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/debian/control b/debian/control
index ca3ec92..9d24dec 100644
--- a/debian/control
+++ b/debian/control
@@ -3,6 +3,7 @@ Section: libs
 Priority: optional
 Maintainer: Dmitry Smirnov <[email protected]>
 Build-Depends: dpkg-dev (>= 1.22.5), debhelper (>= 11~)
+Build-Depends-Arch: architecture-is-little-endian
 Build-Depends-Indep: texlive-binaries
                     ,texlive-latex-base
                     ,texlive-latex-extra
@@ -15,7 +16,7 @@ Package: libcrcutil0t64
 Provides: ${t64:Provides}
 Replaces: libcrcutil0
 Breaks: libcrcutil0 (<< ${source:Version})
-Architecture: any-alpha any-amd64 any-arm any-arm64 any-i386 any-ia64 any-mipsel any-mips64el any-ppc64el any-riscv64 any-sh4 any-x32 loong64
+Architecture: any
 Multi-Arch: same
 Pre-Depends: ${misc:Pre-Depends}
 Depends: ${misc:Depends}, ${shlibs:Depends}
@@ -45,7 +46,7 @@ Description: library for cyclic redundancy check (CRC) computation
 
 Package: libcrcutil-dev
 Section: libdevel
-Architecture: any-alpha any-amd64 any-arm any-arm64 any-i386 any-ia64 any-mipsel any-mips64el any-ppc64el any-riscv64 any-sh4 any-x32 loong64
+Architecture: any
 Depends: ${misc:Depends}, libcrcutil0t64 (= ${binary:Version})
 Suggests: libcrcutil-doc
 Description: library for cyclic redundancy check (CRC) computation - development files
-- 
2.51.0

Reply via email to