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

[Bazel] Update for bazel 7.1.2 #80

Merged
merged 6 commits into from
Jun 11, 2024

Conversation

tinder-kaijing
Copy link
Collaborator

to make it work for Bazel 7.1.2, to fix below error found in ios-tooling:

ERROR: /private/var/tmp/_bazel_kaijing/08f4a62c75916b9cf4fa5a910738a7ee/external/GitQuery/core/BUILD.bazel:3:15: in kt_jvm_library rule @@GitQuery//core:core: 
Traceback (most recent call last):
        File "/private/var/tmp/_bazel_kaijing/08f4a62c75916b9cf4fa5a910738a7ee/external/io_bazel_rules_kotlin/kotlin/internal/jvm/impl.bzl", line 216, column 36, in kt_jvm_library_impl
                _kt_jvm_produce_jar_actions(ctx, "kt_jvm_library") if ctx.attr.srcs else export_only_providers(
        File "/private/var/tmp/_bazel_kaijing/08f4a62c75916b9cf4fa5a910738a7ee/external/io_bazel_rules_kotlin/kotlin/internal/jvm/compile.bzl", line 491, column 50, in kt_jvm_produce_jar_actions
                outputs_struct = _run_kt_java_builder_actions(
        File "/private/var/tmp/_bazel_kaijing/08f4a62c75916b9cf4fa5a910738a7ee/external/io_bazel_rules_kotlin/kotlin/internal/jvm/compile.bzl", line 729, column 32, in _run_kt_java_builder_actions
                _run_merge_jdeps_action(
        File "/private/var/tmp/_bazel_kaijing/08f4a62c75916b9cf4fa5a910738a7ee/external/io_bazel_rules_kotlin/kotlin/internal/jvm/compile.bzl", line 264, column 70, in _run_merge_jdeps_action
                inputs = depset(jdeps, transitive = [depset([], transitive = [dep.transitive_deps for dep in deps])])
Error: 'JavaInfo' value has no field or method 'transitive_deps'
Available attributes: _compile_time_java_dependencies, _constraints, _neverlink, _transitive_full_compile_time_jars, annotation_processing, api_generating_plugins, compilation_info, compile_jars, full_compile_jars, java_outputs, module_flags_info, outputs, plugins, runtime_output_jars, source_jars, transitive_compile_time_jars, transitive_native_libraries, transitive_runtime_jars, transitive_source_jars
ERROR: /private/var/tmp/_bazel_kaijing/08f4a62c75916b9cf4fa5a910738a7ee/external/GitQuery/core/BUILD.bazel:3:15: Analysis of target '@@GitQuery//core:core' failed
Use --verbose_failures to see the command lines of failed build steps.
ERROR: Analysis of target '//src/Proto/Tests:ProtoUnitTests' failed; build aborted: Analysis failed

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

github-actions bot commented Jun 7, 2024

Unit Test Results

16 tests   16 ✔️  9s ⏱️
  5 suites    0 💤
  5 files      0

Results for commit 8fa50d8.

♻️ This comment has been updated with latest results.

@tinder-kaijing tinder-kaijing force-pushed the fix_missing_transitive_deps_error branch from 1aa5e49 to 8a8febe Compare June 10, 2024 17:29
@tinder-kaijing tinder-kaijing force-pushed the fix_missing_transitive_deps_error branch from b5b0d18 to 9ed0b49 Compare June 10, 2024 18:48
@tinder-aminghadersohi tinder-aminghadersohi merged commit eb8a6be into main Jun 11, 2024
3 checks passed
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.

4 participants