-
-
Notifications
You must be signed in to change notification settings - Fork 670
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
Migration to dune #8801
Merged
Merged
Migration to dune #8801
Changes from 24 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
1441899
kill merlin
Simn 83c1e3f
ignore merlin
Simn 9ca7589
start on migration to dune
Simn 97a629c
add generated files to source for now
Simn 122a77f
get something working
Simn 4748153
move linker flags to libs
Simn edf1bdc
deal with prebuild properly
Simn ad53221
naively try linux CI
Simn 6c6c061
actually...
Simn 1e2ed00
try windows
Simn f48ee5a
unwrap libs for now to reduce diff
Simn 580a105
don't run weird things on mac
Simn ac0ecfd
don't sedlex so much
Simn f308a23
don't include sedlex.ppx as a library
Simn f2d47c9
we don't have to wrap
Simn ffb8de9
reduce diff
Simn 38e016a
always link statically for now so we can get the CI green maybe
Simn a188327
Revert "always link statically for now so we can get the CI green maybe"
Simn fc768da
try some hackerino to get the link flags into dune
Simn 0a60ede
quote me on this
Simn 212c506
add awkward version.ml handling
Simn d6ddcad
reduce diff
Simn b47da99
remove simn-make leftovers
Simn c965805
don't scan everything
Simn 25ac43b
unify envs
Simn d80d856
add a separate step to build environment dependencies to makefile
jdonaldson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
{ | ||
"version": "2.0.0", | ||
"version": "2.0.0", | ||
"tasks": [ | ||
{ | ||
"label": "make: haxe", | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
(data_only_dirs extra lib std tests) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
(lang dune 1.11) | ||
(name haxe) | ||
|
||
(package | ||
(name haxe) | ||
) | ||
|
||
(package | ||
(name haxe_prebuild) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
(lang dune 1.11) | ||
(profile release) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name extc) | ||
(libraries extlib) | ||
(c_names extc_stubs) | ||
(modules extc) | ||
(wrapped false) | ||
) | ||
|
||
(library | ||
(name extproc) | ||
(c_names process_stubs) | ||
(modules process) | ||
(wrapped false) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name extlib_leftovers) | ||
(libraries extlib) | ||
(wrapped false) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name ilib) | ||
(modules_without_implementation ilData ilMeta) | ||
(libraries extlib) | ||
(wrapped false) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name javalib) | ||
(libraries extlib) | ||
(wrapped false) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name json) | ||
(preprocess (pps sedlex.ppx)) | ||
) |
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name neko) | ||
(libraries extlib) | ||
(wrapped false) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name objsize) | ||
(c_names c_objsize) | ||
(c_flags (-I../../../../libs/objsize)) ; TODO: This is stupid | ||
(wrapped false) | ||
(modules objsize) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name pcre) | ||
(c_names pcre_stubs) | ||
(wrapped false) | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
(include_subdirs no) | ||
|
||
(library | ||
(name swflib) | ||
(libraries extc extlib extlib_leftovers) | ||
(modules_without_implementation as3 as3hl) | ||
(wrapped false) | ||
) |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Do we still need the suffix rules?
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.
Probably not.
I was thinking that we could absorb most of the makefile into the prebuild program anyway. That should be more fun to maintain. We would then just use the makefile targets to pass information to prebuild.