[foreign] RFR 8216483: jextract tests have to be modified to run on Windows platform

Jorn Vernee jbvernee at xs4all.nl
Fri Jan 11 13:00:03 UTC 2019


I put copyright headers into the files that I created.

There are also some other files missing copyright headers I noticed. 
Should I add them as well, if so, what year should I put in the header?

Thanks,
Jorn

Sundararajan Athijegannathan schreef op 2019-01-11 07:16:
> Looks good.
> 
> Tests run fine on Linux as well. If all fine on Windows, please push.
> 
> PS. New test source files should have copyright year 2019
> 
> -Sundar
> 
> On 11/01/19, 7:51 AM, Sundararajan Athijegannathan wrote:
>> All test run fine on Mac with this latest patch! I'll test on Linux as 
>> well and then send you review comments.
>> 
>> -Sundar
>> 
>> On 10/01/19, 9:40 PM, Jorn Vernee wrote:
>>> Hi,
>>> 
>>> Continuation of [1].
>>> 
>>> updated webrev: 
>>> http://cr.openjdk.java.net/~jvernee/panama/webrevs/windows_tests/webrev.04/
>>> 
>>> For the TestJextractFFI test I'm now just checking the operating 
>>> system in the LibClang.java patch file:
>>> 
>>>     String libName = 
>>> System.getProperty("os.name").startsWith("Windows")
>>>             ? "libclang"
>>>             : "clang";
>>>     Library libclang = Libraries.loadLibrary(MethodHandles.lookup(), 
>>> libName);
>>> 
>>> The other problem is that the dll is in the bin folder vs lib folder 
>>> on Windows, and the test was using the linker path which is the lib 
>>> folder on Windows. There was no build system variable available for 
>>> the folder that contains the library file, so instead I'm deriving it 
>>> from the library file path.
>>> 
>>> Cheers,
>>> Jorn
>>> 
>>> [1] : 
>>> https://mail.openjdk.java.net/pipermail/panama-dev/2019-January/003691.html


More information about the panama-dev mailing list