On Fri, Nov 4, 2016 at 10:36 PM, Paul Eggert <egg...@cs.ucla.edu> wrote: > Jim Meyering wrote: >> >> The first one is a clear bug fix. > > > Oops, my bad. Thanks for catching that. I installed a simpler fix (first > attached patch). > >> For the second, since --no-time (-T) never worked (-m does), >> I could also just remove its entry from longopts. We've >> done without it for so long, there's little point to adding >> an undocumented --no-time, now. > > > Thanks, I like that idea and installed the 2nd attached patch. > > In rereading the code I noticed other problems likely to bite after the year > 2038 (3rd attached patch, also installed). This stuff is a pain, as it won't > see realistic testing for another 20 years or so and I don't see easy test > cases for it, partly because GNU/Linux seems to mishandles these time stamps > now on my platform (Fedora 24 x86-64; I'll try to file a bug report about > this to Fedora). > > Boldly marking the bug as done.
All good. Thank you.