[krbdev.mit.edu #7458] add more strftime format strings for klist

Russ Allbery via RT rt-comment at krbdev.mit.edu
Thu Dec 20 15:44:25 EST 2012


"Tom Yu via RT" <rt-comment at krbdev.mit.edu> writes:

> It's also worth noting that there's an fallback to use sprintf to format
> using the equivalent of "%d/%m/%Y %H:%M" if none of the strftime formats
> works.  We should probably remove this fallback sprintf format because
> it is ambiguous (European vs US date order).  Can we also assume a
> working strftime as well, considering that we appear to attempt to build
> a replacement if it's not found?

strftime is sufficiently universal that gnulib doesn't even bother to cope
with its absence.  I've never encountered a system where someone wanted to
build my software that didn't have it.

gnulib says this about the Windows implementation:

    The Windows C runtime library (which is used by MinGW) does not
    support the %e specifier (and possibly the other more recent SUS
    specifiers too, i.e., %C, %D, %h, %n, %r, %R, %t, and %T).

My recommendation would be to even remove the replacement version and
assume that the system has it, but ensure that you have a fallback format
that doesn't include any of the above specifiers.

-- 
Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>



More information about the krb5-bugs mailing list