Subject: bug#26783: closed (Re: bug#26783: Discrepancy with uptime added as part of "[PATCH] Document uptime.")

From: [email-masked] (GNU bug Tracking System)
Date: Thu, 11 May 2017 11:57:03 +0000

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 [email-masked].

-- 
26783: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26783
GNU Bug Tracking System
Contact [email-masked] with problems

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

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

Subject: Discrepancy with uptime added as part of "[PATCH] Document uptime."
From: Prateek saxena <[email-masked]>
To: [email-masked]
Cc: [email-masked]
Date: Fri, 5 May 2017 11:33:14 +0530
List-Id: <debbugs-submit.debbugs.gnu.org>

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