-
Notifications
You must be signed in to change notification settings - Fork 216
Resolve confusion about Makefile projects in wizards. #1384
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
base: main
Are you sure you want to change the base?
Conversation
|
Hi @jonahgraham and @jld01 , would you like to review this change? |
Test Results 585 files - 34 585 suites - 34 13m 52s ⏱️ - 22m 46s Results for commit a122d65. ± Comparison against base commit 559c9a6. This pull request removes 1217 tests.♻️ This comment has been updated with latest results. |
7109992 to
7a8055e
Compare
The following confusions are resolved: * To create a legacy non-managed project one had to select a "Managed Build" project. * The label "Makefile Project" was used for both lecacy Managed Build and new Core Build Makefile projects. After discussion in eclipse-cdt#1305 we came to the following renaming of labels in Makefile project wizards: * Legacy projects, based on the Managed Build System (can be managed and non-managed), will be named "Classic C/C++" projects. * Makefile projects based on the new Core Build System will be named "Core Makefile" projects. Fixes eclipse-cdt#1149
7a8055e to
a122d65
Compare
|
Hi @jonahgraham and @jld01 , I think the remaining errors are unrelated to my change. |
|
The change looks comprehensive to me and I will leave it for @jld01 to review some of the content as he has been intimately involved in the discussions of #1305 to get us to this point. As for the build errors: 😢 I will figure out what is happening here, my guess is something changed in platform and this is the first commit to expose it. I am looking into it now and will rebase this change if needed. |
The problem was not ☝️ - it was the build happened to be running while LSP4E was updating its snapshot:
Build is green now. |
The following confusions are resolved:
After discussion in #1305 we came to the following renaming of labels in Makefile project wizards:
Legacy projects, based on the Managed Build System (can be managed and non-managed), will be named "Classic C/C++" projects.
Makefile projects based on the new Core Build System will be named "Core Makefile" projects.
Fixes #1149