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

TARGET_GRAPH_CALCULATION_IMPL not set (internal error) #257

Closed
sluongng opened this issue May 13, 2023 · 6 comments
Closed

TARGET_GRAPH_CALCULATION_IMPL not set (internal error) #257

sluongng opened this issue May 13, 2023 · 6 comments
Assignees

Comments

@sluongng
Copy link
Contributor

I got this from using Buck2 from main branch today

TARGET_GRAPH_CALCULATION_IMPL not set (internal error)
TARGET_GRAPH_CALCULATION_IMPL not set (internal error)
2 BUILDS FAILED

Reverting 9bc4bbc and then f104c0a fixed it.

@stepancheg
Copy link
Contributor

Please confirm you are on macOS and build with cargo.

It broke a few days ago.

The problem is described here mmastrac/rust-ctor#220 now looking for a fix.

Revert may fix this issue but there are multiple places which can trigger this problem. Now looking for proper fix.

@stepancheg stepancheg self-assigned this May 13, 2023
@stepancheg
Copy link
Contributor

Temporary workaround is to build in debug more switching opt-level to 0.

@sluongng
Copy link
Contributor Author

Yeah I am on MacOS + arm64 and built buck2 binary using Cargo.
Cargo is still the recommended way to build buck2 binary correct? (the doc currently said to avoid using buck2-compiled buck2 as it's missing jemmaloc and other goodies)

@stepancheg
Copy link
Contributor

Cargo is still the recommended way to build buck2 binary correct?

Yes. Published binaries are built with Cargo.

$CARGO build --release --bin buck2 --target ${{ matrix.target.triple }}

(these builds are not affected by this problem, likely different version of linker or something)

@sluongng
Copy link
Contributor Author

I think the current "latest" release model is hard for us to track when/how to upgrade the binary, thus I rely on building the buck2 binary using the prelude git submodule commit instead.

I would much prefer the release model watchman is using, which could then be packaged by Homebrew.

@stepancheg
Copy link
Contributor

Fixed in 7947156.

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

No branches or pull requests

2 participants