Skip to content

tests/od: fix two tests on big-endian hosts - #14671

Merged
sylvestre merged 1 commit into
uutils:mainfrom
pkubaj:big-endian-test-fix
Sep 18, 2026
Merged

sylvestre merged 1 commit into
uutils:mainfrom
pkubaj:big-endian-test-fix

Conversation

@pkubaj

@pkubaj pkubaj commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

test_suppress_duplicates and test_od_options_after_filename dump
multi-byte units (-x, -O, -t x2) and compare the result against
little-endian expected output without selecting a byte order. They
fail on big-endian hosts, where od correctly defaults to the native
order:

expected: " 1c68 fdbb\n"
actual: " 681c bbfd\n"

Pass --endian=little like the other tests in this file do. In
test_od_options_after_filename the option goes after the file name, in
keeping with what that test exercises.

Found on FreeBSD/powerpc64.

test_suppress_duplicates and test_od_options_after_filename dump
multi-byte units (-x, -O, -t x2) and compare the result against
little-endian expected output without selecting a byte order. They
fail on big-endian hosts, where od correctly defaults to the native
order:

  expected: " 1c68 fdbb\n"
  actual:   " 681c bbfd\n"

Pass --endian=little like the other tests in this file do. In
test_od_options_after_filename the option goes after the file name, in
keeping with what that test exercises.

Found on FreeBSD/powerpc64.
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/tail/retry (passes in this run but fails in the 'main' branch)

@sylvestre
sylvestre merged commit 48429c0 into uutils:main Sep 18, 2026
97 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants