RFR: 8046765 : (s) makefiles should use parameterized $(CP) and $(MV) rather than explicit commands

Mike Duigou mike.duigou at oracle.com
Sat Jul 12 17:19:54 UTC 2014


Hello all;

After further testing some additional changes were found to be needed to support building hotspot without configure support. There are a small number of additional changes in various buildtree.make and the windows build.make files to ensure that locations are defined for CP and MV commands. If there's a more appropriate location for these definitions please suggest it.

The patch is otherwise unchanged from the month ago version except for line number offset changes owing to other intervening changesets.

jbsbug: https://bugs.openjdk.java.net/browse/JDK-8046765
webrev: http://cr.openjdk.java.net/~mduigou/JDK-8046765/2/webrev/

Thanks!

Mike

On Jun 13 2014, at 12:43 , Mike Duigou <mike.duigou at oracle.com> wrote:

> Hello all;
> 
> This is a small changeset to the hotspot makefiles to have them use expansions of the $(CP) and $(MV) variables rather than explicit commands for all operations involving files in the deliverables. This changes is needed by static code analysis software which provides replacement cp and mv commands that track error reports in executables back to the source from which they are generated.
> 
> jbsbug: https://bugs.openjdk.java.net/browse/JDK-8046765
> webrev: http://cr.openjdk.java.net/~mduigou/JDK-8046765/0/webrev/
> 
> I've checked to make sure that patch doesn't change the build output on linux x64 and am currently checking on other platforms.
> 
> It is probably easier to review this change by looking at the patch than by looking at the file diffs.
> 
> Mike
> 



More information about the hotspot-dev mailing list