New issue
Advanced search Search tips

Issue 601800 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Android Cronet Builders failing compile

Project Member Reported by hartma...@chromium.org, Apr 8 2016

Issue description

see bots:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20ARM64%20Builder%20%28dbg%29

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20x86%20Builder%20%28dbg%29


Specific failing builds:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20x86%20Builder%20%28dbg%29/builds/17598

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20ARM64%20Builder%20%28dbg%29/builds/20994

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20ARM64%20Builder%20%28dbg%29/builds/20995



build log failure snippet:

FAILED: python ../../components/cronet/tools/generate_javadoc.py --output-dir cronet --input-dir ../../components/cronet --overview-file cronet/README.md.html --readme-file ../../components/cronet/README.md --depfile gen/components/cronet/android/generate_javadoc.d --lib-java-dir lib.java/components/cronet/android
Traceback (most recent call last):
  File "../../components/cronet/tools/generate_javadoc.py", line 75, in <module>
    sys.exit(main())
  File "../../components/cronet/tools/generate_javadoc.py", line 65, in main
    GenerateJavadoc(options)
  File "../../components/cronet/tools/generate_javadoc.py", line 48, in GenerateJavadoc
    raise build_utils.CalledProcessError(working_dir, javadoc_cmd, stdout)
build_utils.CalledProcessError: Command failed: ( cd /mnt/data/b/build/slave/Android_Cronet_ARM64_Builder__dbg_/build/src/components/cronet/android/api; ant -Dsource.dir=src -Ddoc.dir=/mnt/data/b/build/slave/Android_Cronet_ARM64_Builder__dbg_/build/src/out/Debug/cronet/javadoc -Dlib.java.dir=/mnt/data/b/build/slave/Android_Cronet_ARM64_Builder__dbg_/build/src/out/Debug/lib.java/components/cronet/android -Doverview=/mnt/data/b/build/slave/Android_Cronet_ARM64_Builder__dbg_/build/src/out/Debug/cronet/README.md.html doc )
Buildfile: /mnt/data/b/build/slave/Android_Cronet_ARM64_Builder__dbg_/build/src/components/cronet/android/api/build.xml
 
Labels: -Pri-3 Pri-2
Owner: pauljensen@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8151567e903ccf8af49d97ee633f8126eb02d01f

commit 8151567e903ccf8af49d97ee633f8126eb02d01f
Author: pauljensen <pauljensen@chromium.org>
Date: Fri Apr 08 15:47:31 2016

Revert of Download doclava when building for Android and use to build cronet Javadocs (patchset #10 id:180001 of https://codereview.chromium.org/1557233003/ )

Reason for revert:
Passes once and then subsequent builds fail with "javadoc: error - Cannot find doclet class com.google.doclava.Doclava".
My guess is download_from_google_storage is somehow not extracting the next time, and some script is clearing the extracted directory but not the downloaded archive.

BUG= 601800 

Original issue's description:
> [Cronet] Use doclava to build Cronet Javadocs
>
> During "gclient runhooks", download doclava from Google Storage when building for Android.
> Switching to doclava allows using the @hide tag instead of
> the @deprecated tag which leads to build warnings.
> Doclava does not have a nodeprecated option so deprecated
> types are @hide'd.
>
> BUG= 539519 
>
> Committed: https://crrev.com/62bd31bc7e6d863e87e167e4a6701b64cf584a54
> Cr-Commit-Position: refs/heads/master@{#386077}

TBR=mikecase@chromium.org,agrieve@chromium.org,xunjieli@chromium.org,jbudorick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 539519 

Review URL: https://codereview.chromium.org/1868203002

Cr-Commit-Position: refs/heads/master@{#386088}

[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/DEPS
[delete] https://crrev.com/7d09f9eaa7e7f7dd04a505de54b7cac4f3fd6fd7/build/android/download_doclava.py
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/build.xml
[add] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/package-list
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/src/org/chromium/net/BidirectionalStream.java
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/src/org/chromium/net/CronetEngine.java
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/src/org/chromium/net/NetworkQualityRttListener.java
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/src/org/chromium/net/NetworkQualityThroughputListener.java
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/android/api/src/org/chromium/net/UrlRequest.java
[modify] https://crrev.com/8151567e903ccf8af49d97ee633f8126eb02d01f/components/cronet/tools/generate_javadoc.py

Status: Fixed (was: Assigned)

Sign in to add a comment