[Bug general/4087] New: New gcj doesn't provide jv-scan

[Bug general/4087] New: New gcj doesn't provide jv-scan

[mark at klomp dot org]

The latest gcj does away with its own .java source parser and doesn't provide
jv-scan or something equivalent anymore.

jv-scan is used to select .java source classes that have a public static void
main() method in common/Makefile.gen.sh has_main(). And to generate the classes
list in the CNi header .java.h target in common/Makefile.rules.

-- 
           Summary: New gcj doesn't provide jv-scan
           Product: frysk
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
        AssignedTo: frysk-bugzilla at sourceware dot org
        ReportedBy: mark at klomp dot org
OtherBugsDependingO 1600
             nThis:


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[cagney at redhat dot com]

------- Additional Comments From cagney at redhat dot com  2007-02-21 16:58 -------
The search for "main" was originally a grep.  It was changed to jv-scan to make
it more robust (kept finding main in comments and the like).  It can always be
reverted back to grep.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[cagney at redhat dot com]

------- Additional Comments From cagney at redhat dot com  2007-02-21 17:01 -------
the .java.o rule is an optional check that there is no duplicate class (which
gcj has in the past botched).  It can be wrapped in a HAVE_JV_SCAN conditional.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[cagney at redhat dot com]

------- Additional Comments From cagney at redhat dot com  2007-02-21 17:10 -------
The .java.h rule is generating the .h file for all sub-classes of the .java
file.  The sequence:

- get list of classes from .java using jv-scan
- compile to .class .java using gcj
- generate .h file for each class using gcjh

can probably fudge this using a find looking for the classes; or somehow
directly get all .h files generated.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[cagney at redhat dot com]

------- Additional Comments From cagney at redhat dot com  2007-02-21 17:49 -------
This fixes the .java.o problem:

Index: frysk-core/common/ChangeLog
2007-02-21  Andrew Cagney  <cagney@redhat.com>

        * frysk-common.ac (JV_SCAN): Default to "no".
        (HAVE_JV_SCAN): Set using AM_CONDITIONAL.
        * Makefile.rules (.java.o): Wrap use of JV_SCAN in HAVE_JV_SCAN.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[cagney at redhat dot com]

------- Additional Comments From cagney at redhat dot com  2007-02-21 18:40 -------
This removes the use in the Makefile.rules' JUnitTests.java rule:

Index: common/ChangeLog
2007-02-21  Andrew Cagney  <cagney@redhat.com>

        * Makefile.rules ($(GEN_SOURCENAME)/JUnitTests.java): Simplify,
        remove check for a main program in the test file.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[cagney at redhat dot com]

------- Additional Comments From cagney at redhat dot com  2007-02-21 19:01 -------
This eliminates jv-scan from Makefile.gen.sh.

Index: common/ChangeLog
2007-02-21  Andrew Cagney  <cagney@redhat.com>

        * Makefile.gen.sh (has_java_main): New function.
        (has_main): Use.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug general/4087] New gcj doesn't provide jv-scan

[mark at klomp dot org]

------- Additional Comments From mark at klomp dot org  2007-02-21 19:45 -------
And this should eliminate the last usage:

2007-02-21  Mark Wielaard  <mark@klomp.org>

        * frysk-common.ac (AC_CHECK_PROGS): Try gjavah also.
        * Makefile.rules (.java.h): Don't use jv-scan, compile and use
        generated classes found by find and sed.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=4087

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.