From: Lars Ingebrigtsen <larsi@gnus.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 37130@debbugs.gnu.org, vincent@vinc17.net
Subject: bug#37130: 27.0.50; Tar mode should use the ISO8601 format to display the timestamps (or support TIME_STYLE or locales)
Date: Sun, 25 Aug 2019 08:03:21 +0200 [thread overview]
Message-ID: <87d0gt93yu.fsf@gnus.org> (raw)
In-Reply-To: <83ef1cb7nf.fsf@gnu.org> (Eli Zaretskii's message of "Fri, 23 Aug 2019 11:36:20 +0300")
Eli Zaretskii <eliz@gnu.org> writes:
>> I think it makes sense to follow GNU tar's lead here, so I've now done
>> this on the trunk.
>
> Yuck! what an ugly format!
Yes, I didn't check what GNU tar really did, so I just put in ISO8601.
I've now checked and replicated the format, which looks nicer.
>> > Other formats (e.g. one corresponding to the locales) could be used
>> > via an option; in particular, the support of the TIME_STYLE
>> > environment variable (see coreutils manual) would be a good idea.
>>
>> I think that sounds like overkill for tar-mode, but perhaps there should
>> be a general function somewhere for using TIME_STYLE throughout Emacs
>> whenever time/dates are displayed.
>
> I think we should just make the format used for the time stamp a user
> option, that would be a good-enough solution. It will also provide a
> "fire escape" to get the old behavior for those, like me, who might
> dislike the ISO8601 format. We must provide such
> backward-compatibility options anyway when we change behavior in
> incompatible ways.
We don't, in general, add such options unless we think it's going to be
used a lot. For tar mode, my feeling is that nobody will ever care
enough to bother to customise it.
> Finally, this change should be in NEWS (including the to-be-added
> backward-compatibility shim).
Or even notice it, which is why I didn't mention it in NEWS. It seem
like a not particularly NEWS-worthy item to me.
If you think this really warrants a user customisation and a NEWS item,
I'd rather just revert the change and mark this bug as a WONTFIX,
because it doesn't seem like that to me.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
next prev parent reply other threads:[~2019-08-25 6:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-21 14:31 bug#37130: 27.0.50; Tar mode should use the ISO8601 format to display the timestamps (or support TIME_STYLE or locales) Vincent Lefevre
2019-08-23 3:46 ` Lars Ingebrigtsen
2019-08-23 8:36 ` Eli Zaretskii
2019-08-25 6:03 ` Lars Ingebrigtsen [this message]
2019-08-25 7:41 ` Eli Zaretskii
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87d0gt93yu.fsf@gnus.org \
--to=larsi@gnus.org \
--cc=37130@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=vincent@vinc17.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.