libgcj/1421: jni.h not installed

[tromey@cygnus.com]

>Number:         1421
>Category:       libgcj
>Synopsis:       jni.h not installed
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    tromey
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Dec 20 12:19:42 PST 2000
>Closed-Date:    Sun Mar 05 18:53:36 PST 2000
>Last-Modified:  Sun Mar  5 19:00:01 PST 2000
>Originator:     Tom Tromey
>Release:        cvs trunk as of today
>Organization:
>Environment:

>Description:
Currently, jni.h is not installed.
I decided not to install it since it is not complete,
and nobody can really use it right now.
However, once we do install it, the question becomes
where.
If we install it in $(includedir), it seems like it
will shadow (or perhaps overwrite) a jni.h from some
other Java implementation.  (A problem for system
integrators like Debian.)
We could install it in the target include directory,
but we'd still have the shadowing problem.

Anthony's idea is to install it in $(includedir)/gcj,
but then have a jni.h in the target directory that
looks like this:

#ifdef _GCJ
#include <gcj/jni.h>
#else
#include_next <jni.h>
#endif

I don't know if that is the right thing to do either.
>How-To-Repeat:

>Fix:

>Release-Note:

>Audit-Trail:

Formerly PR libgcj/43

State-Changed-From-To: open->closed
State-Changed-By: tromey
State-Changed-When: Sun Mar  5 18:53:36 2000
State-Changed-Why:
    I fixed this by installing jni.h into includedir.
    I have two reasons:
    1. This is an ok place to install.  In theory a given jni.h
       should work with any Java implementation.
       (our jni.h is gcc-specific, but that is a separate problem,
       sort of)
    2. If there are problems, hopefully a system integrator
       will contact us and provide useful advice.

From: tromey@sourceware.cygnus.com
To: java-gnats@sourceware.cygnus.com
Cc:  
Subject: libgcj/43
Date: 6 Mar 2000 02:50:38 -0000

 CVSROOT:	/cvs/java
 Module name:	libgcj
 Changes by:	tromey@sourceware.cygnus.com	00/03/05 18:50:38
 
 Modified files:
 	libjava        : ChangeLog 
 	libjava/include: Makefile.am Makefile.in 
 
 Log message:
 	Fix for PR libgcj/43:
 	* include/Makefile.in: Rebuilt.
 	* include/Makefile.am (include_HEADERS): New define.
 
 Patches:
 http://gcc.gnu.org/cgi-bin/gcc/libjava/ChangeLog.diff?cvsroot=java&r1=1.306&r2=1.307
 http://gcc.gnu.org/cgi-bin/gcc/libjava/include/Makefile.am.diff?cvsroot=java&r1=1.1&r2=1.2
 http://gcc.gnu.org/cgi-bin/gcc/libjava/include/Makefile.in.diff?cvsroot=java&r1=1.5&r2=1.6
 

From: tromey@cygnus.com
To: java-gnats@sourceware.cygnus.com, tromey@cygnus.com
Cc:  
Subject: Re: libgcj/43
Date: 6 Mar 2000 02:53:36 -0000

 Synopsis: jni.h not installed
 
 State-Changed-From-To: open->closed
 State-Changed-By: tromey
 State-Changed-When: Sun Mar  5 18:53:36 2000
 State-Changed-Why:
     I fixed this by installing jni.h into includedir.
     I have two reasons:
     1. This is an ok place to install.  In theory a given jni.h
        should work with any Java implementation.
        (our jni.h is gcc-specific, but that is a separate problem,
        sort of)
     2. If there are problems, hopefully a system integrator
        will contact us and provide useful advice.
 
 http://sourceware.cygnus.com/cgi-bin/gnatsweb.pl?cmd=view&pr=43&database=java
>Unformatted: