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

Build: Fix jdbc jar pom to not include deps #36036

Merged
merged 1 commit into from
Nov 29, 2018
Merged

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Nov 29, 2018

This commit adds back the exclusion of dependencies from the pom file
for the jdbc jar, which was accidentally lost in #32014

This commit adds back the exclusion of dependencies from the pom file
for the jdbc jar, which was accidentally lost in elastic#32014
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@rjernst rjernst requested a review from nik9000 November 29, 2018 09:21
@alpar-t
Copy link
Contributor

alpar-t commented Nov 29, 2018

@elasticmachine test this please

1 similar comment
@rjernst
Copy link
Member Author

rjernst commented Nov 29, 2018

@elasticmachine test this please

Copy link
Member

@nik9000 nik9000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this is what I had to do before I started using the bundled configuration, yeah.

@rjernst rjernst merged commit 95d9cef into elastic:master Nov 29, 2018
@rjernst rjernst deleted the jdbc_pom branch November 29, 2018 17:39
rjernst added a commit that referenced this pull request Nov 29, 2018
This commit adds back the exclusion of dependencies from the pom file
for the jdbc jar, which was accidentally lost in #32014
rjernst added a commit that referenced this pull request Nov 29, 2018
This commit adds back the exclusion of dependencies from the pom file
for the jdbc jar, which was accidentally lost in #32014
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team v6.5.2 v6.6.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants