-
Notifications
You must be signed in to change notification settings - Fork 729
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
Migrate to CMake buildsystem #90
Comments
Thanks @dnakamura. Can you outline the plan (including the great details in PR #78) for how we'll transition to cmake? |
Yeah working on fleshing out some details now |
I'd completely understand if the decision to use CMake is set in stone now, but I wondered if Bazel was at all considered as an option? By my understanding, it is made specifically for building and testing large code bases quickly, which may be beneficial for OpenJ9. |
@jbduncan I don't recall all the details but I do remember looking into Bazel somewhat. IIRC it didn't really get us anything that CMake can't do. Also it has some drawbacks compared with CMake, namely lack of platform support (I don't think it can handle AIX or z/OS) |
@dnakamura Okay, that's totally fair! Many thanks for explaining a bit of the history behind the decision to use CMake for me. :) |
This is the initial commit of the CMake build system. Its not 100% complete, but it does a large chunk. See eclipse-openj9#90 Signed-off-by: Devin Nakamura <[email protected]>
This won't complete in |
DDR has proven to be a bigger issue than anticipated. Have WIP pr open at omr. Plan to get finishing touches done today and move on (although I suspect the PR will spend considerable time in review) |
DDR PRs pending review, as is one or two changes regarding staging the source files. |
Things running behind due to some old PRs that fell on the back burner and got forgotten about, as well as having to deal with some unexpected flooding at my house. WIll work through the weekend to see if we can get things back on track |
Good luck with the cleanup! And hopefully the rising temps don't lead to more. |
FYI a bunch of z/OS CMake issues are fixed in [1]. I think we'll need all of them for OpenJ9 as well. |
Status update: My current monolthic working tree can now successfully run a full Looking at making sanity tests pass in order to run cmake builds/testing as part of nightly builds. Out of sanity test set, there are currently 20 failing tests. Will look at resolving once DDR related prs are open
|
@dnakamura FYI I just completed full migration of OMR configure/build/test to CMake in eclipse-omr/omr#3693. I have DDR disabled at the moment in the pipeline file (search of |
Discussed with @dnakamura:
|
Long running issue to track progress on migrating openj9 from UMA to CMake/
Platform Support:
Timeline
Nov 23CMake: Add support for NLS and CP tools #3675Dec 17
class library nativesCMake: Build libffi as a static library #3659CMake: Add support for NLS and CP tools #3675CMake: Don't append version suffix for jextractnatives #4041CMake: add JDK11 redirector #4042CMake: add j9vm_jdk11 #4043Dec 21
Use NASM assembler on x86-64 Linux #3293Update CMake code to handle NASM eclipse-omr/omr#3124March 1
DDR: fix ddrgen bugs involving empty structs eclipse-omr/omr#3209CMake: Add initial support for ddrgen eclipse-omr/omr#3308March 1
Rework extensible enums eclipse-omr/omr#3189Rename extensible header files in the the compiler eclipse-omr/omr#3213A number of compiler sources assume that omr is a subdirectory of openj9 #3725Use renamed headers in the compiler #4928Stop prepending includes with "omr/" where unnecessary #3726CMake: Properly add nlsgen dependencies to omrutil #5341Milestone: First pass of linux x86-64 cmake build modulo any bugs
March 20
CMake: Search for windows message compiler eclipse-omr/omr#3677CMake: Add OMR_EXE_LAUNCHER variable eclipse-omr/omr#3669Fix use after free bug in Port::omrfile_stat eclipse-omr/omr#3670CMake: Pass VERBATIM option to add_custom_command #5227CMake: Add windows support for m4 generated assembly #5228CMake: specify RUNTIME DESTINATION when installing shared libraries #5240CMake: Enable ASM_MASM on windows #5267CMake: Update libffi to support windows #5269CMake: add windows support to j9util #5270CMake: Add windows support to the vm #5271CMake: On windows link j9realtime against winmm.lib #5280March 22
Milestone: First pass of win x86-64 cmake build modulo any bugs
April 12
Validation
Misc TODOS
The text was updated successfully, but these errors were encountered: