From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?G=C3=A1bor_Boskovits?= Subject: Re: java: switch to icedtea-8 as default JDK Date: Mon, 18 Dec 2017 11:02:26 +0100 Message-ID: References: <20171204191506.GC28066@jasmine.lan> <87609h3dzk.fsf@gmail.com> <87o9n3gz43.fsf@gmail.com> <20171213103248.GA12880@macbook41> <877etlo2fq.fsf@elephly.net> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="94eb2c058af691454f05609a7263" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:33678) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eQsFj-0003SN-Qw for guix-devel@gnu.org; Mon, 18 Dec 2017 05:02:35 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eQsFf-0008KU-Qx for guix-devel@gnu.org; Mon, 18 Dec 2017 05:02:31 -0500 Received: from mail-it0-x22c.google.com ([2607:f8b0:4001:c0b::22c]:40005) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eQsFf-0008Jl-Kh for guix-devel@gnu.org; Mon, 18 Dec 2017 05:02:27 -0500 Received: by mail-it0-x22c.google.com with SMTP id f190so26922121ita.5 for ; Mon, 18 Dec 2017 02:02:27 -0800 (PST) In-Reply-To: List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Ricardo Wurmus Cc: guix-devel --94eb2c058af691454f05609a7263 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 2017-12-17 19:59 GMT+01:00 G=C3=A1bor Boskovits : > Yes, we could do that, however, I would prefer to fix these if we can. > > > 2017-12-17 15:26 GMT+01:00 Ricardo Wurmus : > >> >> G=C3=A1bor Boskovits writes: >> >> > Currently I cannot compile java-aqute-bndlib, >> > because java-classpathx-servletapi-3.0 does not work with java8. >> > This is a known incompatibility between java7 and java8 definition of >> Map, >> > affecting multiple projects. >> > I will focus on fixing that for now, and file a bug upstream. >> >> If a package does not build with Java 8 this does not mean that we >> cannot use the latest JDK to build it. For those packages we might have >> to patch the build.xml to ensure that the JDK considers the source to be >> of a different version. >> >> javac supports the =E2=80=9C-target=E2=80=9D and =E2=80=9C-source=E2=80= =9D options. This might be >> helpful in situations like this. >> >> -- >> Ricardo >> >> GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC >> https://elephly.net >> >> >> > We now have an upstream fix for classpathx, it is pushed as revision 1244 to the savannah subversion repo. May I change the source to use this revision? Or should I use a patch? I think using the upstream revision is a better option. WDYT? --94eb2c058af691454f05609a7263 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable


2017-12-17 19:59 GMT+01:00 G=C3=A1bor Boskovits <<= a href=3D"mailto:boskovits@gmail.com" target=3D"_blank">boskovits@gmail.com= >:
Yes, we= could do that, however, I would prefer to fix these if we can.


2017-12-17 15:26 GMT+01:00 Ricardo Wurmus <= span dir=3D"ltr"><rekado@elephly.net>:

G=C3=A1bor Boskovits <boskovits@gmail.com> writes:

> Currently I cannot compile java-aqute-bndlib,
> because java-classpathx-servletapi-3.0 does not work with java8.
> This is a known incompatibility between java7 and java8 definition of = Map,
> affecting multiple projects.
> I will focus on fixing that for now, and file a bug upstream.

If a package does not build with Java 8 this does not mean that we cannot use the latest JDK to build it.=C2=A0 For those packages we might ha= ve
to patch the build.xml to ensure that the JDK considers the source to be of a different version.

javac supports the =E2=80=9C-target=E2=80=9D and =E2=80=9C-source=E2=80=9D = options.=C2=A0 This might be
helpful in situations like this.

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6=C2=A0 2150 197A 5888 235F ACAC
https:= //elephly.net



We now have an upstream fix for classpathx, = it is pushed as revision 1244 to the savannah subversion repo.
May I change the source to use this revision?
Or should I use a patch?

I think using the upstream revisi= on is a better option.
WDYT?


--94eb2c058af691454f05609a7263--