-
Notifications
You must be signed in to change notification settings - Fork 38
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
[DH-317] Install quarto in Dev hub (dashboard image) #5829
Conversation
I recommend not using R code to script the installation of quarto. I also think this method will fail because I believe the user running install.R in repo2docker doesn't have root privileges. I believed this passed CI because it is the secondary image and hubploy doesn't build secondary images. repo2docker doesn't make quarto installation easy because quarto is not in Ubuntu jammy repos
Lastly, it'd probably worth creating an issue to request that quarto be installed (along with RStudio, shiny, etc.) in repo2docker's R build packs. |
Actually, there is a quarto package in conda, https://anaconda.org/conda-forge/quarto. So it should just go into the environment.yml. |
Thanks @ryanlovett for the suggestion. I just made a commit which added the quarto package to the environment.yml file. That seems to be a simple process to enable quarto. Good to know! |
@@ -1,5 +1,6 @@ | |||
#!/usr/bin/env r | |||
#!/usr/bin/env R |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was wondering about this since it is technically correct, but hasn't been a problem. This script is evidently passed to R rather than made executable and run directly. The upstream example doesn't even have a hashbang.
@ryanlovett I will merge the PR for now. Will fix it if there are issues. |
So that Sophie can explore quarto dashboards in Dev hub