-
Notifications
You must be signed in to change notification settings - Fork 57
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
Distill template can't be rendered on Mac #207
Comments
I'm having the same issue since I last built my blog 4 days ago. It seems to be a problem just with distill blogs rather than distill websites as generating a brand new Things I've tried which haven't worked:
Here is the error:
And here is my
|
This may have been a change we made in the last release to emit locale independent RSS feeds for blogs. I've reverted that change here: c9c083b. Could you try again installing from GitHub? |
I installed from c9c083b but I'm still getting the same error. Going down the debugger rabbit hole I can see the error is coming from this line in
Not sure how to fix it though. |
@csdaw What is the output of @vspinu Here is the code we are executing: tz <- Sys.timezone()
tz <- ifelse(is.na(tz), "UTC", tz)
lubridate::mdy("12/22/22", tz = tz, quiet = TRUE) What condition would cause lubridate to throw this error?
Is there a workaround you know of to ensure we don't halt when this occurs? |
I think I've solved the cause, its a bug with R v4.0.3. I have access to another Mac with R v4.0.2 and distill blog posts render fine with no errors. And then just now when I upgrade to R v.4.0.3 the timezone error is produced. Here is my
I can give you the |
Sure enough there were timezone related changes in R 4.0.3, from NEWS:
|
Also reported on the lubridate repo: tidyverse/lubridate#928 |
@relund There are some things for you to try/investigate here: tidyverse/lubridate#928. Let us know what you find. |
This appears to be the result of a bug in R 4.0.3 related to setting the TZDIR environment variable (which downstream code expects to be a path). Temporary workaround until this is fixed is here: cdcbd56 (note: timezone will be forced to UTC for parsing dates when this workaround is in effect) |
Eric, my workaround may have been too narrowly targeted. I just pushed a
fix to master that applies the workaound in all cases when running R 4.0.3.
Could you try installing w/ devtools from master again and let me know what
you find?
…On Sun, Nov 1, 2020 at 5:57 PM Eric Fletcher ***@***.***> wrote:
@jjallaire <https://github.com/jjallaire>
Can you please let me know how I can properly implement the temporary
workaround provided above? I tried reinstalling the distill package with
devtools as well as through Files / Plots / Packages / Help panel. Both
throw the following error:
[image: distill_blog_-main-_RStudio]
<https://user-images.githubusercontent.com/64165327/97817550-1be4f400-1c6b-11eb-841b-4b48b2423a36.png>
R:
1R version 4.0.3 (2020-10-10) -- "Bunny-Wunnies Freak Out"
Platform: x86_64-apple-darwin17.0 (64-bit)
RStudio:
1.4.953
"Wax Begonia"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#207 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAZPR7CFHERFE3UMMDLLETSNXRUNANCNFSM4TDPKIUQ>
.
|
Everything is working fine now. I am able to knit, build, and push to GitHub with no errors. Thank you! |
Great!
…On Sun, Nov 1, 2020 at 7:18 PM Eric Fletcher ***@***.***> wrote:
@jjallaire <https://github.com/jjallaire>
Everything is working fine now. I am able to knit, build, and push to
GitHub with no errors.
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#207 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAZPR6W4FY42ZW3M74VWHDSNX3HBANCNFSM4TDPKIUQ>
.
|
@jjallaire Is this has been reported/confirmed as a bug in R? |
Not that I'm aware of.
…On Mon, Nov 2, 2020 at 9:37 AM Vitalie Spinu ***@***.***> wrote:
This appears to be the result of a bug in R 4.0.3 related to setting the
TZDIR environment variable (which downstream code expects to be a path).
@jjallaire <https://github.com/jjallaire> Is this has been
reported/confirmed as a bug in R?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#207 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAZPRY7523UAGDGMNZHULTSN27ZRANCNFSM4TDPKIUQ>
.
|
lubridate has now the fix for timezone on macOS. So the workaround for #207 is no more needed.
Trying to knit the template
Gives error
Any hints on how to fix?
SessionInfo
The text was updated successfully, but these errors were encountered: