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

Use non-prebuild Java toolchain configuration #451

Merged
merged 5 commits into from
Nov 24, 2023

Conversation

avdv
Copy link
Member

@avdv avdv commented Nov 23, 2023

On NixOS we cannot use the pre-built ijar and singlejar tools as they are not statically build and fail to find the dynamic linker.

Fixes #278

On NixOS we cannot use the pre-built `ijar` and `singlejar` tools as they are not
statically build and fail to find the dynamic linker.

Fixes #278
@avdv avdv requested a review from benradf as a code owner November 23, 2023 17:16
We were still on version 4.0.0 but already got version 5.5.1 in the resolved dependency graph.

We need to use at least version 6.1.0 since former versions of rules_java failed to declare a
dependency on rules_license which is now referenced.
Copy link
Member

@aherrmann aherrmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you!

This fixes CI errors for toolchains/java:
```
external/bazel_tools/tools/osx/xcode_configure.bzl:93:14: Invoking xcodebuild failed, developer dir: /Applications/Xcode_12.4.app/Contents/Developer ,return code 1, stderr: src/main/tools/process-wrapper-legacy.cc:80: "execvp(xcrun, ...)": No such file or directory
```
Bazel requires `libtool` for creating static archives on Darwin instead of the `ar` tool.
@avdv avdv added the merge-queue merge on green CI label Nov 24, 2023
@mergify mergify bot merged commit ae279df into master Nov 24, 2023
15 checks passed
@mergify mergify bot deleted the use-nonprebuilt-java-tools branch November 24, 2023 12:46
@mergify mergify bot removed the merge-queue merge on green CI label Nov 24, 2023
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.

Java toolchain: remote java tools don't work on NixOS
2 participants