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

Update Dockerfile #96

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update Dockerfile #96

wants to merge 2 commits into from

Conversation

meztez
Copy link

@meztez meztez commented Dec 7, 2024

for stringi, pak will install icu-dev if not found

for stringi, pak will install icu-dev if not found
@meztez meztez closed this Dec 7, 2024
@gaborcsardi
Copy link
Collaborator

So the update is not needed? That's why you closed it?

@meztez
Copy link
Author

meztez commented Dec 7, 2024

its icu-libs for stringi, i'm spending the weekend migrating all the wordload over to alpine on aws.

Anything that will depend on stringi will install icu-dev if not specified. stringi stopped shipping its own icu subset last year from what I read.

@meztez meztez reopened this Dec 7, 2024
@gaborcsardi
Copy link
Collaborator

Yes, we could probably turn off the automatic system dependency installation on r-minimal until pak knows about run-time and build-time dependencies.

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.

2 participants