Skip to content

Commit

Permalink
CMake: Fixed the correct path for starting 'cmake-gui'
Browse files Browse the repository at this point in the history
This PR fixed the wrong source flag which was set by spawning the cmake-gui command.

Closes #532
  • Loading branch information
gravemalte authored and jonahgraham committed Sep 5, 2023
1 parent 5cedafe commit 54a7f61
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cmake/org.eclipse.cdt.cmake.ui/META-INF/MANIFEST.MF
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ Manifest-Version: 1.0
Bundle-ManifestVersion: 2
Bundle-Name: %pluginName
Bundle-SymbolicName: org.eclipse.cdt.cmake.ui;singleton:=true
Bundle-Version: 1.4.100.qualifier
Bundle-Version: 1.4.200.qualifier
Bundle-Activator: org.eclipse.cdt.cmake.ui.internal.Activator
Bundle-Vendor: %providerName
Require-Bundle: org.eclipse.core.runtime,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ public void widgetSelected(SelectionEvent e) {
String sourceDir = project.getLocation().toOSString();
String buildDir = project.getLocation().append("build").append(configName).toOSString(); //$NON-NLS-1$

Runtime.getRuntime().exec(new String[] { "cmake-gui", "-H" + sourceDir, "-B" + buildDir }); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
Runtime.getRuntime().exec(new String[] { "cmake-gui", "-S" + sourceDir, "-B" + buildDir }); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
} catch (CoreException | IOException e1) {
MessageDialog.openError(parent.getShell(),
Messages.CMakePropertyPage_FailedToStartCMakeGui_Title,
Expand Down

0 comments on commit 54a7f61

Please sign in to comment.