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

The glibc patch for the Dockerfile was only applied to the builder #2335

Merged
merged 7 commits into from
Oct 26, 2022

Conversation

rafaeltab
Copy link
Contributor

As the title explains, the Glibc patch for the Dockerfile in the with-docker example was only applied to the builder stage and not the installer stage. This caused an error during the docker builds.

I also updated the docs to include the updated Dockerfile.

Closes #2309 and #2286
PR #2228 previously fixed this for the builder stage, but not for the installer stage

@vercel
Copy link

vercel bot commented Oct 25, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

5 Ignored Deployments
Name Status Preview Updated
examples-basic-web ⬜️ Ignored (Inspect) Oct 26, 2022 at 7:42PM (UTC)
examples-designsystem-docs ⬜️ Ignored (Inspect) Oct 26, 2022 at 7:42PM (UTC)
examples-kitchensink-blog ⬜️ Ignored (Inspect) Oct 26, 2022 at 7:42PM (UTC)
examples-native-web ⬜️ Ignored (Inspect) Oct 26, 2022 at 7:42PM (UTC)
examples-svelte-web ⬜️ Ignored (Inspect) Oct 26, 2022 at 7:42PM (UTC)

@vercel
Copy link

vercel bot commented Oct 25, 2022

@rafaeltab is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@mehulkar mehulkar added area: docs Improvements or additions to documentation area: examples Improvements or additions to examples labels Oct 25, 2022
Copy link
Member

@chris-olszewski chris-olszewski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing up the example!

@verygoodsoftwarenotvirus

I can confirm I've applied the core fix made in this change locally and it works. The confusion this caused cost me something like a day and a half of productivity, so for whatever my comment is worth, I think this should be merged.

@chris-olszewski chris-olszewski merged commit a2a04ed into vercel:main Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs Improvements or additions to documentation area: examples Improvements or additions to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

with-docker build failure yarn install
4 participants