On Sat, 23 Mar 2024 00:46:45 GMT, Maurizio Cimadamore
wrote:
> I'd like to uplevel the discussion a bit. This PR started off to tweak the
> way in which `System::load` worked in AIX. We then discussed a bunch of
> options, talked about `Symbol::libraryLookup` and verified that this lookup
> a
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Fri, 22 Mar 2024 19:56:30 GMT, Martin Doerr wrote:
> The symbols get found and the JVM can really call into the
> `libclang.a(libclang.so.16)`. Impressive! (That doesn't mean that jextract is
> working. clang crashes the way we are calling it. Maybe because of a thread
> stack size or other
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Fri, 22 Mar 2024 16:53:58 GMT, Martin Doerr wrote:
> In case of jextract (jdk22 branch), we would then need something like the
> following if we want AIX to behave like the other platforms?
>
> ```
> diff --git a/src/main/java/org/openjdk/jextract/clang/libclang/Index_h.java
> b/src/main/ja
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Fri, 22 Mar 2024 14:53:17 GMT, Martin Doerr wrote:
> Only the Java developer can select the right member.
Right, and I think this problem is isomporphic to the problem we have in Linux
distros of selecting between libclang.so.1 and libclang.so.2.
`System::loadLibrary` cannot do it. So, I th
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Thu, 21 Mar 2024 22:04:28 GMT, Maurizio Cimadamore
wrote:
> This problem seems relatively similar to what happens for versioned library
> names on e.g. linux distributions - e.g. `libclang.so.2`. In such cases users
> are stuck between a rock and a hard place: using
> `System.loadLibrary("
On Thu, 21 Mar 2024 22:23:02 GMT, Maurizio Cimadamore
wrote:
> For this reason, the only thing that would make sense for `loadLibrary` to
> support is `clang` which will be expanded (by `mapLibraryName`) to
> `clang(libclang.so)`. But, even assuming this works: wouldn't we still have
> an iss
On Thu, 21 Mar 2024 22:23:02 GMT, Maurizio Cimadamore
wrote:
> > (I'm pessimistic)
>
> To summarize: I think that allowing version-specific names (even if
> surrounded by parenthesis) in `System::loadLibrary` would be very odd. After
> all, `System::loadLibrary` doesn't support versioned name
On Thu, 21 Mar 2024 20:29:16 GMT, Mandy Chung wrote:
> As @jaikiran suggests, `loadLibraryOnlyIfPresent()` should return false for
> AIX as the file does not exist. The library implementation may need to adjust
> as the current implementation uses a file path name.
Had kept it as true, as it w
On Thu, 21 Mar 2024 22:04:28 GMT, Maurizio Cimadamore
wrote:
> (I'm pessimistic)
To summarize: I think that allowing version-specific names (even if surrounded
by parenthesis) in `System::loadLibrary` would be very odd. After all,
`System::loadLibrary` doesn't support versioned names, even on
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incremen
> Allow support for both .a and .so files in AIX.
> If .so file is not found, allow fallback to .a extension.
> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
Suchismith Roy has updated the pull request incrementally with one additional
commit since the last revision:
tr
19 matches
Mail list logo