GNU bug report logs - #49990
[core-updates-frozen] Ant-bootstrap broken by classpath-bootstrap

Previous Next

Full log

View this message in rfc822 format

From: Ludovic Courtès <ludo <at> gnu.org>
To: Julien Lepiller <julien <at> lepiller.eu>
Cc: Efraim Flashner <efraim <at> flashner.co.il>, 49990 <at> debbugs.gnu.org
Subject: bug#49990: [core-updates-frozen] Ant-bootstrap broken by classpath-bootstrap
Date: Wed, 01 Sep 2021 15:53:52 +0200

[Message part 1 (text/plain, inline)]

Hi,

Julien Lepiller <julien <at> lepiller.eu> skribis:

> Le Wed, 01 Sep 2021 00:32:39 +0200,
> Ludovic Courtès <ludo <at> gnu.org> a écrit :
>
>> Hello!
>> 
>> As discussed on IRC, I had a semi-victory with the attached patch,
>> which works around a miscompilation issue in
>> ‘Java_java_io_VMFile_isFile’.
>
> another possibility I tested was to disable optimization by adding
> CFLAGS=-O0 to the configure-flags. This creates an unoptimized version,
> but that should be fine since it's only for the bootstrap. However, I
> noticed that my debug messages from my previous attempt were still
> visible when compiling icedtea. I don't know what happened, since
> there is at least another classpath version before it.
>
>> 
>> Unfortunately, with this patch applied, ‘ant-bootstrap’ fails to build
>> with:

The root cause is that, even tough we’re passing “-classpath
build/classes:src”, only the first element of the classpath is
searched.  If we pass “-classpath foobar:build/classes”, then Main.class
is not found.  That suggests another problem with ‘stat’-related
functions.

>> Ideas?  What a wonderful puzzle we have!  :-)

> Even more wonderful is the fact I do not get this error at all when
> using CFLAGS=-O0, but I do with your patch. I didn't have any issue
> either with my previous attempt with debugging. Could there be other
> problems with optimizations in classpath?

Hmm -O0 is brute-force.  It doesn’t work for me though if I also remove
the ‘remove-call-to-free’ phase, though.

I also tried this Classpath patch:

[Message part 2 (text/x-patch, inline)]

diff --git a/include/jni_md.h b/include/jni_md.h
index 989dbfe..f7867d7 100644
--- a/include/jni_md.h
+++ b/include/jni_md.h
@@ -32,7 +32,7 @@ executable file might be covered by the GNU General Public License. */
 #define JNIEXPORT
 #define JNIIMPORT
 
-typedef unsigned char jboolean;
+typedef int jboolean;
 typedef signed char jbyte;
 typedef unsigned short jchar;
 typedef short jshort;

[Message part 3 (text/plain, inline)]

It seems to have the same effect as my initial patch.

I’m lacking inspiration now!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.