GNU bug report logs - #26783
Discrepancy with uptime added as part of "[PATCH] Document uptime."

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Prateek saxena <prateeksaxena2 <at> gmail.com>
Subject: bug#26783: closed (Re: bug#26783: Discrepancy with uptime added
 as part of "[PATCH] Document uptime.")
Date: Thu, 11 May 2017 11:57:03 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#26783: Discrepancy with uptime added as part of "[PATCH] Document uptime."

which was filed against the coreutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 26783 <at> debbugs.gnu.org.

-- 
26783: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26783
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Pádraig Brady <P <at> draigBrady.com>
To: Prateek saxena <prateeksaxena2 <at> gmail.com>
Cc: 26783-done <at> debbugs.gnu.org
Subject: Re: bug#26783: Discrepancy with uptime added as part of "[PATCH]
 Document uptime."
Date: Thu, 11 May 2017 12:56:14 +0100

On 11/05/17 06:40, Prateek saxena wrote:
> Hi Pádraig,
> 
> Thanks for replying.
> 
> When we can expect the fix.
> 
> Can we use the attached patch as workaround for time been. whether it
> would be sufficient.

Your patch is now applied at:
http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=v8.27-35-g82911c5
and will be made available in the next release.

thanks,
Pádraig

[Message part 3 (message/rfc822, inline)]

From: Prateek saxena <prateeksaxena2 <at> gmail.com>
To: bug-coreutils <at> gnu.org
Cc: jay <at> gnu.org
Subject: Discrepancy with uptime added as part of "[PATCH] Document uptime."
Date: Fri, 5 May 2017 11:33:14 +0530

[Message part 4 (text/plain, inline)]

Hi,

We found some discrepancy with uptime utility of coreutils package.

In my system using coreutils package with version 8.26, uptime output prints
PM/AM for 24 hour time format.

Logs:
------------------------------------------------------------------------------------
# uptime
 17:19pm  up 6 days 10:02,  1 user,  load average: 0.18, 0.14, 0.10

# uptime --version
uptime (GNU coreutils) 8.26
Copyright (C) 2016 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 Joseph Arceneaux, David MacKenzie, and Kaveh Ghazi
------------------------------------------------------------------------------------

The time is in 24 hours format and PM is not expected to be printed.
We can see latest patch for uptime utility, use of fprintftime to print the
time
 (http://lists.gnu.org/archive/html/bug-coreutils/2008-08/msg00035.html).

------------------------------------------------------------------------------------
diff --git a/src/uptime.c b/src/uptime.c
index 9e3384f..5bdc230 100644
--- a/src/uptime.c
+++ b/src/uptime.c
@@ -36,6 +36,7 @@
 #include "long-options.h"
 #include "quote.h"
 #include "readutmp.h"
+#include "fprintftime.h"

 /* The official name of this program (e.g., no `g' prefix).  */
 #define PROGRAM_NAME "uptime"
@@ -126,12 +127,10 @@ print_uptime (size_t n, const STRUCT_UTMP *this)
   uphours = (uptime - (updays * 86400)) / 3600;
   upmins = (uptime - (updays * 86400) - (uphours * 3600)) / 60;
   tmn = localtime (&time_now);
+  /* procps' version of uptime also prints the seconds field, but
+     previous versions of coreutils don't. */
   if (tmn)
-    printf (_(" %2d:%02d%s  up "),
-           ((tmn->tm_hour % 12) == 0 ? 12 : tmn->tm_hour % 12),
-           /* FIXME: use strftime, not am, pm.  Uli reports that
-              the german translation is meaningless.  */
-           tmn->tm_min, (tmn->tm_hour < 12 ? _("am") : _("pm")));
+    fprintftime (stdout, _(" %H:%M%P  up "), tmn, 0, 0);
   else
     printf (_(" ??:????  up "));
   if (uptime == (time_t) -1)
-----------------------------------------------------------------------------------

Can you please verify whether PM/AM should be avoided from uptime print.

Please find attach patch to remove AM/PM from the print for your reference.

Please provide your inputs for same.

Thanks & Regards,
Prateek

[Message part 5 (text/html, inline)]

[0001-uptime_am_pm_print_remove.patch (application/octet-stream, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.