Skip to content
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 UI for specifying CMake tools and generators' locations #1067

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

DangMinhTam382
Copy link

Add UI for specifying CMake tools and generators locations

Adding UI into CMake Preference page that allow user to specify the
location of the CMake tool and the location of CMake generators.
Added CMakeBuildEnvironmentSupplier.class for ICBuildConfiguration

Addresses Issue: CDT CMake Improvements #1000, IDE-82683-REQ-004 and
IDE-82683-REQ-005

Adding UI into CMake Preference page that allow user to specify the
location of the CMake tool and the location of CMake generators.

Addresses Issue: CDT CMake Improvements eclipse-cdt#1000, IDE-82683-REQ-004 and
IDE-82683-REQ-005
Adding UI into CMake Preference page that allow user to specify the
location of the CMake tool and the location of CMake generators.

Addresses Issue: CDT CMake Improvements eclipse-cdt#1000, IDE-82683-REQ-004 and
IDE-82683-REQ-005
Adding UI into CMake Preference page that allow user to specify the
location of the CMake tool and the location of CMake generators.

Addresses Issue: CDT CMake Improvements eclipse-cdt#1000, IDE-82683-REQ-004 and
IDE-82683-REQ-005
Adding UI into CMake Preference page that allow user to specify the
location of the CMake tool and the location of CMake generators.

Addresses Issue: CDT CMake Improvements eclipse-cdt#1000, IDE-82683-REQ-004 and
IDE-82683-REQ-005
Copy link

Test Results

   600 files   -    34     600 suites   - 34   13m 40s ⏱️ - 22m 30s
10 206 tests  - 1 217  10 183 ✅  - 1 097  23 💤  - 120  0 ❌ ±0 
10 244 runs   - 1 194  10 221 ✅  - 1 076  23 💤  - 118  0 ❌ ±0 

Results for commit 18ad0ef. ± Comparison against base commit 889a5f1.

This pull request removes 1217 tests.
org.eclipse.cdt.debug.gdbjtag.core.tests.jtagdevice.GDBJtagDeviceContributionTest ‑ testGdbJtagDeviceContribution
org.eclipse.cdt.debug.gdbjtag.core.tests.launch.GDBJtagLaunchTest ‑ testGdbJtagLaunch[gdb]
org.eclipse.cdt.debug.gdbjtag.core.tests.launch.GDBJtagLaunchTest ‑ testGdbJtagLaunch[gdbserver]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithQuotes[gdb]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithQuotes[gdbserver]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithSpecialSymbols[gdb]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithSpecialSymbols[gdbserver]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithSymbols[gdb]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithSymbols[gdbserver]
org.eclipse.cdt.tests.dsf.gdb.tests.CommandLineArgsTest ‑ testSettingArgumentsWithTabs[gdb]
…

@betamaxbandit
Copy link
Contributor

Thankyou @DangMinhTam382 , I will review tomorrow (Th).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants