From: "Gábor Boskovits" <boskovits@gmail.com>
To: Ricardo Wurmus <rekado@elephly.net>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: java: switch to icedtea-8 as default JDK
Date: Fri, 15 Dec 2017 15:21:41 +0100 [thread overview]
Message-ID: <CAE4v=pimvt2Q65fpQWvGEAP36yjnsCJFZetEb4j1HU5=1Axifw@mail.gmail.com> (raw)
In-Reply-To: <CAE4v=pjFSDQ40HWbmvmtfEiX+rV69+GzbKwoaXqucx5cMOdywg@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 9749 bytes --]
Ok, it seems, that java-asm does not currently uses test anyway, so I can
remove the junit native input.
Will check if it is also true on master...
2017-12-15 14:52 GMT+01:00 Gábor Boskovits <boskovits@gmail.com>:
> I've found the problem with java-hamcrest-core.
> The root cause of this is that java-jarjar uses a bundled copy of asm,
> version 4.0.
> This asm version does not support icedtea8.
>
> I propose to use java-asm instead of the bundled one.
>
> In my opinion the best course of action should be:
> 1. create a package: java-asm-notest with tests diabled to break the
> dependency cycle on java-hamcrest-core
> 2. modify java-jarjar to use java-asm-notest instead of the bundled asm.
> 3. test if java-hamcrest-core now builds.
>
> It is a question if we want to iterate this cycle once more, with a
> tests-enabled java-asm.
> WDYT?
>
>
> 2017-12-13 23:50 GMT+01:00 Gábor Boskovits <boskovits@gmail.com>:
>
>> 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 <boskovits@gmail.com>:
>>
>>> 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/doc
>>>>> book/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/k3ykppkqdw6gyj115q
>>>>> jhbi50fa6hx317-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/k3ykppkqdw6gyj115q
>>>>> jhbi50fa6hx317-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: 13454 bytes --]
next prev parent reply other threads:[~2017-12-15 14:21 UTC|newest]
Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-17 22:02 java: switch to icedtea-8 as default JDK Ricardo Wurmus
2017-11-22 8:57 ` Chris Marusich
2017-11-22 12:09 ` Ricardo Wurmus
2017-11-22 17:03 ` Chris Marusich
2017-11-24 7:01 ` Gábor Boskovits
2017-11-24 10:15 ` Ricardo Wurmus
2017-11-24 11:46 ` Gábor Boskovits
2017-11-29 6:48 ` Chris Marusich
2017-11-29 7:12 ` Chris Marusich
2017-11-29 19:05 ` Gábor Boskovits
2017-11-29 19:58 ` Ricardo Wurmus
2017-11-30 6:58 ` Chris Marusich
2017-11-30 19:27 ` Efraim Flashner
2017-12-01 20:41 ` Gábor Boskovits
2017-12-02 7:06 ` Chris Marusich
2017-12-03 9:58 ` Gábor Boskovits
2017-12-03 11:30 ` Gábor Boskovits
2017-12-03 21:46 ` Chris Marusich
2017-12-03 22:04 ` Gábor Boskovits
2017-12-03 22:08 ` Gábor Boskovits
2017-12-03 22:20 ` Gábor Boskovits
2017-12-03 22:25 ` Gábor Boskovits
2017-12-04 12:03 ` Gábor Boskovits
2017-12-04 12:18 ` Gábor Boskovits
2017-12-04 12:21 ` Gábor Boskovits
2017-12-04 12:33 ` Gábor Boskovits
2017-12-04 12:59 ` Gábor Boskovits
2017-12-04 14:19 ` Gábor Boskovits
2017-12-04 15:44 ` Gábor Boskovits
2017-12-04 16:34 ` Gábor Boskovits
2017-12-04 16:35 ` Gábor Boskovits
2017-12-04 19:15 ` Leo Famulari
2017-12-05 7:07 ` Gábor Boskovits
2017-12-07 17:50 ` Gábor Boskovits
2017-12-08 6:13 ` Gábor Boskovits
2017-12-08 6:55 ` Chris Marusich
2017-12-08 7:01 ` Gábor Boskovits
2017-12-10 15:56 ` Gábor Boskovits
2017-12-12 11:30 ` Gábor Boskovits
2017-12-12 12:27 ` Gábor Boskovits
2017-12-13 2:06 ` Chris Marusich
2017-12-13 8:53 ` Gábor Boskovits
2017-12-13 9:54 ` Gábor Boskovits
2017-12-13 9:59 ` Gábor Boskovits
2017-12-13 10:03 ` Gábor Boskovits
2017-12-13 10:32 ` Efraim Flashner
2017-12-13 11:07 ` Gábor Boskovits
2017-12-13 18:04 ` Gábor Boskovits
2017-12-13 22:50 ` Gábor Boskovits
2017-12-15 13:52 ` Gábor Boskovits
2017-12-15 14:21 ` Gábor Boskovits [this message]
2017-12-15 14:33 ` Gábor Boskovits
2017-12-17 7:39 ` Gábor Boskovits
2017-12-17 14:26 ` Ricardo Wurmus
2017-12-17 18:59 ` Gábor Boskovits
2017-12-18 10:02 ` Gábor Boskovits
2017-12-18 12:33 ` Ricardo Wurmus
2017-12-19 8:07 ` Chris Marusich
2017-12-19 9:11 ` Gábor Boskovits
2017-12-19 20:58 ` Gábor Boskovits
2017-12-19 22:11 ` Ricardo Wurmus
2017-12-20 10:34 ` Gábor Boskovits
2017-12-20 12:29 ` Gábor Boskovits
2017-12-21 7:53 ` Gábor Boskovits
2018-01-03 13:06 ` Gábor Boskovits
2018-01-05 15:44 ` Ricardo Wurmus
2018-01-05 16:01 ` Gábor Boskovits
2018-01-06 19:29 ` Chris Marusich
2018-01-06 20:16 ` Gábor Boskovits
2018-01-07 14:52 ` Gábor Boskovits
2018-01-07 17:25 ` Ricardo Wurmus
2018-01-07 18:19 ` Gábor Boskovits
2018-01-08 2:54 ` Chris Marusich
[not found] ` <87d13jhoka.fsf@gmail.com>
2017-12-12 22:49 ` Gábor Boskovits
2017-12-12 22:50 ` Gábor Boskovits
2017-11-29 19:59 ` Ricardo Wurmus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAE4v=pimvt2Q65fpQWvGEAP36yjnsCJFZetEb4j1HU5=1Axifw@mail.gmail.com' \
--to=boskovits@gmail.com \
--cc=guix-devel@gnu.org \
--cc=rekado@elephly.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.