Ubuntu
groovy package

[FTBFS] groovy 2.4.15-1 ftbfs due to invalid javadoc links

Bug #1765885 reported by Tiago Stürmer Daitx
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
groovy (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Starting with openjdk-10 all invalid, unreachable, or nonexistent javadoc links cause the javadoc to throw out an error - until openjdk-9 they were simply ignored with at most a warning.

groovy currently FTBFS due to:

Starting process 'command '/usr/lib/jvm/java-11-openjdk-amd64/bin/javadoc''. Working directory: /<<PKGBUILDDIR>> Command: /usr/lib/jvm/java-11-openjdk-amd64/bin/javadoc @/<<PKGBUILDDIR>>/target/tmp/javadocAll/javadoc.options -J-Xmx1g
Successfully started process 'command '/usr/lib/jvm/java-11-openjdk-amd64/bin/javadoc''
javadoc: error - Error fetching URL: http://docs.oracle.com/javase/8/docs/api/
javadoc: error - Error fetching URL: http://docs.oracle.com/javaee/7/api/
javadoc: error - Error fetching URL: http://commons.apache.org/proper/commons-cli/javadocs/api-release/
javadoc: error - Error fetching URL: http://junit.org/junit4/javadoc/latest/
javadoc: error - Error fetching URL: http://docs.oracle.com/javaee/6/api/
javadoc: error - Error fetching URL: http://www.antlr2.org/javadoc/

The fix is to replace all links with locally available apis and have the package build depend upon the packages that provide these files.

Tags: ftbfs patch
Revision history for this message
Tiago Stürmer Daitx (tdaitx) wrote :

This patch requires bug #1765883 to be uploaded.

The alternative to that is to drop the default-jdk-headless build-dep and use default-jdk in its place.

tags: added: patch
Revision history for this message
Steve Langasek (vorlon) wrote :

+- links('http://docs.oracle.com/javase/8/docs/api/', 'http://docs.oracle.com/javaee/7/api/',
+- 'http://commons.apache.org/proper/commons-cli/javadocs/api-release/', 'http://junit.org/junit4/javadoc/latest/',
+- 'http://docs.oracle.com/javaee/6/api/', 'http://www.antlr2.org/javadoc/')
++ //links('http://docs.oracle.com/javase/8/docs/api/', 'http://docs.oracle.com/javaee/7/api/',
++ // 'http://commons.apache.org/proper/commons-cli/javadocs/api-release/', 'http://junit.org/junit4/javadoc/latest/',
++ // 'http://docs.oracle.com/javaee/6/api/', 'http://www.antlr2.org/javadoc/')
++ links('file:///usr/share/doc/default-jdk-doc/api/', 'file:///usr/share/doc/libservlet3.1-java-doc/api',
++ 'file:///usr/share/doc/junit4/api/')

FWIW I strongly encourage dropping such lines instead of commenting them out. It makes diffs shorter and easier to review, and the information is not lost because it's still available in the diff itself.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package groovy - 2.4.15-1ubuntu1

---------------
groovy (2.4.15-1ubuntu1) bionic; urgency=medium

  * Replace HTTP URLs with local files: LP: #1765885.
    - debian/control: build depends on -doc packages so javadoc can
      properly link the apis.
    - debian/patches/10_use_local_docs.patch, debian/control: include
      javadoc apis locally as invalid, unreacheable, or nonexistent
      links cause the build to faild with openjdk-10.

 -- Tiago Stürmer Daitx <email address hidden> Fri, 20 Apr 2018 23:33:44 +0000

Changed in groovy (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.