public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* xfstests: heads up about coreutils v8.20+ related failures
@ 2013-04-11  9:45 Dave Chinner
  0 siblings, 0 replies; only message in thread
From: Dave Chinner @ 2013-04-11  9:45 UTC (permalink / raw)
  To: xfs

Folks,

Debian unstable just transistined to coreutils v8.20 recently, ad
this has caused a bunch of tests to fail due to golden output
mismatches.

A while back, various GNU projects have decided to use the gcc quote
method, which means that the longstanding format of output messages
has changed to what gcc has been using for some time.

This is the typical sort of diff you'll see in a failure:

-mv: cannot move `TEST_DIR/test-mv/ab/aa/' to `TEST_DIR/test-mv/aa': File exists
+mv: cannot move 'TEST_DIR/test-mv/ab/aa/' to 'TEST_DIR/test-mv/aa': File exists

It's subtle, but the difference is `...' vs '...'.(*)

What this means is that we are going to have to add filters to all
the coreutils output in xfstests to convert the quotes to a common
format for the golden output.

I'm not going to get to this in the next couple of weeks (travel to
LF collab summit/LSFMM next week), so someone else might want to
have a look at this in the mean time...

Cheers,

Dave.

(*) Well, it's even more subtle than that, because I've had problems
with GCC quotes for years - they simply don't don't copy-n-paste
very well, even when all terminals and machines are using the same
locale and UTF-8 aware applications:

v8.13
$ LC_ALL=en_AU.UTF-8 mv a /etc
mv: cannot move `a' to `/etc/a': Permission denied

v8.20:
$ LC_ALL=en_AU.UTF-8 mv a /etc
mv: cannot move ¿a¿ to ¿/etc/a¿: Permission denied

Yeah, the quotes from v8.20 didn't paste as quotes. gcc has
displayed this behaviour for years, and now coreutils will behave
the same. Welcome to the wonderful new world of screwed up quotes in
bug reports....
-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2013-04-11  9:45 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-11  9:45 xfstests: heads up about coreutils v8.20+ related failures Dave Chinner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox