Page MenuHomeFreeBSD

hexdump test: take into account byte order
ClosedPublic

Authored by renato.riolino_eldorado.org.br on Jan 15 2020, 2:25 PM.
Tags
Referenced Files
Unknown Object (File)
Mon, May 11, 3:29 PM
Unknown Object (File)
Mon, May 11, 3:28 PM
Unknown Object (File)
Mon, May 11, 5:00 AM
Unknown Object (File)
Mon, May 11, 4:59 AM
Unknown Object (File)
Mon, May 11, 3:40 AM
Unknown Object (File)
Sat, May 9, 1:27 PM
Unknown Object (File)
Sat, May 9, 1:22 PM
Unknown Object (File)
Fri, May 8, 3:54 AM
Subscribers

Details

Summary

Hexdump test was failling on big endian systems when testing decimal, octal and hexa outputs

Test Plan

cd /usr/src/usr.bin/hexdump/tests/
make
make install
kyua test -k /usr/tests/usr.bin/hexdump/Kyuafile

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Seems like a good approach to me; the only nit I have to offer is that we oftentimes prefer spelling these "el" and "eb," but I suspect we have enough of a mix that it doesn't matter.

This revision is now accepted and ready to land.Jan 15 2020, 2:36 PM

Fix speeling "be" and "le" to "eb" and "el"

This revision now requires review to proceed.Jan 15 2020, 4:07 PM

Perfect, thanks! =-) May I ask which big-endian arch you're running these tests on? This is just my curiosity- the tests are indisputably broken as-is.

This revision is now accepted and ready to land.Jan 15 2020, 4:10 PM

Perfect, thanks! =-) May I ask which big-endian arch you're running these tests on? This is just my curiosity- the tests are indisputably broken as-is.

Sure! I'm on ppc64 :p

This revision was automatically updated to reflect the committed changes.