[kaffe] sound related configure bug

2003-01-23 Thread Timothy Stack

hi,

I'm having problems building from the current CVS repository.  The
configure script is detecting libesd in /usr/local/lib, but
/usr/local/include isn't in the include directory list when compiling.  
So, it goes boom.

Looking at configure.in, it defaults to adding /usr/local/lib if 
--with-libraries isn't specified.  So, i would suggest doing a similar 
thing for --with-includes.

AC_ARG_WITH(includes,
  [  --with-includes=dirsSearch specified directories for header files],
  [ CPPFLAGS=$CPPFLAGS`echo  $withval |
sed -e 's/ / -I/g' -e 's/-I //g' -e 's/-I$//'` ],
  [ CPPFLAGS=$CPPFLAGS ])

Changes to:

AC_ARG_WITH(includes,
  [  --with-includes=dirsSearch specified directories for header files],
  [ CPPFLAGS=$CPPFLAGS`echo  $withval |
sed -e 's/ / -I/g' -e 's/-I //g' -e 's/-I$//'` ],
  [ CPPFLAGS=$CPPFLAGS -I/usr/local/include ])


How's that sound?

thanks,

tim stack

___
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe



[kaffe] --enable-pure-java-math not working yet

2003-01-23 Thread Timothy Stack

hi,

The --enable-pure-java-math option doesn't seem to work quite right yet.  
When trying to link the main executable it can't find libmath since it was 
edited out of the build process.  I think you just need to change 
configure.in to leave libmath out of the JAVA_LIBS variable if pure java 
math is turned on.

JAVA_LIBS= \
\$(top_builddir)/libraries/clib/native/libnative.la \
\$(top_builddir)/libraries/clib/net/libnet.la \
\$(top_builddir)/libraries/clib/io/libio.la \
\$(top_builddir)/libraries/clib/zip/libzip.la \
\$(top_builddir)/libraries/clib/math/libmath.la \
\$(top_builddir)/libraries/clib/management/libmanagement.la \
\$(top_builddir)/libraries/clib/security/libsecurity.la \
\$(top_builddir)/libraries/extensions/microsoft/clib/libmicrosoft.la \
$AWT_LIB \
$JAVA_LIBS \


changes to:

JAVA_LIBS= \
\$(top_builddir)/libraries/clib/native/libnative.la \
\$(top_builddir)/libraries/clib/net/libnet.la \
\$(top_builddir)/libraries/clib/io/libio.la \
\$(top_builddir)/libraries/clib/zip/libzip.la \
$MATH_LIB \
\$(top_builddir)/libraries/clib/management/libmanagement.la \
\$(top_builddir)/libraries/clib/security/libsecurity.la \
\$(top_builddir)/libraries/extensions/microsoft/clib/libmicrosoft.la \
$AWT_LIB \
$JAVA_LIBS \


