-
Notifications
You must be signed in to change notification settings - Fork 305
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
rust_v8 0.98.0,0.99.0 build failure #1541
Comments
Are you able to get the stdout/stderr from the command failure? |
still seeing the same error log in the 0.99.0 release build, how should I get the stdout/stderr logs?
relates to Homebrew/homebrew-core#178635 |
I think you can change this code: assert!(Command::new(gn()).arg(format!("--script-executable={}", python())).arg("args").arg(gn_out_dir).arg("--list").status().unwrap().success()); to something like let output = Command::new(gn()).arg(format!("--script-executable={}", python())).arg("args").arg(gn_out_dir).arg("--list").output().unwrap();
std::io::stderr().write_all(&output.stdout).unwrap();
std::io::stderr().write_all(&output.stderr).unwrap();
assert!(output.status.success()); If there's a way to reproduce the homebrew build locally I can also try to debug it at some point. |
you can download https://raw.githubusercontent.com/Homebrew/homebrew-core/f1dfb9ef2aa27c6767f26ee10d07158a5cc435d9/Formula/d/deno.rb and run |
@chenrui333 sorry i'm getting this
|
Ok fixed the above, turns out the needed command is |
Does |
I can actually dump out the docs from CI runner, is that good? |
@chenrui333 |
@chenrui333 can you try setting |
yeah, totally. |
somehow it builds fine now 😓 |
since I can no longer reproduce this issue, gonna close it now. Thanks! |
👋 while running deno 1.45.3 build, found some build failure as below:
full build log, https://github.com/Homebrew/homebrew-core/actions/runs/10058665735/job/27807237240?pr=178170
relates to Homebrew/homebrew-core#178170
The text was updated successfully, but these errors were encountered: