Re: java: switch to icedtea-8 as default JDK

2017-12-13 23:50 GMT+01:00 Gábor Boskovits <address@hidden>:

I've built the whole covering on my wip-change-default-java8 branch.
The covering is 42 packages.
19 builds fine.
19 does not build because java-hamcrest-core does not build.
I'll have a look at this first, this is the most critical.
This is the issue link for reference:
https://github.com/Boskovits/guix/issues/16
2017-12-13 19:04 GMT+01:00 Gábor Boskovits <address@hidden>:
The patches I made:
This is the trivial:
From f53ad84059786e0769a21a3a90a15189bcf2d61f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?G=C3=A1bor=20Boskovits?= <address@hidden>
Date: Mon, 11 Dec 2017 21:18:37 +0100
Subject: [PATCH] Change java toolchain defaults to use java8.

* gnu/packages/java.scm(icedtea): Point to icedtea-8.
* gnu/packages/java.scm(ant): Point to ant/java8.
---
 gnu/packages/java.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm
index 6dfe554f3..547cc2dc1 100644
--- a/gnu/packages/java.scm
+++ b/gnu/packages/java.scm
@@ -1691,7 +1691,7 @@ IcedTea build harness.")
                  '("jdk" "openjdk-src" "corba-drop" "jaxp-drop" "jaxws-drop"
                    "jdk-drop" "langtools-drop" "hotspot-drop")))))))
 
-(define-public icedtea icedtea-7)
+(define-public icedtea icedtea-8)
 
 
 (define-public ant/java8
@@ -1754,7 +1754,7 @@ IcedTea build harness.")
     (native-inputs
      `(("jdk" ,icedtea-7 "jdk")))))
 
-(define-public ant ant/java7)
+(define-public ant ant/java8)
 
 (define-public ant-apache-bcel
   (package
This is what I'd like to have your opinions on, is it ok to do it this way, or should I rethink it
(I like this, because this way we can leave ant-build-system alone, and handling of
default version in java.scm became consistent):
From 1629a53ac5122d229b157a33806264772f18f6bc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?G=C3=A1bor=20Boskovits?= <address@hidden>
Date: Mon, 11 Dec 2017 21:13:58 +0100
Subject: [PATCH] gnu:add ant/java7.

* gnu/packages/java.scm(ant/java7): New variable.
* gnu/packages/java.scm(ant): Changed variable semantics to point to default ant.
---
 gnu/packages/java.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm
index 0be488dec..6dfe554f3 100644
--- a/gnu/packages/java.scm
+++ b/gnu/packages/java.scm
@@ -1741,7 +1741,7 @@ IcedTea build harness.")
 
 ;; The 1.9.x series is the last that can be built with GCJ.  The 1.10.x series
 ;; requires Java 8.
-(define-public ant
+(define-public ant/java7
   (package (inherit ant/java8)
     (version "1.9.9")
     (source (origin
@@ -1754,6 +1754,8 @@ IcedTea build harness.")
     (native-inputs
      `(("jdk" ,icedtea-7 "jdk")))))
 
+(define-public ant ant/java7)
+
 (define-public ant-apache-bcel
   (package
     (inherit ant/java8)
2017-12-13 12:07 GMT+01:00 Gábor Boskovits <address@hidden>:
Hello!

It was getting quite intractable to see what I was doing, so I started to fill my github issue tracker. Issues are labeled with change-default-java8.

2017-12-13 11:32 GMT+01:00 Efraim Flashner <address@hidden>:
On Tue, Dec 12, 2017 at 06:06:52PM -0800, Chris Marusich wrote:
> Gábor Boskovits <address@hidden> writes:
>
> > I will wait for your results for now...
>
....

>
> * address@hidden: failed because of "potential infinite
> recursion"; also it seems that it's looking for a DTD on the Internet
> when instead it should be looking to find it in the inputs.
>
> --8<---------------cut here---------------start------------->8---
> [3684/3799] Linking default/source3/modules/libvfs_module_worm.so
> [3685/3799] Linking default/source3/modules/libvfs_module_crossrename.so
> [3686/3799] Linking default/source3/modules/libvfs_module_cap.so
> [3687/3799] Linking default/source3/modules/libvfs_module_catia.so
> [3688/3799] pidl.1: pidl/pidl -> bin/default/pidl/pidl.1
> [3689/3799] Parse::Pidl::Dump.3: pidl/lib/Parse/Pidl/Dump.pm -> bin/default/pidl/Parse::Pidl::Dump.3
> [3690/3799] Parse::Pidl::Wireshark::Conformance.3: pidl/lib/Parse/Pidl/Wireshark/Conformance.pm -> bin/default/pidl/Parse::Pidl::Wireshark::Conformance.3
> [3691/3799] Parse::Pidl::Util.3: pidl/lib/Parse/Pidl/Util.pm -> bin/default/pidl/Parse::Pidl::Util.3
> [3692/3799] Parse::Pidl::NDR.3: pidl/lib/Parse/Pidl/NDR.pm -> bin/default/pidl/Parse::Pidl::NDR.3
> [3693/3799] Parse::Pidl::Wireshark::NDR.3: pidl/lib/Parse/Pidl/Wireshark/NDR.pm -> bin/default/pidl/Parse::Pidl::Wireshark::NDR.3
> [3694/3799] Generating manpages/smb.conf.5
> I/O error : Attempt to load network entity http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd
> default/docs-xml/manpages/smb.conf.5.xml:2: warning: failed to load external entity "http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd"
> D DocBook XML V4.2//EN" "http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd"
> ^
> runtime error: file /gnu/store/q0irgfv31s8m9s54zrc908qb4cim08cg-docbook-xsl-1.79.1/xml/xsl/docbook-xsl-1.79.1/lib/lib.xsl line 58 element choose
> xsltApplySequenceConstructor: A potential infinite template recursion was detected.
> You can adjust xsltMaxDepth (--maxdepth) in order to raise the maximum number of nested template calls and variables/params (currently set to 3000).
> Templates:
> #0 name string.subst
> #1 name string.subst
> #2 name string.subst
> #3 name string.subst
> #4 name string.subst
> #5 name string.subst
> #6 name string.subst
> #7 name string.subst
> #8 name string.subst
> #9 name string.subst
> #10 name string.subst
> #11 name string.subst
> #12 name string.subst
> #13 name string.subst
> #14 name string.subst
> Variables:
> #0
> replacement
> target
> string
> #1
> target
> string
> #2
> string
> #3
> replacement
> target
> string
> #4
> target
> string
> #5
> string
> #6
> replacement
> target
> string
> #7
> target
> string
> #8
> string
> #9
> replacement
> target
> string
> #10
> target
> string
> #11
> string
> #12
> replacement
> target
> string
> #13
> target
> string
> #14
> string
> error: file default/docs-xml/manpages/smb.conf.5.xml
> xsltRunStylesheet : run failed
> Waf: Leaving directory `/tmp/guix-build-samba-4.7.3.drv-0/samba-4.7.3/bin'
> Build failed: -> task failed (err #11):
> {task: manpages/smb.conf.5 smb.conf.5.xml,parameters.all.xml -> smb.conf.5}
> make: *** [Makefile:8: all] Error 1
> phase `build' failed after 1063.6 seconds
> note: keeping build directory `/tmp/guix-build-samba-4.7.3.drv-0'
> builder for `/gnu/store/k3ykppkqdw6gyj115qjhbi50fa6hx317-samba-4.7.3.drv' failed with exit code 1
> @ build-failed /gnu/store/k3ykppkqdw6gyj115qjhbi50fa6hx317-samba-4.7.3.drv - 1 builder for `/gnu/store/k3ykppkqdw6gyj115qjhbi50fa6hx317-samba-4.7.3.drv' failed with exit code 1
> --8<---------------cut here---------------end--------------->8---
>
> I have not begun troubleshooting any of these existing issues yet.
>
> --
> Chris

I tested building samba on aarch64 on core-updates and it failed in the
same manner. For the purposes of updating the default JRE I would ignore
it and the resulting not being able to build kodi.

--
Efraim Flashner <address@hidden> אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

From:	Gábor Boskovits
Subject:	Re: java: switch to icedtea-8 as default JDK
Date:	Fri, 15 Dec 2017 14:52:02 +0100