(I'd submit a patch but i hate messing with autoconf/automake stuff...)

thanks,

tim stack

___
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe



Re: [kaffe] verifier and threads

2003-01-23 Thread Helmer Krämer
On Tue, 21 Jan 2003 11:13:57 -0700
Patrick Tullmann [EMAIL PROTECTED] wrote:

 Helmer wrote:
  seems like I tracked this one down. The problem was that
  Class.forName () catched the VerifyError and threw a
  ClassNotFoundException instead. I've attached a pretty
  simple patch that should fix this.
 
 While I admit that's weird to replace one exception with the other I'm
 quite certain it was done for a reason.  Since there aren't any
 comments of note, I'm guessing that the code you removed intentionally
 changed an unchecked Error into a checked Exception, so that users of
 the method would have a chance to deal with it.

Yep, I don't think it was changed without reason either, but

- the api spec states that forName throws a LinkageError and
  VerifyError is a subclass of it

- sun's jdk throws VerifyErrors

- if you simply convert a VerifyError into a ClassNotFoundException
  you can't tell whether the class was not found or whether the
  class was found but contains invalid byte code (or whatever)

- how would you handle a VerifyError? 

Greetings,
Helmer

___
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe



[kaffe] Kaffe CVS: kaffe dalibor

2003-01-23 Thread Kaffe CVS

CVSROOT:/cvs/kaffe
Module name:kaffe
Changes by: dalibor 03/01/23 16:07:32

Modified files:
.  : ChangeLog Makefile.in configure configure.in 
config : Makefile.in 
include: Makefile.in 
kaffe  : Makefile.in 
kaffe/kaffe: Makefile.in 
kaffe/kaffeh   : Makefile.in support.c 
kaffe/kaffevm  : Makefile.in 
kaffe/kaffevm/gcj: Makefile.in 
kaffe/kaffevm/intrp: Makefile.in 
kaffe/kaffevm/jit: Makefile.in 
kaffe/kaffevm/jit3: Makefile.in 
kaffe/kaffevm/systems: Makefile.in 
kaffe/kaffevm/systems/beos-native: Makefile.in 
kaffe/kaffevm/systems/oskit-pthreads: Makefile.in 
kaffe/kaffevm/systems/unix-jthreads: Makefile.in 
kaffe/kaffevm/systems/unix-pthreads: Makefile.in 
kaffe/man  : Makefile.in 
kaffe/scripts  : Makefile.in 
kaffe/scripts/bat: Makefile.in 
kaffe/scripts/compat: Makefile.in 
kaffe/xprof: Makefile.in 
libraries  : Makefile.in 
libraries/clib : Makefile.in 
libraries/clib/awt: Makefile.in 
libraries/clib/awt/X: Makefile.in 
libraries/clib/awt/qt: Makefile.in 
libraries/clib/io: Makefile.in 
libraries/clib/management: Makefile.in 
libraries/clib/math: Makefile.in 
libraries/clib/native: Makefile.in 
libraries/clib/net: Makefile.in 
libraries/clib/security: Makefile.in 
libraries/clib/zip: Makefile.in 
libraries/extensions: Makefile.in 
libraries/extensions/comm: Makefile.in 
libraries/extensions/comm/javalib: Makefile.in 
libraries/extensions/microsoft: Makefile.in 
libraries/extensions/microsoft/clib: Makefile.in 
libraries/extensions/microsoft/javalib: Makefile.in 
libraries/extensions/pjava: Makefile.in 
libraries/extensions/pjava/javalib: Makefile.in 
libraries/extensions/servlet: Makefile.in 
libraries/extensions/servlet/javalib: Makefile.in 
libraries/extensions/sound: Makefile.in 
libraries/extensions/sound/alsa: Makefile.in 
libraries/extensions/sound/common: Makefile.in 
libraries/extensions/sound/esd: Makefile.in 
libraries/javalib: Makefile.in 
libraries/javalib/java/lang: Class.java 
test   : Makefile.in 
test/regression: Makefile.in 

Log message:
2003-01-23 Dalibor Topic [EMAIL PROTECTED]

* configure.in:
Fixed --enable-pure-java-math.

Reported by: Timothy S Stack [EMAIL PROTECTED]

* libraries/javalib/java/lang/Class.java:
(forName) removed now unnecessary comments.

* Makefile.in,
config/Makefile.in,
configure,
include/Makefile.in,
kaffe/Makefile.in,
kaffe/kaffe/Makefile.in,
kaffe/kaffeh/Makefile.in,
kaffe/kaffevm/Makefile.in,
kaffe/kaffevm/gcj/Makefile.in,
kaffe/kaffevm/intrp/Makefile.in,
kaffe/kaffevm/jit/Makefile.in,
kaffe/kaffevm/jit3/Makefile.in,
kaffe/kaffevm/systems/Makefile.in,
kaffe/kaffevm/systems/beos-native/Makefile.in,
kaffe/kaffevm/systems/oskit-pthreads/Makefile.in,
kaffe/kaffevm/systems/unix-jthreads/Makefile.in,
kaffe/kaffevm/systems/unix-pthreads/Makefile.in,
kaffe/man/Makefile.in,
kaffe/scripts/Makefile.in,
kaffe/scripts/bat/Makefile.in,
kaffe/scripts/compat/Makefile.in,
kaffe/xprof/Makefile.in,
libraries/Makefile.in,
libraries/clib/Makefile.in,
libraries/clib/awt/Makefile.in,
libraries/clib/awt/X/Makefile.in,
libraries/clib/awt/qt/Makefile.in,
libraries/clib/io/Makefile.in,
libraries/clib/management/Makefile.in,
libraries/clib/math/Makefile.in,
libraries/clib/native/Makefile.in,
libraries/clib/net/Makefile.in,
libraries/clib/security/Makefile.in,
libraries/clib/zip/Makefile.in,
libraries/extensions/Makefile.in,
libraries/extensions/comm/Makefile.in,
libraries/extensions/comm/javalib/Makefile.in,
libraries/extensions/microsoft/Makefile.in,
libraries/extensions/microsoft/clib/Makefile.in,
libraries/extensions/microsoft/javalib/Makefile.in,
libraries/extensions/pjava/Makefile.in,
libraries/extensions/pjava/javalib/Makefile.in,
libraries/extensions/servlet/Makefile.in,
libraries/extensions/servlet/javalib/Makefile.in,
libraries/extensions/sound/Makefile.in,
libraries/extensions/sound/alsa/Makefile.in,
libraries/extensions/sound/common/Makefile.in,
libraries/extensions/sound/esd/Makefile.in,
libraries/javalib/Makefile.in,
test/Makefile.in,
test/regression/Makefile.in:
regenerated.

2003-01-21 Helmer Kraemer  [EMAIL PROTECTED]

* libraries/javalib/java/lang/Class.java:
(forName) don't catch VerifyErrors

Reported by: Robert N. Gonzalez [EMAIL PROTECTED]

* kaffe/kaffeh/support.c:
(_methodRing) use a boolean flag to decide whether method signature
needs to be mangled.
(addMethod) set needs_mangled_sig accordingly.
(finishMethods) only mangle sig if needs_mangled_sig has been set.

Reported by: Marc Kleine-Budde [EMAIL PROTECTED]

2003-01-21 Patrick Tullmann  [EMAIL PROTECTED]

* configure.in:

Re: [kaffe] kaffeh buserror

2003-01-23 Thread Dalibor Topic
Hallo Helmer,

--- Helmer Krämer [EMAIL PROTECTED] wrote:

 You could try whether the attached patch solves the
 problem
 or if I've managed to mess up something else ;)

I don't have access to a powerpc platform, but the
patch looks good to me, as it simplifies the code, and
makes it more understandable. I've checked it in.

best regards,
dalibor topic

__
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com

___
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe



Re: [kaffe] verifier and threads

2003-01-23 Thread Dalibor Topic
Hallo Helmer,

--- Helmer Krämer [EMAIL PROTECTED] wrote:
 On Tue, 21 Jan 2003 11:13:57 -0700
 Patrick Tullmann [EMAIL PROTECTED] wrote:
 
  Helmer wrote:
   seems like I tracked this one down. The problem
 was that
   Class.forName () catched the VerifyError and
 threw a
   ClassNotFoundException instead. I've attached a
 pretty
   simple patch that should fix this.
  
  While I admit that's weird to replace one
 exception with the other I'm
  quite certain it was done for a reason.  Since
 there aren't any
  comments of note, I'm guessing that the code you
 removed intentionally
  changed an unchecked Error into a checked
 Exception, so that users of
  the method would have a chance to deal with it.
 
 Yep, I don't think it was changed without reason
 either, but
 
 - the api spec states that forName throws a
 LinkageError and
   VerifyError is a subclass of it
 
 - sun's jdk throws VerifyErrors
 
 - if you simply convert a VerifyError into a
 ClassNotFoundException
   you can't tell whether the class was not found or
 whether the
   class was found but contains invalid byte code (or
 whatever)
 
 - how would you handle a VerifyError? 

Yeah, and additionally there is no discussion of it on
the mailing list, as I'd assume in the case it was
done to get an application to run, and the comment is
very unclear. Beside that, your patch doesn't make any
regression test fail, so due to the lack of evidence
to the contrary, I've checked it in. ;)

Whatever the reason was to catch the verifier error,
it doesn't seem to be present anymore.

best regards,
dalibor topic


__
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com

___
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe