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

Revert "fix(turbopack): Use vergen-git2 instead of shadow-rs for napi and next-api crates to fix stale git lock files" #76879

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

bgw
Copy link
Member

@bgw bgw commented Mar 6, 2025

Reverts #76773

Looks like we broke compilation for our linux swc builds for musl/gnu

Closes PACK-4082

@ijjk ijjk added created-by: Turbopack team PRs by the Turbopack team. Turbopack Related to Turbopack with Next.js. labels Mar 6, 2025
@ijjk ijjk merged commit ee5c7d3 into canary Mar 6, 2025
13 of 22 checks passed
@ijjk ijjk deleted the revert-76773-bgw/vergen branch March 6, 2025 19:31
bgw added a commit that referenced this pull request Mar 7, 2025
…for napi and next-api crates to fix stale git lock files" (#76879)"

This reverts commit ee5c7d3.
bgw added a commit that referenced this pull request Mar 7, 2025
…) for napi and next-api crates to fix stale git lock files (#76889)

This does not depend on libgit2 which apparently breaks our builds on
some platforms. Instead it uses the git CLI using my PR:
rustyhorde/vergen#406

Original PR: #76773
Revert: #76879

Manually triggered CI run to try to build all platforms:
https://github.com/vercel/next.js/actions/runs/13711866131
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
created-by: Turbopack team PRs by the Turbopack team. Turbopack Related to Turbopack with Next.js.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants