-
Notifications
You must be signed in to change notification settings - Fork 212
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
Conversation
please look also at #156 |
@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. |
Hi Ted,
Can you provide instructions on how to go about resolving the conflicts. I
no longer have the code in my local repo.
…-sherman
On Fri, Aug 9, 2019 at 10:24 AM Ted Thibodeau Jr ***@***.***> wrote:
@sdmonroe </~https://github.com/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.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#587?email_source=notifications&email_token=AAAUR3VWUCPHWTSSKSQOGOTQDWD25A5CNFSM4CSFPKX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD367RXY#issuecomment-519960799>,
or mute the thread
</~https://github.com/notifications/unsubscribe-auth/AAAUR3UZV4PZLKEDPDTZ2TTQDWD25ANCNFSM4CSFPKXQ>
.
--
Thanks,
-sherman
Every good gift and every perfect gift is from above, and cometh down from
the Father of lights, with whom is no variableness, neither shadow of
turning.
(James 1:17)
|
@sdmonroe - "Resolving the conflicts" basically means rebasing your PR against the current state of 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 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 |
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.