[Bug classpath/22878] what version of sed is needed by check_jni_methods?

[gcc-bugzilla at gcc dot gnu dot org]

What version of sed is needed by check_jni_methods?

Understandably, it does not work with BSD sed which is installed by default on
MacOS X.

However, it also doesn't work with GNU sed 4.0.5 which can be installed via
"fink install sed".  The reason is that it uses w, which "info sed" says should
match letters, digits, and underscore.  However, it doesn't seem to match
digits.  With the following patch, the script runs to completion without
complaining:

--- check_jni_methods.sh.virgin Thu Feb 17 20:20:43 2005
+++ check_jni_methods.sh        Thu Feb 17 20:27:11 2005
@@ -7,13 +7,13 @@
 # Find all methods defined in the header files generated
 # from the java source files.
 grep -h '^JNIEXPORT .* Java_' include/*.h | 
-       sed -e 's,^JNIEXPORT .* JNICALL (Java_w*) (.*$,1,' | 
+       sed -e 's,^JNIEXPORT .* JNICALL (Java_[A-Za-z0-9_]*) (.*$,1,' | 
        sort > $TMPFILE

 # Find all methods in the JNI C source files.
 find native/jni -name *.c | 
        xargs grep -h '^Java_' | 
-       sed -e 's,^(Java_w*) *(.*$,1,' 
+       sed -e 's,^(Java_[A-Za-z0-9_]*) *(.*$,1,' 
        | sort > $TMPFILE2

 # Write temporary ignore file.


------- Comment #1 from from-classpath at savannah dot gnu dot org  2005-02-18 
06:30 -------
This is fixed already in CVS. It will be included in GNU classpath 0.14 which
will be released soon (tm).


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22878