unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Mattias Engdegård" <mattiase@acm.org>
To: "Basil L. Contovounesios" <contovob@tcd.ie>
Cc: 45818-done@debbugs.gnu.org
Subject: bug#45818: 28.0.50; Test solar-sunrise-sunset fails
Date: Tue, 12 Jan 2021 22:02:15 +0100	[thread overview]
Message-ID: <59D8F21E-DA21-4F8E-AB91-550B3C154985@acm.org> (raw)
In-Reply-To: <87ft35j4xl.fsf@tcd.ie>

12 jan. 2021 kl. 21.06 skrev Basil L. Contovounesios <contovob@tcd.ie>:

> So I'm guessing dst-adjust-time adjusts for DST in my locale (based on
> some cache), but not yours?  The following change lets the test succeed
> for me:

Thank you, and I agree that it's probably the pragmatic choice. Much as I would have liked to track down and explain the exact reason for the observed phenomenon, I gave up and pushed your suggested patch.

If I would hazard a guess, it's related to the fact that Ireland uses daylight saving in winter (with opposite adjustment) rather than in summer; this may trigger a daylight-saving adjustment for the dates in question (late December). The fact that IST is used for both Irish and Indian time may be significant or a mere coincidence; it certainly didn't help matters.

Date and time calculations can be both infuriatingly and delightfully messy, but in this case the code didn't really help either, with way too many implicit input parameters -- global variables, caches, system settings, time-zone files, and so on. Not something to make a test writer happy!

Thanks again for the report, and for persevering!






  reply	other threads:[~2021-01-12 21:02 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-12 16:28 bug#45818: 28.0.50; Test solar-sunrise-sunset fails Basil L. Contovounesios
2021-01-12 18:50 ` Mattias Engdegård
2021-01-12 20:06   ` Basil L. Contovounesios
2021-01-12 21:02     ` Mattias Engdegård [this message]
2021-01-12 22:09       ` Basil L. Contovounesios
2021-01-13 12:12         ` Mattias Engdegård
2021-01-13 13:04           ` Stefan Kangas
2021-01-13 13:29             ` Mattias Engdegård
2021-01-13 19:03               ` Basil L. Contovounesios

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=59D8F21E-DA21-4F8E-AB91-550B3C154985@acm.org \
    --to=mattiase@acm.org \
    --cc=45818-done@debbugs.gnu.org \
    --cc=contovob@tcd.ie \
    /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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).