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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Zach van Rijn <me <at> zv.io>
Subject: bug#60339: closed (Re: bug#60339: tests/cp/proc-short-read.sh may
 fail due to architecture / case sensitivity assumption)
Date: Mon, 26 Dec 2022 17:13:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#60339: tests/cp/proc-short-read.sh may fail due to architecture / case sensitivity assumption

which was filed against the coreutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 60339 <at> debbugs.gnu.org.

-- 
60339: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=60339
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

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][.]

[Message part 3 (message/rfc822, inline)]

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 4 (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)]

Previous Next

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