Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release 2.5.1 #232

Closed
wants to merge 4 commits into from
Closed

Release 2.5.1 #232

wants to merge 4 commits into from

Conversation

lionel-
Copy link
Member

@lionel- lionel- commented Sep 26, 2023

No description provided.

Fixes:

── Error ('test-timezone.R:7:5'): with_timezone: changes the time zone ─────────
Error in `format.POSIXlt(as.POSIXlt(x, tz), format, usetz, ...)`: output string exceeded 2048 bytes
Backtrace:
▆
1. ├─testthat::expect_true(...) at test-timezone.R:7:4
2. │ └─testthat::quasi_label(enquo(object), label, arg = "object")
3. │ └─rlang::eval_bare(expr, quo_get_env(quo))
4. ├─... %in% c("PDT", "PST")
5. ├─withr::with_timezone("US/Pacific", format(Sys.time(), "%Z"))
6. │ └─withr::with_envvar(c(TZ = tz), code)
7. │ └─base::force(code)
8. ├─base::format(Sys.time(), "%Z")
9. └─base::format.POSIXct(Sys.time(), "%Z")
10. ├─base::structure(...)
11. └─base::format.POSIXlt(as.POSIXlt(x, tz), format, usetz, ...)

Notes from Kurt:

The errors for r-devel-linux-x86_64-debian-gcc (which will eventually
also show for the other Debian-based checks) are from a recent system
upgrade of the tzdata package which dropped the legacy timezone
symlinks that are not following the current rule of using the
geographical region
(continent or ocean) and city name (the "backward" links from tzdata),
such as

  GB            -> Europe/London
  NZ            -> Pacific/Auckland
  US/Eastern    -> America/New_York
  US/Pacific    -> America/Los_Angeles
@lionel- lionel- changed the base branch from main to release-2.5.0 September 26, 2023 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant