Re: pkg/44598 (lang/gcc44)

["Kai-Uwe Eckhardt" <kuehro%gmx.de@localhost>]

The following reply was made to PR pkg/44598; it has been noted by GNATS.

From: "Kai-Uwe Eckhardt" <kuehro%gmx.de@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/44598 (lang/gcc44)
Date: Wed, 09 Mar 2011 22:58:30 +0100

 Hi,
 
 I resend this to enter the database. It was sent to the package
 maintainer four days ago. On NetBSD-5.99.45 (i386) gcc44 failed to
 compile because a pthread function is detected and used in libjava,
 but pthread is not linked in. On NetBSD 5.1 (i386) the patch is
 unnecessary but as expected, it compiled fine with this in place.
 Things are complicated on NetBSD, where 5.1 and current behave
 differently and make test does not even work on amd64 and shows
 lot of failures on i386.
 I suggest closing the bug 44598, after the package has been compiled
 on amd64 under current. It builds fine on NetBSD 5.1 amd64. I had
 opened it because fortran, c++ and java needed patches to compile
 at all and to bring gfortran into a usable state for math/lapack.
 It compiles now and gfortran builds a working lapack.
 I have just started the build on NetBSD-4.99.47 amd64 to check
 that the four most commonly used NetBSD versions/platforms work.
 I will send results tomorrow and we can wait for pbulk results
 before closing 44598 to be sure.
 
 
 Kai-Uwe
 
 
 ====
 
 I have tested gcc44 checked out today. It built fine on NetBSD 5.1
 i386 but failed on current i386 in libjava/boehm.cc. The following
 patch was necessary to build. I will test this on a 5.1 system but
 it should work. It worked without the patch in a chroot sandbox.
 I am puzzled about this, but I tested it on a fresh install to
 be sure and the patch was necessary.
 
 Kai-Uwe
 
 $NetBSD$
 
 --- libjava/boehm.cc.orig    2011-03-05 18:09:36.000000000 +0000
 +++ libjava/boehm.cc
 @@ -747,7 +747,8 @@ _Jv_GCAttachThread ()
    // The registration interface is only defined on posixy systems and
    // only actually works if pthread_getattr_np is defined.
    // FIXME: until gc7 it is simpler to disable this on solaris.
 -#if defined(HAVE_PTHREAD_GETATTR_NP) && !defined(GC_SOLARIS_THREADS)
 +#if defined(HAVE_PTHREAD_GETATTR_NP) && !defined(GC_SOLARIS_THREADS) \
 +    && !defined(__NetBSD__)
    GC_register_my_thread ();
  #endif
  }
 @@ -755,7 +756,8 @@ _Jv_GCAttachThread ()
  void
  _Jv_GCDetachThread ()
  {
 -#if defined(HAVE_PTHREAD_GETATTR_NP) && !defined(GC_SOLARIS_THREADS)
 +#if defined(HAVE_PTHREAD_GETATTR_NP) && !defined(GC_SOLARIS_THREADS) \
 +   && !defined(__NetBSD__)
    GC_unregister_my_thread ();
  #endif
  }
 
 
 
 
 -- 
 --