RFR(XXS): 8062312: OpenJDK build fails when bundling freetype libraries
volker.simonis at gmail.com
Wed Oct 29 09:58:51 UTC 2014
thanks for the review.
On Wed, Oct 29, 2014 at 10:01 AM, Erik Joelsson
<erik.joelsson at oracle.com> wrote:
> Hello Volker,
> The change looks good, but I would like a comment explaining why
> install-file isn't used. Otherwise it's quite possible we will break this
> again in future cleanups.
I forget that and I'm to lazy to create a new webrev:)
Would you be fine with the following:
+ # We can't use $(install-file) in this rule because it preserves
symbolic links and
+ # libfreetype.so is usually a symbolic link to something like
libfreetype.so.6 on Unix.
+ $(MKDIR) -p $(@D)
+ $(CP) $< $@
> On 2014-10-28 20:18, Volker Simonis wrote:
>> could you please review the following tiny change which fixes the copy
>> step of the freetype libraries after 8058756:
>> After the fix for "JDK-8058756: OpenJDK builds fail on Windows -
>> cannot copy freetype.dll" the build fails on Unix if the freetype
>> library which should be bundled is a symbolic link. This is because
>> 8058756 replaced the copy operation by a call to $(install-file) which
>> preserves symlinks.
>> The fix is easy: just roll back to the initial copy operation and
>> insert an 'mkdir -p' for the target directory (as initially proposed
>> as fix for 8058756).
>> Built and smoke tested on Linux, Solaris, Windows and MacOS X.
>> Thank you and best regards,
More information about the build-dev