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