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).

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Reply via email to