fix(rpm): debuginfo support on more distributions

This enables RPM debuginfo package generation across different Linux
distributions (tested on Rocky and Ubuntu in CI) by implementing RPM
version-based detection if possible, and falling back to the existing OS
release-based detection otherwise.

There's indeed a clear pivot in RPM versions for that matter:
- RPM < 4.18 uses `buildsubdir .` (which happens to be called the
  "CENTOS" type in the present project),
- RPM >= 4.18 is incompatible with the former:
  ```
  rm: refusing to remove '.' or '..' directory: skipping '.'
  ```
  It therefore uses `buildsubdir BUILD_SUB` (called the "FEDORA" type
  here).

RedHat distros have `redhat-rpm-config` with `%_enable_debug_packages`
that auto-invokes `%debug_package` (rpm-software-management/rpm#2204).
Debian/Ubuntu ship vanilla upstream RPM without this configuration:
```
output 'tests/rpm/test_debuginfo_rpm-debuginfo-1-0..rpm' was not created
```
The present change therefore adds `%debug_package` only when applicable:
`%{!?_enable_debug_packages:%debug_package}`.

Also, since RPM 4.14, unique debug package filenames are enabled by
default, leading to variadic filenames being generated:
```
Executing tests from //tests/rpm:test_golden_debuginfo_rpm_contents
-----------------------------------------------------------------------------
29c29
< /usr/lib/debug/test_debuginfo-1-0.x86_64.debug
---
> /usr/lib/debug/test_debuginfo.debug
FAIL: files "tests/rpm/test_debuginfo_rpm_contents.txt" and "tests/rpm/test_debuginfo_rpm_contents.txt.golden" differ
```
The change makes debug package filenames consistent across distributions
by means of: `%undefine _unique_debug_names` (safe no-op on older RPM
versions).

Author: rdesgroppes

.bazelci/tests.yml

@@ -37,8 +37,6 @@ default_tests_with_rpm: &default_tests_with_rpm
     - "//pkg/..."
     - "//tests/..."
     - "//toolchains/..."
-    # This has started to fail, even on CentOS.
-    - "-//tests/rpm:test_golden_debuginfo_rpm_contents"
 
 win_tests: &win_tests
   test_flags:
@@ -66,7 +64,10 @@ win_tests: &win_tests
 ubuntu2204: &ubuntu
   platform: ubuntu2204
   <<: *common
-  <<: *default_tests
+  <<: *default_tests_with_rpm
+  shell_commands:
+    - sudo apt-get update
+    - sudo apt-get install -y rpm elfutils  # for rpmbuild & eu-strip
 
 centos7: &centos
   platform: centos7_java11_devtoolset10

pkg/rpm_pfg.bzl

@@ -604,6 +604,13 @@ def _pkg_rpm_impl(ctx):
     if ctx.attr.architecture:
         preamble_pieces.append("BuildArch: " + ctx.attr.architecture)
 
+    if ctx.attr.debuginfo:
+        # RedHat distros have redhat-rpm-config with %_enable_debug_packages macro; others need explicit declaration
+        preamble_pieces.append("%{{!?_enable_debug_packages:%debug_package}}")  # set %debug_package unless macro exists
+
+        # https://rpm.org/wiki/Releases/4.14.0: "Add support for unique debug file names"
+        preamble_pieces.append("%undefine _unique_debug_names")  # no-op if not defined
+
     preamble_file = ctx.actions.declare_file(
         "{}.spec.preamble".format(rpm_name),
     )

toolchains/rpm/rpmbuild_configure.bzl

@@ -81,15 +81,6 @@ def _build_repo_for_rpmbuild_toolchain_impl(rctx):
     if rctx.attr.debuginfo_type not in DEBUGINFO_VALID_VALUES:
         fail("debuginfo_type must be one of", DEBUGINFO_VALID_VALUES)
 
-    debuginfo_type = rctx.attr.debuginfo_type
-    if debuginfo_type == DEBUGINFO_TYPE_AUTODETECT:
-        if rctx.path(RELEASE_PATH).exists:
-            rctx.watch(RELEASE_PATH)
-            os_name, _ = _parse_release_info(rctx.read(RELEASE_PATH))
-            debuginfo_type = DEBUGINFO_TYPE_BY_OS_RELEASE.get(os_name, debuginfo_type)
-        else:
-            debuginfo_type = DEBUGINFO_TYPE_NONE
-
     rpmbuild_path = rctx.which("rpmbuild")
     if rctx.attr.verbose:
         if rpmbuild_path:
@@ -99,13 +90,32 @@ def _build_repo_for_rpmbuild_toolchain_impl(rctx):
 
     version = "unknown"
     if rpmbuild_path:
+        rctx.watch(rpmbuild_path)
         res = rctx.execute([rpmbuild_path, "--version"])
         if res.return_code == 0:
             # expect stdout like: RPM version 4.16.1.2
             parts = res.stdout.strip().split(" ")
             if parts[0] == "RPM" and parts[1] == "version":
                 version = parts[2]
 
+    debuginfo_type = rctx.attr.debuginfo_type
+    if debuginfo_type == DEBUGINFO_TYPE_AUTODETECT:
+        version_parts = version.split(".")
+        if len(version_parts) > 1 and version_parts[0].isdigit() and version_parts[1].isdigit():
+            major = int(version_parts[0])
+            minor = int(version_parts[1])
+            if major < 4 or (major == 4 and minor < 18):
+                debuginfo_type = DEBUGINFO_TYPE_CENTOS
+            else:
+                # https://rpm.org/wiki/Releases/4.18.0: "Make %{buildsubdir} settable outside %setup"
+                debuginfo_type = DEBUGINFO_TYPE_FEDORA
+        elif rctx.path(RELEASE_PATH).exists:
+            rctx.watch(RELEASE_PATH)
+            os_name, _ = _parse_release_info(rctx.read(RELEASE_PATH))
+            debuginfo_type = DEBUGINFO_TYPE_BY_OS_RELEASE.get(os_name, debuginfo_type)
+        else:
+            debuginfo_type = DEBUGINFO_TYPE_NONE
+
     _write_build(
         rctx = rctx,
         path = rpmbuild_path,