Offer a JNIBuildItem to let extensions enable JNI

[emmanuelbernard]

See https://groups.google.com/d/topic/quarkus-dev/pLwbEB1VCP0/discussion for the conversation

[dmlloyd]

Related: https://groups.google.com/d/topic/quarkus-dev/qJCKz-CU3RU/discussion

[Dufgui]

quarkusio/quarkus-quickstarts#235
I can continue on it ? Are you ok with my quickstart/integration test case ?

[dmlloyd]

I think that we're going to want to get the native image extension work underway first (see #3013).

[emmanuelbernard]

I don't know, is the native image extension planned for soon? The JNI thing seems relatively easy to squeeze in a week.

…

On Mon, Jul 1, 2019 at 1:53 PM David M. Lloyd ***@***.***> wrote: I think that we're going to want to get the native image extension work underway first (see #3013 <#3013>). — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2999?email_source=notifications&email_token=AACJNWAK7ZPGH3ETZMK6FWDP5HVZVA5CNFSM4H3ZQXZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY54FGI#issuecomment-507232921>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACJNWEGFT2FWXWUGHYLI2TP5HVZVANCNFSM4H3ZQXZA> .

[dmlloyd]

Is it? It didn't seem easy to me, but that might be due to my familiarity (or lack thereof) with the current build phase system and the way it integrates with Maven/Gradle.

[emmanuelbernard]

I was thinking it would look like SslNativeConfigBuildItem but I did not evaluate further.

…

On Mon, Jul 1, 2019 at 2:10 PM David M. Lloyd ***@***.***> wrote: Is it? It didn't seem easy to me, but that might be due to my familiarity (or lack thereof) with the current build phase system and the way it integrates with Maven/Gradle. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2999?email_source=notifications&email_token=AACJNWHZOU7LRC2NYZK7JTLP5HX4HA5CNFSM4H3ZQXZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY55PAY#issuecomment-507238275>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACJNWGGFTTLH5SYAJNMW4DP5HX4HANCNFSM4H3ZQXZA> .

[gsmet]

Yeah, it's a hack that was present before and I refined for the SSL native support.

Basically, we push properties to the native-image.properties file which is read in NativeImagePhase (see https://github.com/quarkusio/quarkus/blob/master/core/creator/src/main/java/io/quarkus/creator/phase/nativeimage/NativeImagePhase.java#L384 ).

So, while having native image build as an extension is definitely more future proof, we could probably use this machinery temporarily to fix this one (and #3014 while we're at it).

[Dufgui]

I do the code just by following what SSL native do. Now I will think about what must really done to work and test it ;o)

[emmanuelbernard]

oops