[Fwd: Re: Build breakage with system jpeg and lcms and jdk-11+18]

[Fridrich Strba]

Yes, I have tested this consolidated version and it builds just fine for me.

Cheers

F.

On 21/06/18 11:57, John Paul Adrian Glaubitz wrote:
> Hi Fridrich!
> 
> Is this still an issue? Shall I open a bug report in JBS?
> 
> Adrian
> 
> On 06/15/2018 06:02 PM, Fridrich Strba wrote:
>> Hello, Magnus,
>>
>> Your original patch had a problem, since the second statement overwrote
>> the $1_SRC_HEADER_FLAGS instead of appending to it. With that, things
>> like osSupport.hpp, jawt_md.h and so on could not be found in build of
>> tests. So, I modified your patch to read as attached and the build
>> finished. It is basically a 2-liner.
>>
>> Cheers
>>
>> Fridrich
>>
>> On 15/06/18 13:10, Magnus Ihse Bursie wrote:
>>> Here's a better patch. Please try and see if it solves your problems.
>>> Unfortunately, I'm leaving on vacation for today and will be gone most
>>> of the summer, so I can't help sponsor this patch. But if it works, I'm
>>> sure someone else can take over it.
>>>
>>> If it does not work, the original patch should at the very least be
>>> changed so that EXTRA_HEADER_DIRS gets an additional "$(call
>>> GetJavaHeaderDir, $(MODULE))", instead of the -I (which I've been trying
>>> hard to eradicate from the CFLAGS).
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: generated-headers.patch
Type: text/x-patch
Size: 718 bytes
Desc: not available
URL: <https://mail.openjdk.org/pipermail/build-dev/attachments/20180621/1c85f0a1/generated-headers.patch>