-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Missing Math::BigInt Perl module silently breaks build #7074
Labels
bug
Something isn't working
Comments
Electroid
pushed a commit
that referenced
this issue
Nov 13, 2023
If src/codegen/create_hash_table can't be run due to some reason (e.g. due to missing Math::BigInt Perl module) the build continues but fails later, during bindings compilation, because the generated files are empty. Fix it by handling the failure to run create_hash_table properly. Closes #7074
@Electroid Please reopen this ticket, given that the PR was reverted? |
2 tasks
paperdave
pushed a commit
that referenced
this issue
Nov 17, 2023
…ld (#7110) * Stop swallowing errors from create_hash_table during the build If src/codegen/create_hash_table can't be run due to some reason (e.g. due to missing Math::BigInt Perl module) the build continues but fails later, during bindings compilation, because the generated files are empty. Fix it by handling the failure to run create_hash_table properly. Closes #7074 * [autofix.ci] apply automated fixes --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
ryoppippi
pushed a commit
to ryoppippi/bun
that referenced
this issue
Feb 1, 2024
…ld (oven-sh#7110) * Stop swallowing errors from create_hash_table during the build If src/codegen/create_hash_table can't be run due to some reason (e.g. due to missing Math::BigInt Perl module) the build continues but fails later, during bindings compilation, because the generated files are empty. Fix it by handling the failure to run create_hash_table properly. Closes oven-sh#7074 * [autofix.ci] apply automated fixes --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What version of Bun is running?
1.0.10+d85bd49d8
What platform is your computer?
Fedora 39, Linux 6.5.5-200.fc38.x86_64 x86_64 unknown
What steps can reproduce the bug?
dnf --installroot
)What is the expected behavior?
Bun build fails due to missing Math::BigInt Perl module used by code generator tool.
What do you see instead?
Generated files are nearly empty, and build mysteriously fails due to missing C++ symbols that weren't written into the generated files:
Additional information
No response
The text was updated successfully, but these errors were encountered: