-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add a dune file and a new package extlib_wrapped #54
Conversation
I think the changes from dune-universe/ocaml-extlib@4e178dc to build with @ygrek what is your opinion regarding building the entire thing with dune? |
This still seems to be a problem with
|
I am ok to add it in parallel to Makefile. |
What is the state of this pr ? Can I help ? |
@@ -1,4 +1,22 @@ | |||
EXCLUDE_QUERY_DIR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be ignored by git
(action | ||
(run ./genconfig.exe))) | ||
|
||
(copy_files src/*.ml*) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can declare this dune stanza in src folder and avoid copy.
(name extlib) | ||
(public_name extlib_wrapped) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why this breakage ? if named extLib then user could still consume without noticing the switch to dune.
(name extlib) | |
(public_name extlib_wrapped) | |
(name extLib) | |
(public_name extlib) |
] | ||
build: ["dune" "build" "-p" name] | ||
depends: [ | ||
"ocaml" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dune is obviously omitted 👀 their is dune_generate_opam_file
if you want.
@@ -0,0 +1 @@ | |||
(-w -3-6-9-27-32-33-39-50 -no-strict-sequence) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't be committed
Superceded by #59, thanks! After switching to dune will make a breaking change with version bump to the wrapped library. |
This intends to solve same issue as #53