RFR: JDK-8189095; Import JMC from artifactory using Jib and main makefiles

Erik Joelsson erik.joelsson at oracle.com
Thu Oct 12 13:40:37 UTC 2017


Unfortunately, it didn't stay as easy as that. After hitting snag after 
snag, I finally decided to implement some kind of general support for 
file names with spaces in them, with support in CacheFind and 
SetupCopyFiles, as well as the various install-file variants. This got a 
little bit more messy than I would have liked, but at least I added a 
test for SetupCopyFiles to verify the functionality. Also note that this 
only works fully with gnu make 4.0 or later. With 3.81, I only get 
spaces to work in the leaf file and not in any directory. We certainly 
don't want to encourage anyone to use file names with spaces anywhere 
however, so even limited support is ok IMO.

While working on the tests I found some problems with the current tests 
that I also fixed to get a clean baseline for these changes.

Webrev: http://cr.openjdk.java.net/~erikj/8189095/webrev.02/

/Erik


On 2017-10-11 18:44, Erik Joelsson wrote:
> Please review this small fix for the mac-bundles target. The current 
> solution does not handle files with spaces in them. By changing this 
> to a single rule with a recursive copy, any filenames that includes 
> spaces will be handled correctly.
>
> Webrev: http://cr.openjdk.java.net/~erikj/8189095/webrev.01/
>
> /Erik
>




More information about the build-dev mailing list