GNU bug report logs - #60339
tests/cp/proc-short-read.sh may fail due to architecture / case sensitivity assumption

Previous Next

Package: coreutils;

Reported by: Zach van Rijn <me <at> zv.io>

Date: Mon, 26 Dec 2022 15:30:02 UTC

Severity: normal

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60339 in the body.
You can then email your comments to 60339 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to bug-coreutils <at> gnu.org:
bug#60339; Package coreutils. (Mon, 26 Dec 2022 15:30:02 GMT) Full text and rfc822 format available.
Acknowledgement sent to Zach van Rijn <me <at> zv.io>:
New bug report received and forwarded. Copy sent to bug-coreutils <at> gnu.org. (Mon, 26 Dec 2022 15:30:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Zach van Rijn <me <at> zv.io>
To: bug-coreutils <at> gnu.org
Subject: tests/cp/proc-short-read.sh may fail due to architecture / case
 sensitivity assumption
Date: Mon, 26 Dec 2022 08:38:29 -0600

[Message part 1 (text/plain, inline)]
The test tests/cp/proc-short-read.sh attempts to remove common
differences in the output of /proc/cpuinfo:

    # adjust varying parts
    sed '/MHz/d; /bogomips/d;' 1 > proc.cp || framework_failure_
    sed '/MHz/d; /bogomips/d;' 2 > proc.cat || framework_failure_

However the string "bogomips" may differ in capitalization across
different architectures [1].

Testing on a big.LITTLE system uncovered this assumption.

    Linux DietPi 5.4.225-odroidxu4 #22.11.1 SMP PREEMPT Wed Nov
    30 10:55:16 UTC 2022 armv7l GNU/Linux

Still present as of 9581c4b59d2e11cd090613cd300a833b4441d2b5.

Proposed patch attached, which aims to be minimally invasive.


References:

[1]: https://git.adelielinux.org/adelie/packages/-/issues/943

[tests-cp-proc-short-read.patch (text/x-patch, attachment)]
Reply sent to Paul Eggert <eggert <at> cs.ucla.edu>:
You have taken responsibility. (Mon, 26 Dec 2022 17:13:01 GMT) Full text and rfc822 format available.
Notification sent to Zach van Rijn <me <at> zv.io>:
bug acknowledged by developer. (Mon, 26 Dec 2022 17:13:02 GMT) Full text and rfc822 format available.

Message #10 received at 60339-done <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Zach van Rijn <me <at> zv.io>
Cc: 60339-done <at> debbugs.gnu.org
Subject: Re: bug#60339: tests/cp/proc-short-read.sh may fail due to
 architecture / case sensitivity assumption
Date: Mon, 26 Dec 2022 09:12:48 -0800

I installed the fix. [Tt][Hh][Aa][Nn][Kk][Ss][.]
bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Tue, 24 Jan 2023 12:24:05 GMT) Full text and rfc822 format available.
This bug report was last modified 2 years and 206 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.