pkg/29643: sablevm-classpath (and probably vablevm-classpath-vm) build fails

[reilles%loria.fr@localhost]

>Number:         29643
>Category:       pkg
>Synopsis:       sablevm-classpath fails because it has no JAVA_WRAPPER
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    pkg-manager
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Wed Mar 09 21:15:01 +0000 2005
>Originator:     Antoine Reilles
>Release:        NetBSD 2.99.16
>Organization:
        
>Environment:
        
        
System: NetBSD arcelot.loria.fr 2.99.16 NetBSD 2.99.16 (MYKERNEL) #0: Sun Feb 
20 22:10:25 CET 2005 
root%arcelot.loria.fr@localhost:/usr/obj/sys/arch/i386/compile/MYKERNEL i386
Architecture: i386
Machine: i386
>Description:
I'm having a problem building lang/sablevm-classpath (and problably also
sablevm-classpath-gui), because they include mk/java-env.mk without
defining JAVA_WRAPPERS. This make the build fails at the end, with an
error like: make: don't know how to build work/.ALTERNATIVES

I've patched locally mk/java-env.mk to get rid of this problem (just
moving the endif part of a
.if !empty(JAVA_WRAPPERS)                                                       
               
.endif
to the end of the file, since all this code is there to handle
JAVA_WRAPPERS.

>How-To-Repeat:
        cd /usr/pkgsrc/lang/sablevm-classpath 
        make 
>Fix:
        mk/java-env.mk can be changed to handle not defined JAVA_WRAPPERS
        A proposition of patch:

Index: mk/java-env.mk
===================================================================
RCS file: /cvsroot/pkgsrc/mk/java-env.mk,v
retrieving revision 1.4
diff -b -u -r1.4 java-env.mk
--- mk/java-env.mk      30 Jan 2005 23:18:38 -0000      1.4
+++ mk/java-env.mk      9 Mar 2005 11:10:32 -0000
@@ -37,7 +37,6 @@
 
 .if !empty(JAVA_WRAPPERS)
 INSTALLATION_DIRS+=    bin
-.endif
 
 ALTERNATIVES_SRC=      ${WRKDIR}/.ALTERNATIVES
 ${WRKDIR}/.ALTERNATIVES:
@@ -78,6 +77,7 @@
 
 .endfor
 .undef w
+.endif
 
 # Handle the ${PREFIX}/java shared directory automatically.
 USE_PKGINSTALL=                YES

>Unformatted: