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

rust: don't build out of tree #22804

Merged
merged 1 commit into from
Dec 17, 2024
Merged

rust: don't build out of tree #22804

merged 1 commit into from
Dec 17, 2024

Conversation

ognevny
Copy link
Collaborator

@ognevny ognevny commented Dec 13, 2024

this is not supported upstream, but a workaround was used. set build-dir in config.toml, make dest dirs names contain $MSYSTEM to match previous behavior

asking @filnet as a workaround's author is it ok to do so

@ognevny ognevny force-pushed the rust-builddir branch 3 times, most recently from 9524f85 to 88153f7 Compare December 13, 2024 17:24
@filnet
Copy link
Contributor

filnet commented Dec 13, 2024

asking @filnet as a workaround's author is it ok to do so

I am not sure that I understand your question.
But note that I have currently lost interest in Rust (might come back some day though).

@ognevny
Copy link
Collaborator Author

ognevny commented Dec 13, 2024

I meant will be fine to drop workaround for building outside of rustc src and build inside of it?

@filnet
Copy link
Contributor

filnet commented Dec 15, 2024

I meant will be fine to drop workaround for building outside of rustc src and build inside of it?

I think that workaround was already in place when I got involved in Rust maintenance.

@ognevny
Copy link
Collaborator Author

ognevny commented Dec 15, 2024

oh, I didn't look carefully at blame...

@ognevny
Copy link
Collaborator Author

ognevny commented Dec 17, 2024

ok, seems it's fine to do so...

@ognevny ognevny merged commit c71a4a6 into msys2:master Dec 17, 2024
8 checks passed
@ognevny ognevny deleted the rust-builddir branch December 17, 2024 05:46
@jeremyd2019
Copy link
Member

jeremyd2019 commented Dec 17, 2024

I was going to mention, we were running right up against the MAX_PATH limits for this package, and moving the build directory deeper would probably make it too deep. Sure enough:

BUG: compat/win32/fscache.c:128: Trying to allocate fsentry for long path 'B/src/rustc-1.83.0-src/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rustlib/rustc-src/rust/build-CLANGARM64/aarch64-pc-windows-gnullvm/stage0-sysroot/lib/rus

this seems even weirder than that though, some sort of circular link that git is unhappy with. (I'm running rmdir /s B to try to help get this unstuck)

@ognevny
Copy link
Collaborator Author

ognevny commented Dec 17, 2024

oh, but what to do with out-of-tree builds? there should be a better way maybe :(

@jeremyd2019
Copy link
Member

I opened git-for-windows/git#5320 for the git BUG output, git clean seems to be following directory junctions rather than simply removing them

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

Successfully merging this pull request may close these issues.

3 participants