Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Vert.x Oracle Client #215

Merged
merged 1 commit into from
Oct 25, 2022
Merged

Conversation

tsegismont
Copy link
Contributor

No description provided.

@tsegismont tsegismont self-assigned this Jan 12, 2022
@tsegismont tsegismont added this to the 4.3.0 milestone Jan 12, 2022
@vietj vietj modified the milestones: 4.3.0, 4.3.1 May 12, 2022
@vietj vietj modified the milestones: 4.3.2, 4.3.3 Jul 6, 2022
@vietj vietj modified the milestones: 4.3.4, 4.3.5 Oct 1, 2022
tsegismont added a commit to eclipse-vertx/vertx-sql-client that referenced this pull request Oct 20, 2022
This is prep work for vert-x3/vertx-lang-kotlin#215

We didn't have any issue before because Oracle Client has no RxJava / Kotlin binding yet

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
Fix build issues related to Oracle Client

* on GH actions, use Java 11 for testing / deployment
* In the parent POM, change the compiler config when JDK11+ is detected so that Java 8 APIs are enforced in vertx-lang-kotlin sources
* In vertx-lang-kotlin POM, configure toolchain plugin for people running Maven with Java8

Javadoc plugin fails with JDK11
It is stricter than JDK8 and fails because source param should contain a JDK version.

Updated generated files

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
@tsegismont
Copy link
Contributor Author

@vietj fixed that one too, PTAL

@tsegismont tsegismont merged commit 6b1393e into vert-x3:master Oct 25, 2022
@tsegismont tsegismont deleted the add_oracle_client branch October 25, 2022 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants