From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Wurmus Subject: Re: bug#36685: ant-bootstrap fails on core-updates (409 dependents) Date: Fri, 19 Jul 2019 14:56:04 +0200 Message-ID: <878ssufajf.fsf@elephly.net> References: <8736j61n57.fsf@gmail.com> <87o91ugdot.fsf@elephly.net> <87ftn5gjzw.fsf@elephly.net> <871ryogu6j.fsf@elephly.net> <87r26nfwes.fsf@elephly.net> <87a7dafntp.fsf@elephly.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:56004) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hoSQm-0005vS-Be for guix-devel@gnu.org; Fri, 19 Jul 2019 08:56:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hoSQl-0003e7-Ch for guix-devel@gnu.org; Fri, 19 Jul 2019 08:56:12 -0400 Received: from sender-of-o51.zoho.com ([135.84.80.216]:21262) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hoSQk-0003d8-W2 for guix-devel@gnu.org; Fri, 19 Jul 2019 08:56:11 -0400 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: =?utf-8?Q?G=C3=A1bor?= Boskovits Cc: Guix-devel , 36685@debbugs.gnu.org Hi G=C3=A1bor, > Ricardo Wurmus ezt =C3=ADrta (id=C5=91pont: 2019. j= =C3=BAl. 19., P, > 8:09): > >> >> Ricardo Wurmus writes: >> > Here=E2=80=99s a shorter patch: >> > >> > --8<---------------cut here---------------start------------->8--- >> > diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm >> > index 403c446a82..bd98784232 100644 >> > --- a/gnu/packages/java.scm >> > +++ b/gnu/packages/java.scm >> > @@ -152,6 +152,13 @@ and binary format defined in The Java Virtual >> Machine Specification.") >> > "--disable-gjdoc") >> > #:phases >> > (modify-phases %standard-phases >> > + (add-after 'unpack 'foo >> > + (lambda _ >> > + (substitute* "native/jni/java-io/java_io_VMFile.c" >> > + (("result =3D cpio_isFileExists.*" m) >> > + (string-append m " >> > +//Without a long comment the Java side will return \"true\" on x86_64 >> all the time."))) >> > + #t)) >> > (add-after 'install 'install-data >> > (lambda _ (invoke "make" "install-data")))))) >> > (native-inputs >> > --8<---------------cut here---------------end--------------->8--- >> > >> > This only adds a comment. If the comment is too short it won=E2=80=99= t work. >> >> > I confirm this path works. > I tested a modified version, where I took out the comment text. It also > works that way. Yes. It=E2=80=99s just commenting the *following* line (i.e. removal of the call to =E2=80=9Cfree=E2=80=9D) that fixes it. > We might contact the classpath devs to get a proper fix, and maybe a new > release, they > were super responsive the last time. I don=E2=80=99t think the problem is with GNU Classpath. Classpath devs are welcome to look at this, of course, but we are not even using the latest version at this point in the bootstrap. I wonder what could have caused this problem in the first place. The bootstrap hasn=E2=80=99t changed. Using the same compiler as before doesn= =E2=80=99t fix it, so it=E2=80=99s probably not the compiler either. So what is it? >> Illegal instruction? This uses JamVM 2.0.0 as the JVM. I=E2=80=99ll tr= y to >> figure out what instruction this is and where it comes from. > > I hit the same bug now. Good. We need to figure out why =E2=80=93 is it miscompilation? Tuning to= a different CPU type=E2=80=A6? -- Ricardo