On Wed, Jun 04, 2014 at 03:25:36PM +0200, Benno Schulenberg wrote: > > On Wed, Jun 4, 2014, at 14:56, Theodore Ts'o wrote: > > On Wed, Jun 04, 2014 at 02:23:21PM +0200, Benno Schulenberg wrote: > > > > > > Running 'make check' now, I see these messages along the way: > > > > > > mktemp: too few X's in template `r_64bit_big_expand.XXXXXX.tmp' > > This "too few X's" is misleading -- what it intends to say is that it > sees zero X's in the final component of the name, "tmp". In the posix specification, "component" has a specific meaning, and it means the last part of a pathname. For example the "basename" command is described as removing all of the leading directory components. NAME_LEN is defined to be the maximum length of a pathname component, etc. I don't think your reading of the word "component" is correct. > > Hmm... what OS and if Linux, version of coreutils are you using? > > Ubuntu Lucid, 2010.04, mktemp --version | head -1 > mktemp (GNU coreutils) 8.13 OK, interesting. It seems to work just fine using coreutils 8.21 (Debian Testing). And I just checked on one of my Debian Stable machines, which is using coreutils 8.13-3.5, and it's working there. % mktemp /tmp/r_64bit_big_expand.XXXXXX.tmp /tmp/r_64bit_big_expand.QAOy36.tmp ... and I just tried it on a Ubuntu 12.04 machine, and it accepts it too: 8% cat /etc/lsb-release DISTRIB_ID=Ubuntu DISTRIB_RELEASE=12.04 DISTRIB_CODENAME=precise DISTRIB_DESCRIPTION="Ubuntu 12.04.4 LTS" % mktemp -V mktemp (GNU coreutils) 8.13 Copyright (C) 2011 Free Software Foundation, Inc. License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>. This is free software: you are free to change and redistribute it. There is NO WARRANTY, to the extent permitted by law. Written by Jim Meyering and Eric Blake. % mktemp /tmp/r_64bit_big_expand.XXXXXX.tmp /tmp/r_64bit_big_expand.6yNF5W.tmp Hmm... I just checked on a Mac OSX machine, which is using a FreeBSD userspace iirc, and its man page explicitly states that six X's must be at the end of the pathname. And while "mktemp /tmp/foo.XXXXXX.bar" doesn't fail, it returns "/tmp/foo.XXXXXX.bar", and if /tmp/foo.XXXXXX.bar already exists, only then will it fail with a "File exists" error message. So it looks like allowing the placement of the XXX characters in the middle of the filename is a GNU coreutils extension, and it must have been introduced very shortly after Ubuntu 10.04 snapshotted its coreutils package from Debian. And since most of our developers aren't using something quite as antique as Ubuntu 10.04, and while we do have someone doing test compiles on Mac OSX, the fact that mktemp generally works there means that Andreas never saw any problems with it. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
