Skip to content
New issue

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

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

Already on GitHub? # to your account

Switch to dune #51

Closed
wants to merge 2 commits into from
Closed

Switch to dune #51

wants to merge 2 commits into from

Conversation

apeschar
Copy link
Contributor

@apeschar apeschar commented Sep 28, 2023

Since Oasis does not seem to be actively maintained, I think this project should switch to dune.

This PR works as long as you have libmariadbclient-dev installed. It needs to be fixed for the other supported client implementations.

  • Fix compatibility with alternative client implementations, as in the original configure script.
  • Set up automated tests that install the package using opam and run the examples.

@paurkedal
Copy link
Collaborator

Sorry for duplicating your work in #52. Do you wish to proceed with this PR, or shall I ask @ygrek to review my PR?

Switching to dune has become more urgent, as it's part of the transition to the new location, and there are some tasks related to the move which will create conflicts and duplicate work if we don't merge the dune support first.

@apeschar
Copy link
Contributor Author

apeschar commented Aug 9, 2024

Don't worry about it, I did anyway not have time to finish this. So I wholeheartedly support your PR! :)

@paurkedal paurkedal closed this Oct 17, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants