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

["Gábor Boskovits" <boskovits@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 7569 bytes --]

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?= <boskovits@gmail.com>
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)

 \f
 (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?= <boskovits@gmail.com>
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 <boskovits@gmail.com>:

> 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 <efraim@flashner.co.il>:
>
>> On Tue, Dec 12, 2017 at 06:06:52PM -0800, Chris Marusich wrote:
>> > Gábor Boskovits <boskovits@gmail.com> writes:
>> >
>> > > I will wait for your results for now...
>> >
>> ....
>> >
>> > * kodi@18.0_alpha-7-67fd70f: 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/doc
>> book/xml/4.2/docbookx.dtd"
>> >
>>         ^
>> > runtime error: file /gnu/store/q0irgfv31s8m9s54zrc
>> 908qb4cim08cg-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.d
>> rv-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   <efraim@flashner.co.il>   אפרים פלשנר
>> GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
>> Confidentiality cannot be guaranteed on emails sent or received
>> unencrypted
>>
>
>

[-- Attachment #2: Type: text/html, Size: 10386 bytes --]