[7u6] Request for approval for CR 7157855: jvisualvm.1 not included in binaries (original) (raw)
David Holmes david.holmes at oracle.com
Wed Aug 22 01:02:47 UTC 2012
- Previous message (by thread): [7u6] Request for approval for CR 7157855: jvisualvm.1 not included in binaries
- Next message (by thread): [7u6] Request for approval for CR 7157855: jvisualvm.1 not included in binaries
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 21/08/2012 9:37 PM, Seán Coffey wrote:
Looks fine to me but best to get an official jdk8 reviewer.
Looks fine to me too.
I presume the filter-out option on macosx becomes a no-op (on OpenJDK builds) some lines later : (Images.gmk)
264 JDKMANPAGES := (filter−outjvisualvm.1,(filter-out jvisualvm.1, (filter−outjvisualvm.1,(JDKMANPAGES))
Yes.
On the subject of dual Makefile maintenance, are there plans to eventually remove the older/legacy makefile structure on 8? I haven't been monitoring that topic too closely.
Yes. At some point "real soon now" we should make the switch to using the new build system for our official builds. Some of the work for 8 is only being done in the context of the new build system.
David
regards, Sean.
On 21/08/2012 12:13, Andrew Hughes wrote: Ah good catch. I didn't realise this was duplicated in the new build system.
http://cr.openjdk.java.net/~andrew/jvisualvm/webrev.02/ should deal with both cases. If this is ok, is there a preferred forest to push to? I've been testing against build, but can easily push it somewhere else.
- Previous message (by thread): [7u6] Request for approval for CR 7157855: jvisualvm.1 not included in binaries
- Next message (by thread): [7u6] Request for approval for CR 7157855: jvisualvm.1 not included in binaries
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]