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: Sun, 3 Dec 2017 23:25:47 +0100 Message-ID: References: <87375czhqj.fsf@elephly.net> <873756herx.fsf@gmail.com> <87y3myv7k9.fsf@elephly.net> <87d141mvgn.fsf@gmail.com> <878tepmud6.fsf@gmail.com> <87bmjkfe1j.fsf@gmail.com> <87lgiltxr4.fsf@gmail.com> <87efobqybc.fsf@gmail.com> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="001a1144af5e5a099c055f7715a6" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:58964) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eLchq-0006TW-Q3 for guix-devel@gnu.org; Sun, 03 Dec 2017 17:25:53 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eLcho-0005aK-JK for guix-devel@gnu.org; Sun, 03 Dec 2017 17:25:50 -0500 Received: from mail-it0-x230.google.com ([2607:f8b0:4001:c0b::230]:37956) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eLcho-0005a8-CA for guix-devel@gnu.org; Sun, 03 Dec 2017 17:25:48 -0500 Received: by mail-it0-x230.google.com with SMTP id r6so2294315itr.3 for ; Sun, 03 Dec 2017 14:25:48 -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: Chris Marusich Cc: guix-devel --001a1144af5e5a099c055f7715a6 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable It seems, that the fix is already included in the 60.1 release for the xlocal problem. I think we can ignore that. 2017-12-03 23:20 GMT+01:00 G=C3=A1bor Boskovits : > It seems, that they issued that already: > http://bugs.icu-project.org/trac/changeset/40603 > > 2017-12-03 23:08 GMT+01:00 G=C3=A1bor Boskovits : > >> Ok, I found this one: >> https://sourceware.org/glibc/wiki/Release/2.26#Removal_of_.27xlocale.h.2= 7 >> >> It seems, that xlocale.h should not be used as an include, and glibc >> removed that. >> >> So this is ok. >> >> The question is if this issue was addressed by icu4c, so we can just go >> to the next step, or should we do something about this. >> >> >> 2017-12-03 23:04 GMT+01:00 G=C3=A1bor Boskovits : >> >>> I've found that simply reverting the update to icu4c 60 commit brings u= p >>> the issue you just mentioned, with missing xlocale.h. >>> So it seems, that the one you found introduced the xlocale problem, >>> while 4e080fbb0bb73e4444181599676f4e1ef5fdc2ba introduces the new error= . >>> I think we should deal with the one you find first, the see what's with >>> this newer one. >>> >>> We also have a long running discussion on a bug introduced by a later >>> glibc update, ee3ebf1a357bd4eb36a2fa1790a7b549cffb305a which broke a >>> lot of packages. Might these be somehow related? >>> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=3D29537 >>> >>> >>> >>> 2017-12-03 22:46 GMT+01:00 Chris Marusich : >>> >>>> G=C3=A1bor Boskovits writes: >>>> >>>> > Any news on this icu4c thing? >>>> >>>> After about 2 days of running git bisect, my computer has informed me >>>> that the first bad commit is 67d527e35e367c9e9e89ec01cda2ce32cabd2d89. >>>> This is the first commit where icu4c fails to build on core-updates. >>>> The build failure at this commit is as follows: >>>> >>>> --8<---------------cut here---------------start------------->8--- >>>> g++ ... decimfmt.cpp >>>> g++ ... decimalformatpattern.cpp >>>> g++ ... dcfmtsym.cpp >>>> g++ ... digitlst.cpp >>>> g++ ... fmtable_cnv.cpp >>>> digitlst.cpp:67:24: fatal error: xlocale.h: No such file or directory >>>> compilation terminated. >>>> *** Failed compilation command follows: ------------------------------ >>>> ---------------------------- >>>> g++ -D_REENTRANT -DU_HAVE_ELF_H=3D1 -DU_HAVE_ATOMIC=3D1 -DU_HAVE_STRTO= D_L=3D1 >>>> -I. -I../common -DU_ATTRIBUTE_DEPRECATED=3D -DU_I18N_IMPLEMENTATION -O= 2 -W >>>> -Wall -pedantic -Wpointer-arith -Wwrite-strings -Wno-long-long --std= =3Dc++0x >>>> -c -DPIC -fPIC -o digitlst.o digitlst.cpp >>>> --- ( rebuild with "make VERBOSE=3D1 all" to show all parameters ) >>>> -------- >>>> make[1]: *** [../config/mh-linux:51: digitlst.o] Error 1 >>>> make[1]: *** Waiting for unfinished jobs.... >>>> make[1]: Leaving directory '/tmp/guix-build-icu4c-58.2.dr >>>> v-0/icu/source/i18n' >>>> make: *** [Makefile:143: all-recursive] Error 2 >>>> phase `build' failed after 59.0 seconds >>>> builder for `/gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c-58.2.dr= v' >>>> failed with exit code 1 >>>> @ build-failed /gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c-58.2.= drv >>>> - 1 builder for `/gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c-58.= 2.drv' >>>> failed with exit code 1 >>>> guix build: error: build failed: build of `/gnu/store/mdd6glhc0dg65y4w= d11y0b7sbky9cwv6-icu4c-58.2.drv' >>>> failed >>>> --8<---------------cut here---------------end--------------->8--- >>>> >>>> Note that this is NOT the same error as the one that occurs at commit >>>> d6adba786cf2ab1b26ff083928b64262281ff106 (which is the commit on >>>> core-updates from which G=C3=A1bor's branch change-default-icedtea-8 b= ranch >>>> begins). That error is: >>>> >>>> --8<---------------cut here---------------start------------->8--- >>>> g++ ... number_decimalquantity.cpp >>>> g++ ... number_decimfmtprops.cpp >>>> number_decimalquantity.cpp: In member function >>>> =E2=80=98icu_60::number::impl::DecimalQuantity& icu_60::number::impl::= DecimalQ >>>> uantity::setToDouble(double)=E2=80=99: >>>> number_decimalquantity.cpp:333:9: error: =E2=80=98_Generic=E2=80=99 is= not a member of >>>> =E2=80=98std=E2=80=99 >>>> if (std::signbit(n) !=3D 0) { >>>> ^ >>>> In file included from /gnu/store/nz2m4gdvgzcrkqa4xwv >>>> 360iskh7syj7i-gcc-5.5.0/include/c++/cmath:44:0, >>>> from number_decimalquantity.cpp:9: >>>> number_decimalquantity.cpp:333:14: error: expected primary-expression >>>> before =E2=80=98float=E2=80=99 >>>> if (std::signbit(n) !=3D 0) { >>>> ^ >>>> number_decimalquantity.cpp:333:14: error: expected primary-expression >>>> before =E2=80=98default=E2=80=99 >>>> if (std::signbit(n) !=3D 0) { >>>> ^ >>>> number_decimalquantity.cpp:333:14: error: expected primary-expression >>>> before =E2=80=98long=E2=80=99 >>>> if (std::signbit(n) !=3D 0) { >>>> ^ >>>> number_decimalquantity.cpp:333:14: error: expected primary-expression >>>> before =E2=80=98:=E2=80=99 token >>>> if (std::signbit(n) !=3D 0) { >>>> ^ >>>> number_decimalquantity.cpp:337:9: error: =E2=80=98__builtin_isnan=E2= =80=99 is not a >>>> member of =E2=80=98std=E2=80=99 >>>> if (std::isnan(n) !=3D 0) { >>>> ^ >>>> number_decimalquantity.cpp:337:9: note: suggested alternative: >>>> : note: =E2=80=98__builtin_isnan=E2=80=99 >>>> number_decimalquantity.cpp:339:16: error: =E2=80=98__builtin_isfinite= =E2=80=99 is not >>>> a member of =E2=80=98std=E2=80=99 >>>> } else if (std::isfinite(n) =3D=3D 0) { >>>> ^ >>>> number_decimalquantity.cpp:339:16: note: suggested alternative: >>>> : note: =E2=80=98__builtin_isfinite=E2=80=99 >>>> *** Failed compilation command follows: ------------------------------ >>>> ---------------------------- >>>> g++ -D_REENTRANT -DU_HAVE_ELF_H=3D1 -DU_HAVE_ATOMIC=3D1 -DU_HAVE_STRTO= D_L=3D1 >>>> -DU_HAVE_XLOCALE_H=3D0 -I. -I../common -DU_ATTRIBUTE_DEPRECATED=3D >>>> -DU_I18N_IMPLEMENTATION -O2 -W -Wall -pedantic -Wpointer-arith >>>> -Wwrite-strings -Wno-long-long -std=3Dc++11 -c -DPIC -fPIC -o >>>> number_decimalquantity.o number_decimalquantity.cpp >>>> --- ( rebuild with "make VERBOSE=3D1 all" to show all parameters ) >>>> -------- >>>> make[1]: *** [../config/mh-linux:51: number_decimalquantity.o] Error 1 >>>> make[1]: *** Waiting for unfinished jobs.... >>>> make[1]: Leaving directory '/tmp/guix-build-icu4c-60.1.dr >>>> v-0/icu/source/i18n' >>>> make: *** [Makefile:149: all-recursive] Error 2 >>>> phase `build' failed after 122.2 seconds >>>> builder for `/gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c-60.1.dr= v' >>>> failed with exit code 1 >>>> @ build-failed /gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c-60.1.= drv >>>> - 1 builder for `/gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c-60.= 1.drv' >>>> failed with exit code 1 >>>> guix build: error: build failed: build of `/gnu/store/8s6q5cll4knh7y0w= frbjqs2dai0x4sm2-icu4c-60.1.drv' >>>> failed >>>> --8<---------------cut here---------------end--------------->8--- >>>> >>>> The fact that these errors are different suggests that more than one >>>> problem may have been introduced... >>>> >>>> I haven't tried any more recent commits on core-updates. I am not sur= e >>>> what the best way is to debug this, so I would appreciate any advice >>>> anyone has. >>>> >>>> I hope that information helps. Hopefully, together, we can untangle >>>> this! >>>> >>>> -- >>>> Chris >>>> >>> >>> >> > --001a1144af5e5a099c055f7715a6 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
It seems, that the fix is already included in the 60.1 rel= ease for the xlocal problem. I think we can ignore that.

2017-12-03 23:20 GMT+01:00 G= =C3=A1bor Boskovits <boskovits@gmail.com>:
It seems, that they issued that al= ready:

2017-12-03 23:08 GMT+01:00 G=C3=A1bor= Boskovits <boskovits@gmail.com>:
Ok, I found this one:

It seems, that xlocale.h s= hould not be used as an include, and glibc removed that.

So this is ok.

The question is if this issu= e was addressed by icu4c, so we can just go to the next step, or should we = do something about this.


2017-12-03 23:04 GMT+01:00 G=C3=A1= bor Boskovits <boskovits@gmail.com>:
I've found that simply reverting the upd= ate to icu4c 60 commit brings up the issue you just mentioned, with missing= xlocale.h.
So it seems, that the one you found introduced the xlocale = problem, while=C2=A04e080fbb0bb73e4444181599676f4e1ef5fdc2ba introduce= s the new error.
I think we should deal with the one you find fir= st, the see what's with this newer one.

We als= o have a long running discussion on a bug introduced by a later glibc updat= e,=C2=A0ee3ebf1a357bd4eb36a2fa1790a7b549cffb305a which broke a lot of = packages. Might these be somehow related?



2017-12-03 22:46 GMT+0= 1:00 Chris Marusich <cmmarusich@gmail.com>:
G=C3=A1bor Boskovits <boskovits@gmail.com> writes:

