Need reviewer: Added test target and makefile cleanups
    Omair Majid 
    omajid at redhat.com
       
    Thu May  5 23:14:25 UTC 2011
    
    
  
Hi Kelly,
Kelly O'Hair <kelly.ohair at ...> writes:
> 
> 
> Need reviewer: Added test target and makefile cleanups
> 
> 6960335: Add top level 'make test' rule that uses test/Makefile, runs  
> all test batches
> 6960853: Cleanup makefiles, remove unused vars etc.
> 
> http://cr.openjdk.java.net/~ohair/openjdk7/test-target-cleanup/webrev/
> 
I know it's a little late, but I wanted to point out that this patch (sort of)
breaks using ALT_OUTPUTDIR and doing debug builds. If I set
ALT_OUTPUTDIR=/some/dir, the debug build is placed in
/some/$(PLATFORM)-$(ARCH)-$(DEBUGNAME) instead of /some/dir-$(DEBUGNAME). Is
this intentional and using ALT_OUTPUTDIR like this not supported? 
In IcedTea6, we carry a patch [1] that effectively reverts this changeset just
so that we can avoid this strange behaviour.
Thanks,
Omair
[1]
http://icedtea.classpath.org/hg/icedtea6/file/b434625f3fd8/patches/debug-dir.patch
    
    
More information about the build-dev
mailing list