Review Request: 8015644 : makefile changes to allow integration of new features
David Holmes
david.holmes at oracle.com
Mon Jun 3 23:51:44 UTC 2013
Hi Amy,
This just needed to go to build-dev list - now cc'd.
This change is unnecessary:
627 $(JRE_INFO_FILE) : $(OUTPUT_ROOT)/spec.gmk $(OUTPUT_ROOT)/source_tips
A space was added before the colon but it is not needed and is out of
style with the rest of the file.
The main change of moving the include position for the custom Images.gmk
is ok with respect to our custom Images.gmk. I haven't heard of any
external community members utilizing this mechanism yet so the change
should not affect anyone else.
Thanks,
David
On 4/06/2013 8:45 AM, Amy Wang wrote:
> Resend this as it was bounced.
>
> -------- Original Message --------
> Subject: Review Request: 8015644 : makefile changes to allow
> integration of new features
> Date: Mon, 03 Jun 2013 13:54:15 -0700
> From: Amy Wang <amy.y.wang at oracle.com>
> Organization: Oracle Corporation
> To: jdk8-dev at openjdk.java.net
>
>
>
> Hi, All,
>
> Please help in code review of the following change:
> 8015644 <http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8015644> :
> makefile changes to allow integration of new features
>
> The webrev is available at
> http://cr.openjdk.java.net/~katleman/8015644/webrev.jdk.01/
> <http://cr.openjdk.java.net/%7Ekatleman/8015644/webrev.jdk.01/>
>
> Thank you very much!
> Amy
>
>
>
>
>
More information about the build-dev
mailing list