Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package suse-module-tools for openSUSE:Factory checked in at 2022-06-03 14:16:42 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/suse-module-tools (Old) and /work/SRC/openSUSE:Factory/.suse-module-tools.new.1548 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "suse-module-tools" Fri Jun 3 14:16:42 2022 rev:55 rq:980502 version:16.0.20 Changes: -------- --- /work/SRC/openSUSE:Factory/suse-module-tools/suse-module-tools.changes 2022-02-01 14:03:06.528010318 +0100 +++ /work/SRC/openSUSE:Factory/.suse-module-tools.new.1548/suse-module-tools.changes 2022-06-03 14:16:46.929325514 +0200 @@ -1,0 +2,9 @@ +Thu Jun 2 10:55:37 UTC 2022 - Martin Wilck <mwi...@suse.com> + +- Update to version 16.0.20: + * Bump version to 16.0.20 + * driver-check.sh: avoid false positive error messages (boo#1200107) + * don't hardcode /boot for kernel-related files (boo#1199873) + * spec file: use "install -p" consistently + +------------------------------------------------------------------- Old: ---- suse-module-tools-16.0.19.obscpio New: ---- suse-module-tools-16.0.20.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ suse-module-tools.spec ++++++ --- /var/tmp/diff_new_pack.9nkwXG/_old 2022-06-03 14:16:47.433326169 +0200 +++ /var/tmp/diff_new_pack.9nkwXG/_new 2022-06-03 14:16:47.437326174 +0200 @@ -45,7 +45,7 @@ %global modprobe_conf_rpmsave %(echo "%{modprobe_conf_files}" | sed 's,\\([^ ]*\\),%{_sysconfdir}/modprobe.d/\\1.conf.rpmsave,g') Name: suse-module-tools -Version: 16.0.19 +Version: 16.0.20 Release: 0 Summary: Configuration for module loading and SUSE-specific utilities for KMPs License: GPL-2.0-or-later @@ -169,26 +169,25 @@ install -d -m 755 "%{buildroot}%{_unitdir}/systemd-sysctl.service.d" echo '[Unit]' >"%{buildroot}%{sysctl_dropin}" %if %{with kernel_sysctl} -install -m 644 kernel-sysctl.service "%{buildroot}%{_unitdir}" +install -pm 644 kernel-sysctl.service "%{buildroot}%{_unitdir}" echo 'Wants=kernel-sysctl.service' >>"%{buildroot}%{sysctl_dropin}" %endif %if %{with boot_sysctl} -install -m 644 boot-sysctl.service "%{buildroot}%{_unitdir}" +install -pm 644 boot-sysctl.service "%{buildroot}%{_unitdir}" echo 'Wants=boot-sysctl.service' >>"%{buildroot}%{sysctl_dropin}" %endif -# Ensure that the sg driver is loaded early (bsc#1036463) -# Not needed in SLE11, where sg is loaded via udev rule. install -d -m 755 "%{buildroot}%{_modulesloaddir}" -install -pm 644 sg.conf "%{buildroot}%{_modulesloaddir}" +install -pm 644 -t "%{buildroot}%{_modulesloaddir}" modules-load.d/*.conf + %ifarch ppc64 ppc64le install -d -m 755 %{buildroot}/usr/lib/systemd/system-generators -install -m 755 udev-trigger-generator %{buildroot}/usr/lib/systemd/system-generators +install -pm 755 udev-trigger-generator %{buildroot}/usr/lib/systemd/system-generators %endif # udev rules (formerly system-tuning-common-SUSE, udev-extra-rules) install -d -m 755 %{buildroot}%{_udevrulesdir} -install -m 644 udevrules/*.rules %{buildroot}%{_udevrulesdir} +install -pm 644 udevrules/*.rules %{buildroot}%{_udevrulesdir} mkdir -p %{buildroot}%{_defaultlicensedir} ++++++ _servicedata ++++++ --- /var/tmp/diff_new_pack.9nkwXG/_old 2022-06-03 14:16:47.473326221 +0200 +++ /var/tmp/diff_new_pack.9nkwXG/_new 2022-06-03 14:16:47.477326226 +0200 @@ -1,6 +1,6 @@ <servicedata> <service name="tar_scm"> <param name="url">https://github.com/openSUSE/suse-module-tools.git</param> - <param name="changesrevision">3d244d5dfc0e5c3d5c8a7cc90d8e9e81be6577c1</param></service></servicedata> + <param name="changesrevision">7272c960f4b8d3a07cf5ac5c1a946df876c488c7</param></service></servicedata> (No newline at EOF) ++++++ suse-module-tools-16.0.19.obscpio -> suse-module-tools-16.0.20.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/.gitignore new/suse-module-tools-16.0.20/.gitignore --- old/suse-module-tools-16.0.19/.gitignore 1970-01-01 01:00:00.000000000 +0100 +++ new/suse-module-tools-16.0.20/.gitignore 2022-06-02 10:56:33.000000000 +0200 @@ -0,0 +1,5 @@ +*~ +\#* +*.rej +*.orig + diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/.obs/workflows.yml new/suse-module-tools-16.0.20/.obs/workflows.yml --- old/suse-module-tools-16.0.19/.obs/workflows.yml 2022-01-28 21:35:51.000000000 +0100 +++ new/suse-module-tools-16.0.20/.obs/workflows.yml 2022-06-02 10:56:33.000000000 +0200 @@ -1,6 +1,48 @@ workflow: - steps: - - branch_package: - source_project: home:mwilck:suse-module-tools - source_package: suse-module-tools - target_project: home:mwilck + steps: + - branch_package: + source_project: home:mwilck:suse-module-tools + source_package: suse-module-tools + target_project: home:mwilck + - configure_repositories: + project: home:mwilck + repositories: + - name: openSUSE_Tumbleweed + paths: + - target_project: openSUSE:Factory + target_repository: snapshot + architectures: + - x86_64 + - i586 + - name: openSUSE_Leap_15.4 + paths: + - target_project: openSUSE:Tools + target_repository: 15.4 + - target_project: openSUSE:Leap:15.4 + target_repository: standard + architectures: + - x86_64 + - name: SLE_15_SP4 + paths: + - target_project: openSUSE:Tools + target_repository: 15.4 + - target_project: SUSE:SLE-15-SP4:GA + target_repository: pool + architectures: + - x86_64 + - s390x + - ppc64le + - aarch64 + - name: SLE_15_SP3 + paths: + - target_project: openSUSE:Tools + target_repository: 15.3 + - target_project: SUSE:SLE-15-SP3:GA + target_repository: pool + architectures: + - x86_64 + - s390x + - ppc64le + - aarch64 + filters: + event: pull_request diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/driver-check.sh new/suse-module-tools-16.0.20/driver-check.sh --- old/suse-module-tools-16.0.19/driver-check.sh 2022-01-28 21:35:51.000000000 +0100 +++ new/suse-module-tools-16.0.20/driver-check.sh 2022-06-02 10:56:33.000000000 +0200 @@ -10,6 +10,22 @@ trap 'rm -rf "$tmp"' EXIT tmp=$(mktemp -d) +find_usrmerge_boot() { + local filename=$1 + local kver=$2 + local ext=${3:+."$3"} + local f + + for f in "/usr/lib/modules/$kver/$filename$ext" "/boot/$filename-$kver$ext" + do + if [ -e "$f" ]; then + echo "$f" + return + fi + done + echo "WARNING: find_usrmerge_boot: $filename$ext not found for kernel $kver" >&2 +} + find_depmod() { local _d @@ -115,7 +131,7 @@ if ! rpm -q --qf '%{description}\n' "$kernel" | grep -q '^GIT '; then error "$kernel does not look like a SUSE kernel package (no commit id)" fi - if ! rpm -q --qf '%{postin}\n' "$kernel" | grep -q 'weak-modules2'; then + if ! rpm -q --qf '%{postin}\n' "$kernel" | egrep -Eq 'weak-modules2|kernel-scriptlets/rpm-post'; then error "$kernel does not look like a SUSE kernel package (wrong %post script)" fi } @@ -125,8 +141,8 @@ local krel=$1 system_map module_symvers msg res args bad=false local mit_version - system_map="/boot/System.map-$krel" - module_symvers="/boot/symvers-$krel.gz" + system_map=$(find_usrmerge_boot System.map "$krel") + module_symvers=$(find_usrmerge_boot symvers "$krel" gz) if ! test -e "$system_map"; then error "$system_map not found" bad=true @@ -136,7 +152,7 @@ bad=true fi if $bad; then - explain "Each kernel must install /boot/System.map-\$version and /boot/symvers-\$version.gz to be able to check module dependencies." + explain "Each kernel must install System.map and symvers.gz to be able to check module dependencies." return fi set -- $("$DEPMOD" --version | sed -rn 's/.* ([0-9]+)(\.([0-9]+)(\..*)?)?/\1 \3/p') @@ -170,7 +186,7 @@ { local kmp=$1 prefix prev_krel krel path found_module=false - if ! rpm -q --qf '%{postin}\n' "$kmp" | grep -q 'weak-modules2'; then + if ! rpm -q --qf '%{postin}\n' "$kmp" | egrep -Eq 'weak-modules2|kernel-scriptlets/kmp-post'; then error "$kmp does not look like a SUSE kernel module package (wrong %post)" fi if ! rpm -q -R "$kmp" | grep -Eq "$req_re"; then @@ -305,7 +321,7 @@ case "$rpm" in kernel-source-* | kernel-syms-* | kernel-*-debug* | kernel-*-man-* | \ kernel-*-devel-* | kernel-firmware-* | kernel-coverage-* | \ - kernel-docs-* | kernel-devel-* | kernel-macros-*) + kernel-docs-* | kernel-devel-* | kernel-macros-* | kernel-install-tools-*) continue esac # store the filelist to speed up file_owner() diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/modules-load.d/sg.conf new/suse-module-tools-16.0.20/modules-load.d/sg.conf --- old/suse-module-tools-16.0.19/modules-load.d/sg.conf 1970-01-01 01:00:00.000000000 +0100 +++ new/suse-module-tools-16.0.20/modules-load.d/sg.conf 2022-06-02 10:56:33.000000000 +0200 @@ -0,0 +1,2 @@ +# Ensure that the sg driver is loaded early (bsc#1036463) +sg diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/sg.conf new/suse-module-tools-16.0.20/sg.conf --- old/suse-module-tools-16.0.19/sg.conf 2022-01-28 21:35:51.000000000 +0100 +++ new/suse-module-tools-16.0.20/sg.conf 1970-01-01 01:00:00.000000000 +0100 @@ -1 +0,0 @@ -sg diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/suse-module-tools.spec new/suse-module-tools-16.0.20/suse-module-tools.spec --- old/suse-module-tools-16.0.19/suse-module-tools.spec 2022-01-28 21:35:51.000000000 +0100 +++ new/suse-module-tools-16.0.20/suse-module-tools.spec 2022-06-02 10:56:33.000000000 +0200 @@ -45,7 +45,7 @@ %global modprobe_conf_rpmsave %(echo "%{modprobe_conf_files}" | sed 's,\\([^ ]*\\),%{_sysconfdir}/modprobe.d/\\1.conf.rpmsave,g') Name: suse-module-tools -Version: 16.0.19 +Version: 16.0.20 Release: 0 Summary: Configuration for module loading and SUSE-specific utilities for KMPs License: GPL-2.0-or-later @@ -169,26 +169,25 @@ install -d -m 755 "%{buildroot}%{_unitdir}/systemd-sysctl.service.d" echo '[Unit]' >"%{buildroot}%{sysctl_dropin}" %if %{with kernel_sysctl} -install -m 644 kernel-sysctl.service "%{buildroot}%{_unitdir}" +install -pm 644 kernel-sysctl.service "%{buildroot}%{_unitdir}" echo 'Wants=kernel-sysctl.service' >>"%{buildroot}%{sysctl_dropin}" %endif %if %{with boot_sysctl} -install -m 644 boot-sysctl.service "%{buildroot}%{_unitdir}" +install -pm 644 boot-sysctl.service "%{buildroot}%{_unitdir}" echo 'Wants=boot-sysctl.service' >>"%{buildroot}%{sysctl_dropin}" %endif -# Ensure that the sg driver is loaded early (bsc#1036463) -# Not needed in SLE11, where sg is loaded via udev rule. install -d -m 755 "%{buildroot}%{_modulesloaddir}" -install -pm 644 sg.conf "%{buildroot}%{_modulesloaddir}" +install -pm 644 -t "%{buildroot}%{_modulesloaddir}" modules-load.d/*.conf + %ifarch ppc64 ppc64le install -d -m 755 %{buildroot}/usr/lib/systemd/system-generators -install -m 755 udev-trigger-generator %{buildroot}/usr/lib/systemd/system-generators +install -pm 755 udev-trigger-generator %{buildroot}/usr/lib/systemd/system-generators %endif # udev rules (formerly system-tuning-common-SUSE, udev-extra-rules) install -d -m 755 %{buildroot}%{_udevrulesdir} -install -m 644 udevrules/*.rules %{buildroot}%{_udevrulesdir} +install -pm 644 udevrules/*.rules %{buildroot}%{_udevrulesdir} mkdir -p %{buildroot}%{_defaultlicensedir} diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/suse-module-tools-16.0.19/weak-modules2 new/suse-module-tools-16.0.20/weak-modules2 --- old/suse-module-tools-16.0.19/weak-modules2 2022-01-28 21:35:51.000000000 +0100 +++ new/suse-module-tools-16.0.20/weak-modules2 2022-06-02 10:56:33.000000000 +0200 @@ -91,6 +91,22 @@ dlog "DEPMOD=$DEPMOD" } +find_usrmerge_boot() { + local filename=$1 + local kver=$2 + local ext=${3:+."$3"} + local f + + for f in "/usr/lib/modules/$kver/$filename$ext" "/boot/$filename-$kver$ext" + do + if [ -e "$f" ]; then + echo "$f" + return + fi + done + log "WARNING: find_usrmerge_boot: $filename$ext not found for kernel $kver" +} + log() { [ $opt_verbose -gt 0 ] && echo "$@" >&2 return 0 @@ -197,20 +213,23 @@ has_unresolved_symbols() { local krel=$1 basedir=$2 output status args sym_errors _f - if [ ! -e $tmpdir/symvers-$krel ]; then - for _f in /boot/symvers-$krel.gz /usr/lib/modules/$krel/symvers.gz; do - if [ -e $_f ]; then - dlog "has_unresolved_symbols: found $_f" - zcat $_f >$tmpdir/symvers-$krel - break - fi - done + if [ ! -e "$tmpdir/symvers-$krel" ]; then + _f=$(find_usrmerge_boot symvers "$krel" gz) + if [ -n "$_f" ]; then + dlog "has_unresolved_symbols: found $_f" + zcat "$_f" >"$tmpdir/symvers-$krel" + fi fi if [ -e $tmpdir/symvers-$krel ]; then args=(-E $tmpdir/symvers-$krel) else - echo "warning: $tmpdir/symvers-$krel not available" >&2 - args=(-F /boot/System.map-$krel) + echo "WARNING: symvers.gz not found for $krel, symbol resolution will be unreliable" >&2 + _f=$(find_usrmerge_boot System.map "$krel") + if [ -n "$_f" ]; then + args=(-F "$_f") + else + echo "WARNING: System.map not found for $krel, symbol resolution may fail" >&2 + fi fi output="$("$DEPMOD" -b "$basedir" -ae "${args[@]}" $krel 2>&1)" status=$? @@ -434,17 +453,19 @@ run_depmod_build_initrd() { local krel=$1 local status=0 + local system_map - if [ -d /lib/modules/$krel -a -f /boot/System.map-$krel ] ; then - doit "$DEPMOD" -F /boot/System.map-$krel -ae $krel || return 1 + if [ -d /lib/modules/$krel ]; then + system_map=$(find_usrmerge_boot System.map "$krel") + if [ -n "$system_map" ]; then + doit "$DEPMOD" -F "$system_map" -ae "$krel" || return 1 + fi fi if needs_initrd $krel; then - local image + local image x for x in vmlinuz image vmlinux linux bzImage uImage Image zImage; do - if [ -f /boot/$x-$krel ]; then - image=$x - break - fi + image=$(find_usrmerge_boot "$x" "$krel") + [ -z "$image" ] || break done if [ -n "$image" ]; then if test -n "$INITRD_IN_POSTTRANS"; then @@ -500,8 +521,9 @@ kernel_changed() { local krel=$1 flavor=${1##*-} + local system_map=$(find_usrmerge_boot System.map "$krel") - if [ ! -f /boot/System.map-$krel ]; then + if [ -z "$system_map" ]; then # this kernel does not exist anymore dlog "kernel_changed: kernel removed" return 0 @@ -552,6 +574,7 @@ add_kmp() { local kmp=$1 kmpshort=${1%-*-*} local basename=${kmpshort%-kmp-*} flavor=${kmpshort##*-} + local system_map # Find the kmp to be added as well as any previous versions find_kmps "$basename" "$flavor" || return 1 @@ -566,7 +589,9 @@ continue esac dlog "add_kmp: processing $kmp for $krel" - [ -d $dir -a -f /boot/System.map-$krel ] || continue + [ -d $dir ] || continue + system_map=$(find_usrmerge_boot System.map "$krel") + [ -n "$system_map" ] || continue if opt_debug=1 has_unresolved_symbols "$krel" "/"; then echo "Warning: /lib/modules/$krel is inconsistent" >&2 echo "Warning: weak-updates symlinks might not be created" >&2 @@ -611,7 +636,7 @@ sed 's:^/usr::' > $tmpdir/modules-$kmp check_kmp "$kmp" || return 1 - local dir krel status + local dir krel status system_map for dir in /lib/modules/*; do krel=${dir#/lib/modules/} case "$krel" in @@ -620,7 +645,9 @@ *) continue esac - [ -d $dir -a -f /boot/System.map-$krel ] || continue + [ -d $dir ] || continue + system_map=$(find_usrmerge_boot System.map "$krel") + [ -n "$system_map" ] || continue dlog "remove_kmp: processing $kmp for $krel" if kmp_is_present $kmp $krel; then local other_found=0 inconsistent=0 ++++++ suse-module-tools.obsinfo ++++++ --- /var/tmp/diff_new_pack.9nkwXG/_old 2022-06-03 14:16:47.597326382 +0200 +++ /var/tmp/diff_new_pack.9nkwXG/_new 2022-06-03 14:16:47.597326382 +0200 @@ -1,5 +1,5 @@ name: suse-module-tools -version: 16.0.19 -mtime: 1643402151 -commit: 3d244d5dfc0e5c3d5c8a7cc90d8e9e81be6577c1 +version: 16.0.20 +mtime: 1654160193 +commit: 7272c960f4b8d3a07cf5ac5c1a946df876c488c7