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

Mavenized the project #587

Closed
wants to merge 5 commits into from
Closed

Conversation

sdmonroe
Copy link

@sdmonroe sdmonroe commented Oct 7, 2016

Included a readme for adding the virtjdbc4.jar to the repo. The two projects need to be submitted to the Maven Central Repository at some point.

@seralf
Copy link

seralf commented Mar 9, 2017

please look also at #156

@TallTed
Copy link
Collaborator

TallTed commented Aug 9, 2019

@sdmonroe - Can you resolve the current conflicts? I am hopeful this can be acted on in the near future, and that resolution will help speed it along.

@sdmonroe
Copy link
Author

sdmonroe commented Aug 10, 2019 via email

@TallTed
Copy link
Collaborator

TallTed commented Aug 12, 2019

@sdmonroe - "Resolving the conflicts" basically means rebasing your PR against the current state of openlink:develop/7. You may be able to use the github web interface to do this, which it will do against the github-hosted sdmonroe:/develop/7.

You could merge the upstream commits into your branch via this comparison, but that looks potentially painful, with 927 commits and 8,508 changed files to review.

You could instead choose to use this comparison to review the 5 commits you made in October 2016 (basically, click each to view its changes in isolation), and use them to build a new Mavenizing PR (replacing this #587) against the current openlink:develop/7 fork.

@pkleef
Copy link
Collaborator

pkleef commented Jul 12, 2022

OpenLink has published its Virtuoso JDBC drivers as well as its Jena and RDF4j providers to maven central:

https://community.openlinksw.com/t/openlink-virtuoso-java-artifacts-on-maven-central

@pkleef pkleef closed this Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants