bazel-contrib / bazel-contrib/rules_jvm_external

Error publishing Maven with proto dependencies during javadoc

Open
#448 2 comments 0 reactions 1 assignee Claimed by @shs96c View on GitHub
publishing
Dominant language
Java
Stars
373
Forks
301
Avg merge
7d 17h
Merged PRs (30d)
3

Description

We're trying to publish the bazel-bsp adapter to maven using the `java_export` rule, and having some troubles. Our experiment setting it up is here: https://github.com/JetBrains/bazel-bsp/tree/rules_jvm_external
When trying to build with `bazel run --stamp \
--define "maven_repo=file://$HOME/.m2/repository" //:bsp.publish`

it fails on the docs:
```
INFO: From Action bsp-docs.jar:
javadoc -cp external/maven/v1/https/repo.maven.apache.org/maven2/org/codehaus/mojo/animal-sniffer-annotations/1.18/animal-sniffer-annotations-1.18.jar:external/maven/v1/https/oss.sonatype.org/content/repositories/snapshots/ch/epfl/scala/bsp4j/2.0.0-M12%2B25-e4df1538-SNAPSHOT/bsp4j-2.0.0-M12%2B25-e4df1538-SNAPSHOT.jar:bazel-out/darwin-fastbuild/bin/bsp-project.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/guava/failureaccess/1.0.1/failureaccess-1.0.1.jar:external/maven/v1/https/repo.maven.apache.org/maven2/org/eclipse/xtend/org.eclipse.xtend.lib.macro/2.18.0/org.eclipse.xtend.lib.macro-2.18.0.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/guava/guava/28.1-jre/guava-28.1-jre.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/protobuf/protobuf-java/3.13.0/protobuf-java-3.13.0.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/code/gson/gson/2.8.5/gson-2.8.5.jar:external/maven/v1/https/repo.maven.apache.org/maven2/org/eclipse/lsp4j/org.eclipse.lsp4j.generator/0.8.1/org.eclipse.lsp4j.generator-0.8.1.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/guava/listenablefuture/9999.0-empty-to-avoid-conflict-with-guava/listenablefuture-9999.0-empty-to-avoid-conflict-with-guava.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/j2objc/j2objc-annotations/1.3/j2objc-annotations-1.3.jar:external/maven/v1/https/repo.maven.apache.org/maven2/org/eclipse/xtend/org.eclipse.xtend.lib/2.18.0/org.eclipse.xtend.lib-2.18.0.jar:external/maven/v1/https/repo.maven.apache.org/maven2/org/eclipse/lsp4j/org.eclipse.lsp4j.jsonrpc/0.8.1/org.eclipse.lsp4j.jsonrpc-0.8.1.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/code/findbugs/jsr305/3.0.2/jsr305-3.0.2.jar:external/maven/v1/https/repo.maven.apache.org/maven2/org/eclipse/xtext/org.eclipse.xtext.xbase.lib/2.19.0/org.eclipse.xtext.xbase.lib-2.19.0.jar:external/maven/v1/https/repo.maven.apache.org/maven2/com/google/errorprone/error_prone_annotations/2.3.2/error_prone_annotations-2.3.2.jar:external/maven/v1/https/repo.maven.apache.org/maven2/org/checkerframework/checker-qual/2.8.1/checker-qual-2.8.1.jar -html5 --frames -notimestamp -use -quiet -Xdoclint:-missing -encoding UTF8 -d /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/output-dir12359829166935189021 /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/Constants.java /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/Uri.java /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/BazelBspServer.java /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/BepServer.java /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/ActionGraphParser.java /var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/Main.java
javadoc: warning - You have specified to generate frames, by using the --frames option.
The default is currently to not generate frames and the support for
frames will be removed in a future release.
To suppress this warning, remove the --frames option and avoid the use of frames.
/var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/BazelBspServer.java:10: error: package com.google.devtools.build.lib.analysis does not exist
import com.google.devtools.build.lib.analysis.AnalysisProtos;
^
/var/folders/5k/4b5my29x26q_dhrys3ghf9wh0000gn/T/unpacked-sources1483227975236051443/org/jetbrains/bsp/BazelBspServer.java:11: error: package com.google.devtools.build.lib.query2.proto.proto2api does not exist
import com.google.devtools.build.lib.query2.proto.proto2api.Build;
^
[...]
```

From Slack discussion:
it looks like that proto is from `@io_bazel/src/main/java/com/google/devtools/build/lib/buildeventstream/proto:build_event_stream_java_proto` and we should expect them to be in the `maven_project_jar`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.