[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#987851: marked as done (linux-version sort: argv and stdin behaviors differ)



Your message dated Thu, 19 May 2022 02:33:55 +0000
with message-id <E1nrVz9-0002kv-3X@fasolo.debian.org>
and subject line Bug#987851: fixed in linux-base 4.9
has caused the Debian Bug report #987851,
regarding linux-version sort: argv and stdin behaviors differ
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
987851: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=987851
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: linux-base
Version: 4.6
Severity: normal
Tags: patch

Using argv:
$ linux-version sort 5.8.0-50-generic 5.8.0-50-generic-64k
5.8.0-50-generic
5.8.0-50-generic-64k

Using stdin:
$ cat versions.txt
5.8.0-50-generic
5.8.0-50-generic-64k
$ cat versions.txt | linux-version sort
5.8.0-50-generic-64k
5.8.0-50-generic

It seems the trailing "\n" is unexpectedly being used in the comparison.
Now, perl and I have never really gotten along, but this patch seems to work:

--- /usr/bin/linux-version.orig	2020-06-25 17:23:24.000000000 +0000
+++ /usr/bin/linux-version	2021-04-30 20:39:13.708560573 +0000
@@ -70,7 +70,7 @@
 	@versions = map({[$_, "\n"]} @_);
     } else {
 	while (<STDIN>) {
-	    /^([^ ]*)(.*\n?)$/ or die;
+	    /^([^ \n]*)(.*\n?)$/ or die;
 	    push @versions, [$1, $2];
 	}
     }


-- System Information:
Debian Release: 11.0
  APT prefers unstable-debug
  APT policy: (500, 'unstable-debug'), (500, 'unstable'), (1, 'experimental-debug'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.10.0-5-amd64 (SMP w/4 CPU threads)
Kernel taint flags: TAINT_FIRMWARE_WORKAROUND
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages linux-base depends on:
ii  debconf [debconf-2.0]  1.5.76

linux-base recommends no packages.

linux-base suggests no packages.

-- debconf information:
* linux-base/removing-running-kernel: true
  linux-base/removing-title:

--- End Message ---
--- Begin Message ---
Source: linux-base
Source-Version: 4.9
Done: Ben Hutchings <benh@debian.org>

We believe that the bug you reported is fixed in the latest version of
linux-base, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 987851@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Ben Hutchings <benh@debian.org> (supplier of updated linux-base package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Thu, 19 May 2022 04:05:24 +0200
Source: linux-base
Architecture: source
Version: 4.9
Distribution: unstable
Urgency: medium
Maintainer: Debian Kernel Team <debian-kernel@lists.debian.org>
Changed-By: Ben Hutchings <benh@debian.org>
Closes: 906873 983314 987851 990165
Changes:
 linux-base (4.9) unstable; urgency=medium
 .
   [ Ben Hutchings ]
   * d/salsa-ci.yml: Add CI configuration for salsa.debian.org
   * d/control: Use my debian.org email in Uploaders field
   * Remove perf wrappers, as linux-perf now provides an unversioned perf
     command (Closes: #983314, #990165)
     - d/control: Add versioned Breaks on linux-perf to avoid leaving users
       with no perf command
   * linux-version: Skip detached signature files when listing kernels
     (Closes: #906873)
 .
   [ dann frazier ]
   * linux-version: Fix sorting of versions provided via stdin to match
     the output when provided via argv (Closes: #987851)
 .
   [ Gioele Barabucci ]
   * debian/NEWS: Remove outdated file
   * po/fr: Store in canonical form
   * debian/rules: Remove stray `--with bash-completion`
   * debian/control: Remove unused dependency on `bash-completion`
Checksums-Sha1:
 404983f80c704ea46000a2888d17893965411eae 1666 linux-base_4.9.dsc
 8c080f246f372d94603c0bc546a877d0539fb12f 33460 linux-base_4.9.tar.xz
 2048150bfb3a0668e58b7ad50bf7e24396a52d5e 5697 linux-base_4.9_amd64.buildinfo
Checksums-Sha256:
 7eb683066da139b4fa8e02334c0d18ca3036f92b38f33ca8a1f230aac1d73b36 1666 linux-base_4.9.dsc
 4034c97f67fadc75e9f5e857451fbeda23da2758948fce79c4c17ce26f97b5e1 33460 linux-base_4.9.tar.xz
 73ebe52483ea82abce2f2b48e3c424da719034282d819152b658520762f4d1e7 5697 linux-base_4.9_amd64.buildinfo
Files:
 6e90e298808dcf468bf5329b8b10111b 1666 kernel optional linux-base_4.9.dsc
 64b37e51dff9ed265d64a17590c037cf 33460 kernel optional linux-base_4.9.tar.xz
 c1401c54e1f9062f46ba7ef717262fe5 5697 kernel optional linux-base_4.9_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEErCspvTSmr92z9o8157/I7JWGEQkFAmKFqm0ACgkQ57/I7JWG
EQkX0Q/+LQVm+vTXkwBWHglxp7BvzU7V4WWKNI9Ac+gx7zA5+cvRxdJ9K64QNUPY
7pL0GwzWUaL4Dz4K6EBZVW0kM54Op1Gnv7HXyqf6/M4xKbxCgfVJ8kZZ/UcF+xse
nC0kvJiOtQq3ix7Wlabh8e5DbX8JLH9Hut8vCqIYV7Mq07F3QN9LGHtUkJHhWzp7
FikUxrywBbuUnwQEkI7IBn487k+VbKQI0hbvDznomcjr0cysi1yu4GzUaij4aXVi
p3NNfKPAunGQG9res2iSlR6ymfJux9qApFhG9AAH3gwwyvmZXoObca3ClUV/eLRC
SNwnyj98pwZ5Pi5TKzK/PuT8vJ0p7Kbse7BLtkBW+dmEdWhV9cVudnmXOUjKHZme
Ik5SdRJDTJc8ijqXkybhz9z4pY45q9xN9kkvBZMLq681ZNe+gXw5/x/a2bp1n0Rd
3bI4Sm2/csEIBv7dVSxMToSWhxO1RMIYtJ4w72vilVJwilU+M9ZRJERdd+wv0P9z
zy4JxRlPyO5hUiKMzzYRVqtvaWWNfsY9gD8IwSgF61ssKL6+v+4mT9W0I+mtdwO+
9LF+RIHkdrG2vWBV/8rbsJen/U4F5RpYmX3NVSpQvhpjfk2NckjZNg2Xrg8yJcGF
lNPEV43LVHP+w8+DLl4/TTZceHu3x2LXpEBf9+cEq6eZKFdMGqc=
=WlVB
-----END PGP SIGNATURE-----

--- End Message ---

Reply to: