bug-coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#7325: new test failure due to non-portability of printf formats like


From: Jim Meyering
Subject: bug#7325: new test failure due to non-portability of printf formats like %05.3s
Date: Sat, 13 Nov 2010 22:04:31 +0100

Paul Eggert wrote:

> On 11/13/2010 02:45 AM, Jim Meyering wrote:
>
>> Here's what I'm planning.
>
> Looks good, thanks, except for NEWS:
>
>> +  a nanosecond-precision floating point time stamp for %X use %.X;
>
> It's fixed-point, not floating point, so I'd remove "floating point".
>
> And in coreutils.texi:
>
>> +precision, @command{stat} uses 9 digits, so @samp{%.X} is equivalent to
>> address@hidden  When discarding excess precision, time stamps are truncated
>
> There's a period missing after the "@samp{%.9X}" (though ideally the sentence
> would be reworded so as to avoid the need for any period after the sample,
> as the "%.9X." looks bad when it's printed).

"%.9X". doesn't look that bad to me (neither in .info nor in .pdf)
How's this, Paul?

>From f70c7b785b93dd436788d34827b209453157a6f2 Mon Sep 17 00:00:00 2001
From: Paul Eggert <address@hidden>
Date: Sat, 13 Nov 2010 22:02:29 +0100
Subject: [PATCH] doc: tweak NEWS and coreutils.texi

* doc/coreutils.texi (stat invocation): Add sentence-ending period.
* NEWS: Correct stat change description: s/floating point //.
* cfg.mk (old_NEWS_hash): Update, to match this NEWS change.
---
 NEWS               |    8 ++++----
 cfg.mk             |    2 +-
 doc/coreutils.texi |    2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/NEWS b/NEWS
index a40a0aa..1da7db2 100644
--- a/NEWS
+++ b/NEWS
@@ -26,10 +26,10 @@ GNU coreutils NEWS                                    -*- 
outline -*-

   stat's %X, %Y, and %Z directives once again print only the integer
   part of seconds since the epoch.  This reverts a change from
-  coreutils-8.6, that was deemed unnecessarily disruptive.  To obtain
-  a nanosecond-precision floating point time stamp for %X use %.X;
-  if you want (say) just 3 fractional digits, use %.3X.  Likewise
-  for %Y and %Z.
+  coreutils-8.6, that was deemed unnecessarily disruptive.
+  To obtain a nanosecond-precision time stamp for %X use %.X;
+  if you want (say) just 3 fractional digits, use %.3X.
+  Likewise for %Y and %Z.

   stat's new %W format directive would print floating point seconds.
   However, with the above change to %X, %Y and %Z, we've made %W work
diff --git a/cfg.mk b/cfg.mk
index 244358c..7651ab2 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -27,7 +27,7 @@ bootstrap-tools = autoconf,automake,gnulib,bison
 # Now that we have better tests, make this the default.
 export VERBOSE = yes

-old_NEWS_hash = 24f3cbd2c625a297dc4cfb1b076eeaae
+old_NEWS_hash = e2a254a0d4c81397994ea10a15663ac3

 # Add an exemption for sc_makefile_at_at_check.
 _makefile_at_at_check_exceptions = ' && !/^cu_install_program =/'
diff --git a/doc/coreutils.texi b/doc/coreutils.texi
index 6a4257f..1373f94 100644
--- a/doc/coreutils.texi
+++ b/doc/coreutils.texi
@@ -10721,7 +10721,7 @@ stat invocation
 print after the decimal point.  For example, @samp{%.3X} outputs the
 last access time to millisecond precision.  If a period is given but no
 precision, @command{stat} uses 9 digits, so @samp{%.X} is equivalent to
address@hidden  When discarding excess precision, time stamps are truncated
address@hidden  When discarding excess precision, time stamps are truncated
 toward minus infinity.

 @example
--
1.7.3.2.4.g60aa9





reply via email to

[Prev in Thread] Current Thread [Next in Thread]