> Any news on this icu4c thing?

After about 2 days of running git bisect, my computer has informed m= e
that the first bad commit is 67d527e35e367c9e9e89ec01cda2ce32cabd2d89.=
This is the first commit where icu4c fails to build on core-updates.
The build failure at this commit is as follows:

--8<---------------cut here---------------start------------->8--= -
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 decimfmt.cpp
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 decimalformatpattern.cpp
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 dcfmtsym.cpp
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 digitlst.cpp
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 fmtable_cnv.cpp
digitlst.cpp:67:24: fatal error: xlocale.h: No such file or directory
compilation terminated.
*** Failed compilation command follows: ------------------------------= ----------------------------
g++ -D_REENTRANT -DU_HAVE_ELF_H=3D1 -DU_HAVE_ATOMIC=3D1 -DU_HAVE_STRTOD_L= =3D1 -I. -I../common -DU_ATTRIBUTE_DEPRECATED=3D -DU_I18N_IMPLEMENTATION -O= 2 -W -Wall -pedantic -Wpointer-arith -Wwrite-strings -Wno-long-long --std= =3Dc++0x -c -DPIC -fPIC -o digitlst.o digitlst.cpp
--- ( rebuild with "make VERBOSE=3D1 all" to show all parameters = ) --------
make[1]: *** [../config/mh-linux:51: digitlst.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory '/tmp/guix-build-icu4c-58.2.drv-0/icu/s= ource/i18n'
make: *** [Makefile:143: all-recursive] Error 2
phase `build' failed after 59.0 seconds
builder for `/gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c-58.2.dr= v' failed with exit code 1
@ build-failed /gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c-58.2.= drv - 1 builder for `/gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c= -58.2.drv' failed with exit code 1
guix build: error: build failed: build of `/gnu/store/mdd6glhc0dg65y4wd11y0b7sbky9cwv6-icu4c-58.2.drv' failed
--8<---------------cut here---------------end--------------->8--= -

Note that this is NOT the same error as the one that occurs at commit
d6adba786cf2ab1b26ff083928b64262281ff106 (which is the commit on
core-updates from which G=C3=A1bor's branch change-default-icedtea-8 br= anch
begins).=C2=A0 That error is:

--8<---------------cut here---------------start------------->8--= -
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 number_decimalquantity.cpp
=C2=A0 =C2=A0g++=C2=A0 =C2=A0...=C2=A0 number_decimfmtprops.cp= p
number_decimalquantity.cpp: In member function =E2=80=98icu_60::number::imp= l::DecimalQuantity& icu_60::number::impl::DecimalQuantity::se= tToDouble(double)=E2=80=99:
number_decimalquantity.cpp:333:9: error: =E2=80=98_Generic=E2=80=99 is= not a member of =E2=80=98std=E2=80=99
=C2=A0 =C2=A0 =C2=A0if (std::signbit(n) !=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0^
In file included from /gnu/store/nz2m4gdvgzcrkqa4xwv360iskh7syj7i-gcc-= 5.5.0/include/c++/cmath:44:0,
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0from number_d= ecimalquantity.cpp:9:
number_decimalquantity.cpp:333:14: error: expected primary-expression = before =E2=80=98float=E2=80=99
=C2=A0 =C2=A0 =C2=A0if (std::signbit(n) !=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^
number_decimalquantity.cpp:333:14: error: expected primary-expression = before =E2=80=98default=E2=80=99
=C2=A0 =C2=A0 =C2=A0if (std::signbit(n) !=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^
number_decimalquantity.cpp:333:14: error: expected primary-expression = before =E2=80=98long=E2=80=99
=C2=A0 =C2=A0 =C2=A0if (std::signbit(n) !=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^
number_decimalquantity.cpp:333:14: error: expected primary-expression = before =E2=80=98:=E2=80=99 token
=C2=A0 =C2=A0 =C2=A0if (std::signbit(n) !=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^
number_decimalquantity.cpp:337:9: error: =E2=80=98__builtin_isnan=E2= =80=99 is not a member of =E2=80=98std=E2=80=99
=C2=A0 =C2=A0 =C2=A0if (std::isnan(n) !=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0^
number_decimalquantity.cpp:337:9: note: suggested alternative:
<built-in>: note:=C2=A0 =C2=A0=E2=80=98__builtin_isnan=E2=80=99
number_decimalquantity.cpp:339:16: error: =E2=80=98__builtin_isfinite= =E2=80=99 is not a member of =E2=80=98std=E2=80=99
=C2=A0 =C2=A0 =C2=A0} else if (std::isfinite(n) =3D=3D 0) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^
number_decimalquantity.cpp:339:16: note: suggested alternative:
<built-in>: note:=C2=A0 =C2=A0=E2=80=98__builtin_isfinite=E2=80=99
*** Failed compilation command follows: -----------------------= -----------------------------------
g++ -D_REENTRANT -DU_HAVE_ELF_H=3D1 -DU_HAVE_ATOMIC=3D1 -DU_HAVE_STRTOD_L= =3D1 -DU_HAVE_XLOCALE_H=3D0 -I. -I../common -DU_ATTRIBUTE_DEPRECATED=3D -DU= _I18N_IMPLEMENTATION -O2 -W -Wall -pedantic -Wpointer-arith -Wwrite-strings= -Wno-long-long -std=3Dc++11 -c -DPIC -fPIC -o number_decimalquantity.o num= ber_decimalquantity.cpp
--- ( rebuild with "make VERBOSE=3D1 all" to show all parameters = ) --------
make[1]: *** [../config/mh-linux:51: number_decimalquantity.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory '/tmp/guix-build-icu4c-60.1.drv-0/icu/s= ource/i18n'
make: *** [Makefile:149: all-recursive] Error 2
phase `build' failed after 122.2 seconds
builder for `/gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c-60.1.dr= v' failed with exit code 1
@ build-failed /gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c-60.1.= drv - 1 builder for `/gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c= -60.1.drv' failed with exit code 1
guix build: error: build failed: build of `/gnu/store/8s6q5cll4knh7y0wfrbjqs2dai0x4sm2-icu4c-60.1.drv' failed
--8<---------------cut here---------------end--------------->8--= -

The fact that these errors are different suggests that more than one
problem may have been introduced...

I haven't tried any more recent commits on core-updates.=C2=A0 I am not= sure
what the best way is to debug this, so I would appreciate any advice
anyone has.

I hope that information helps.=C2=A0 Hopefully, together, we can untangle this!

--
Chris




--001a1144af5e5a099c055f7715a6--