From 3561f17974d514fff5dd2f12a64f686c1477b541 Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Mon, 24 Sep 2012 20:22:49 +0000 Subject: [PATCH 01/10] git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24632 a326200e-df2c-42e0-81fa-597e909af41d From 7d5ef2579807c2597e897eb84fae30086512eaec Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Tue, 25 Sep 2012 20:53:59 +0000 Subject: [PATCH 02/10] X10DT - PTP6 - initial commit git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24645 a326200e-df2c-42e0-81fa-597e909af41d --- .../CommunicationInterfaceSectionPart.java | 24 ++++++++++--------- .../cpp/editors/ConfNameSectionPart.java | 5 ++-- .../launching/CommunicationInterfaceTab.java | 17 +++++++------ .../cpp/launching/CppApplicationTab.java | 9 ++++++- .../CppLaunchConfigurationDelegate.java | 14 +++++++---- .../cpp/launching/X10CppLaunchShortcut.java | 7 ++++-- .../ConnectionConfiguration.java | 3 ++- .../cpp/platform_conf/LoadLevelerConf.java | 2 +- .../MessagePassingInterfaceConf.java | 3 ++- .../ParallelEnvironmentConf.java | 2 +- .../ui/launch/cpp/utils/PTPConfUtils.java | 14 +++++------ 11 files changed, 59 insertions(+), 41 deletions(-) diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/CommunicationInterfaceSectionPart.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/CommunicationInterfaceSectionPart.java index 96cce975..eb680b26 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/CommunicationInterfaceSectionPart.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/CommunicationInterfaceSectionPart.java @@ -384,17 +384,19 @@ private void createClient(final IManagedForm managedForm, final FormToolkit tool private ICITypeConfigurationPart createCITypeConfigurationPart(final IServiceProvider serviceProvider) { final String remoteServiceId = serviceProvider.getId(); - IResourceManager rm = ModelManager.getInstance().getResourceManagerFromUniqueName(((ResourceManagerServiceProvider)serviceProvider).getUniqueName()); - if (OPEN_MPI_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { - return new OpenMPITypeConfigPart((IOpenMPIResourceManagerConfiguration) rm.getControlConfiguration()); - } else if (MPICH2_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { - return new MPICH2TypeConfigPart((IToolRMConfiguration) rm.getControlConfiguration()); - } else if (PARALLEL_ENVIRONMENT_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { - return new ParallelEnvironmentTypeConfigPart((PEResourceManagerConfiguration) rm.getControlConfiguration()); - } else if (LOAD_LEVELER_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { - //return new LoadLevelerTypeConfigPart((IIBMLLResourceManagerConfiguration) rm.getControlConfiguration()); - return new LoadLevelerTypeConfigPart(); - } else if (SOCKETS_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { +// IResourceManager rm = ModelManager.getInstance().getResourceManagerFromUniqueName(((ResourceManagerServiceProvider)serviceProvider).getUniqueName()); +// if (OPEN_MPI_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { +// return new OpenMPITypeConfigPart((IOpenMPIResourceManagerConfiguration) rm.getControlConfiguration()); +// } else if (MPICH2_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { +// return new MPICH2TypeConfigPart((IToolRMConfiguration) rm.getControlConfiguration()); +// } else if (PARALLEL_ENVIRONMENT_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { +// return new ParallelEnvironmentTypeConfigPart((PEResourceManagerConfiguration) rm.getControlConfiguration()); +// } else if (LOAD_LEVELER_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { +// //return new LoadLevelerTypeConfigPart((IIBMLLResourceManagerConfiguration) rm.getControlConfiguration()); +// return new LoadLevelerTypeConfigPart(); +// } else + + if (SOCKETS_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { return new HostFileAndListTypeConfigPart(); } else if (PAMI_SERVICE_PROVIDER_ID.equals(remoteServiceId)) { return new PAMITypeConfigPart(); diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/ConfNameSectionPart.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/ConfNameSectionPart.java index e8aa5b57..eced882c 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/ConfNameSectionPart.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/editors/ConfNameSectionPart.java @@ -9,6 +9,7 @@ import java.util.Collection; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.core.events.IResourceManagerChangedEvent; import org.eclipse.ptp.core.events.IResourceManagerAddedEvent; @@ -53,14 +54,14 @@ final class ConfNameSectionPart extends AbstractCommonSectionFormPart implements addCompletePartListener(formPage); ServiceModelManager.getInstance().addEventListener(this, IServiceModelEvent.SERVICE_CONFIGURATION_ADDED | IServiceModelEvent.SERVICE_CONFIGURATION_REMOVED); - PTPCorePlugin.getDefault().getModelManager().addListener(this); + ModelManager.getInstance().addListener(this); } // --- IFormPart's interface methods implementation public void dispose() { ServiceModelManager.getInstance().removeEventListener(this); - PTPCorePlugin.getDefault().getModelManager().removeListener(this); + ModelManager.getInstance().removeListener(this); removeCompletePartListener(getFormPage()); } diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CommunicationInterfaceTab.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CommunicationInterfaceTab.java index 601a162d..ae450e28 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CommunicationInterfaceTab.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CommunicationInterfaceTab.java @@ -28,6 +28,7 @@ import org.eclipse.osgi.util.NLS; import org.eclipse.ptp.core.IModelManager; import org.eclipse.ptp.core.IPTPLaunchConfigurationConstants; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.core.elements.IPResourceManager; import org.eclipse.ptp.core.elements.IPUniverse; @@ -95,8 +96,7 @@ public void widgetSelected(final SelectionEvent event) { final IRMLaunchConfigurationDynamicTab rmDynamicTab = getRMLaunchConfigurationDynamicTab(resourceManager); if (rmDynamicTab != null) { updateConfiguration(configuration, rmDynamicTab); - rmDynamicTab.initializeFrom(CommunicationInterfaceTab.this.fComposite, resourceManager, null /* queue */, - getLaunchConfiguration()); + rmDynamicTab.initializeFrom(getLaunchConfiguration()); } } @@ -142,7 +142,7 @@ public void performApply(final ILaunchConfigurationWorkingCopy configuration) { configuration.setAttribute(ATTR_USE_PLATFORM_CONF_DATA, this.fSynchronizationBt.getSelection()); final IRMLaunchConfigurationDynamicTab rmDynamicTab = getRMLaunchConfigurationDynamicTab(this.fResourceManager); if (rmDynamicTab != null) { - final RMLaunchValidation validation = rmDynamicTab.performApply(configuration, this.fResourceManager, null); + final RMLaunchValidation validation = rmDynamicTab.performApply(configuration); if (! validation.isSuccess()) { setErrorMessage(validation.getMessage()); } @@ -151,7 +151,7 @@ public void performApply(final ILaunchConfigurationWorkingCopy configuration) { } public void setDefaults(final ILaunchConfigurationWorkingCopy configuration) { - final IModelManager modelManager = PTPCorePlugin.getDefault().getModelManager(); + final IModelManager modelManager = ModelManager.getInstance(); final IPUniverse universe = modelManager.getUniverse(); IResourceManager resourceManager = null; if (universe != null) { @@ -177,7 +177,7 @@ public void setDefaults(final ILaunchConfigurationWorkingCopy configuration) { setErrorMessage(NLS.bind(LaunchMessages.CIT_NoLaunchConfigForRM, new Object[] { resourceManager.getName() })); } else { - rmDynamicTab.setDefaults(configuration, resourceManager, null); + rmDynamicTab.setDefaults(configuration); } } @@ -279,7 +279,7 @@ public boolean isValid(final ILaunchConfiguration configuration) { if (rmDynamicTab.getControl() == null) { initializeDynamicTab(configuration, rmDynamicTab); } - final RMLaunchValidation validation = rmDynamicTab.isValid(configuration, this.fResourceManager, null); + final RMLaunchValidation validation = rmDynamicTab.isValid(configuration); if (! validation.isSuccess()) { setErrorMessage(validation.getMessage()); return false; @@ -355,14 +355,13 @@ private void initializeDynamicTab(final ILaunchConfiguration configuration, } this.fComposite.setLayout(new GridLayout(1, false)); - rmDynamicTab.createControl(this.fComposite, this.fResourceManager, null /* queue */); + rmDynamicTab.createControl(this.fComposite, this.fResourceManager.getUniqueName()); rmDynamicTab.getControl().setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true)); if (configuration instanceof ILaunchConfigurationWorkingCopy) { updateConfiguration((ILaunchConfigurationWorkingCopy) configuration, rmDynamicTab); } - rmDynamicTab.initializeFrom(this.fComposite, this.fResourceManager, null /* queue */, - getLaunchConfiguration()); + rmDynamicTab.initializeFrom(getLaunchConfiguration()); this.fParent.setMinSize(this.fComposite.computeSize(this.fSynchronizationBt.getBounds().width, SWT.DEFAULT)); } diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java index a3cdf58f..4a412e28 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java @@ -38,6 +38,7 @@ import org.eclipse.jface.viewers.IStructuredSelection; import org.eclipse.jface.window.Window; import org.eclipse.osgi.util.NLS; +import org.eclipse.ptp.core.IPTPLaunchConfigurationConstants; import org.eclipse.ptp.launch.ui.LaunchConfigurationTab; import org.eclipse.ptp.launch.ui.LaunchImages; import org.eclipse.swt.SWT; @@ -78,6 +79,7 @@ import x10dt.ui.launch.cpp.platform_conf.ICppCompilationConf; import x10dt.ui.launch.cpp.platform_conf.IX10PlatformConf; import x10dt.ui.launch.cpp.platform_conf.X10PlatformConfFactory; +import x10dt.ui.launch.cpp.utils.PTPConfUtils; import x10dt.ui.launch.cpp.utils.PlatformConfUtils; import x10dt.ui.utils.LaunchUtils; @@ -126,7 +128,8 @@ public void performApply(final ILaunchConfigurationWorkingCopy configuration) { configuration.setAttribute(org.eclipse.jdt.launching.IJavaLaunchConfigurationConstants.ATTR_MAIN_TYPE_NAME, this.fMainTypeText.getText().trim()); configuration.setAttribute(ATTR_X10_MAIN_CLASS, this.fMainTypeText.getText().trim()); - + configuration.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, PTPConfUtils.findResourceManagerByName(fX10PlatformConf.getName()).getUniqueName()); + final String content = this.fPgrmArgsText.getText().trim(); configuration.setAttribute(ATTR_ARGUMENTS, (content.length() > 0) ? content : null); @@ -135,6 +138,8 @@ public void performApply(final ILaunchConfigurationWorkingCopy configuration) { } public void setDefaults(final ILaunchConfigurationWorkingCopy configuration) { + configuration.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, PTPConfUtils.findResourceManagerByName(fX10PlatformConf.getName()).getUniqueName()); + final IResource context = getCurrentSelectionContext(); if (context == null) { configuration.setAttribute(ATTR_PROJECT_NAME, (String) null); @@ -162,9 +167,11 @@ public void setDefaults(final ILaunchConfigurationWorkingCopy configuration) { CppLaunchCore.log(except.getStatus()); } } + configuration.setAttribute(ATTR_WORKING_DIR, (String) null); configuration.setAttribute(ATTR_ARGUMENTS, (String) null); configuration.setAttribute(ATTR_CONSOLE, true); + } // --- Overridden methods diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java index e05eca10..d483cf07 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java @@ -48,6 +48,7 @@ import org.eclipse.jdt.core.JavaCore; import org.eclipse.jface.dialogs.ErrorDialog; import org.eclipse.osgi.util.NLS; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.core.attributes.ArrayAttribute; import org.eclipse.ptp.core.attributes.AttributeManager; @@ -134,6 +135,8 @@ public void launch(final ILaunchConfiguration configuration, final String mode, new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, Constants.EMPTY_STR)); return; } + + if (!monitor.isCanceled() && shouldProcessToLinkStep(fProject) && createExecutable(configuration, fProject, new SubProgressMonitor(monitor, 5)) == 0) { // Then, performs the launch. @@ -372,7 +375,8 @@ protected void updateAttributes(final ILaunchConfiguration configuration, final //attrMgr.addAttributes(getResourceAttributes(configuration, mode)); // Collects attributes from Environment tab - String[] envArr = getEnvironmentToAppend(configuration); + String[] envArr = null; //org.eclipse.ptp.core.util.LaunchUtils.getEnvironmentToAppend(configuration); + if (this.fIsCygwin) { final StringBuilder sb = new StringBuilder(); sb.append(PATH_ENV).append('='); @@ -420,7 +424,7 @@ protected void updateAttributes(final ILaunchConfiguration configuration, final // Makes sure there is a queue, even if the resources tab doesn't require one to be specified. if (attrMgr.getAttribute(JobAttributes.getQueueIdAttributeDefinition()) == null) { - final IPQueue queue = getQueueDefault((IPResourceManager) this.fResourceManager.getAdapter(IPResourceManager.class)); + final IPQueue queue = org.eclipse.ptp.core.util.LaunchUtils.getQueueDefault((IPResourceManager) this.fResourceManager.getAdapter(IPResourceManager.class)); if (queue == null) { throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoRMQueueError)); } @@ -439,7 +443,7 @@ protected void updateAttributes(final ILaunchConfiguration configuration, final // Collects attributes from Arguments tab attrMgr.addAttribute(JobAttributes.getWorkingDirectoryAttributeDefinition().create(this.fWorkspaceDir)); - final String[] argArr = getProgramArguments(configuration); + final String[] argArr = org.eclipse.ptp.core.util.LaunchUtils.getProgramArguments(configuration); if (argArr != null) { attrMgr.addAttribute(JobAttributes.getProgramArgumentsAttributeDefinition().create(argArr)); } @@ -596,8 +600,8 @@ private IFileStore getMainCppFileStore(final String x10MainType, final String wo } private IResourceManager getResourceManagerControl(final ILaunchConfiguration configuration) throws CoreException { - final IPUniverse universe = PTPCorePlugin.getDefault().getModelManager().getUniverse(); - final String rmUniqueName = getResourceManagerUniqueName(configuration); + final IPUniverse universe = ModelManager.getInstance().getUniverse(); + final String rmUniqueName = org.eclipse.ptp.core.util.LaunchUtils.getResourceManagerUniqueName(configuration); for (final IPResourceManager pResourceManager : universe.getResourceManagers()) { IResourceManager resourceManager = (IResourceManager) pResourceManager.getAdapter(IResourceManager.class); if (resourceManager.getUniqueName().equals(rmUniqueName)) { diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java index 51a3fc3c..358279d4 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java @@ -28,9 +28,11 @@ import org.eclipse.imp.preferences.PreferencesService; import org.eclipse.imp.utils.Pair; import org.eclipse.jdt.launching.IJavaLaunchConfigurationConstants; +import org.eclipse.ptp.core.IPTPLaunchConfigurationConstants; import org.eclipse.ptp.core.elements.IPResourceManager; import org.eclipse.ptp.core.elements.IPMachine; import org.eclipse.ptp.core.elements.IPNode; +import org.eclipse.ptp.core.util.LaunchUtils; import org.eclipse.ptp.rmsystem.IResourceManager; import org.eclipse.ptp.rm.mpi.mpich2.core.launch.MPICH2LaunchConfiguration; import org.eclipse.ptp.rm.mpi.openmpi.core.launch.OpenMPILaunchConfiguration; @@ -74,7 +76,8 @@ protected void setLaunchConfigurationAttributes(final ILaunchConfigurationWorkin workingCopy.setAttribute(ATTR_MAIN_TYPE_NAME, typeInfo.getName()); workingCopy.setAttribute(ATTR_CONSOLE, true); workingCopy.setAttribute(ATTR_X10_MAIN_CLASS, typeInfo.getName()); - + //MV: Making the assumption that the platform conf name is the same as the project name. + workingCopy.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, PTPConfUtils.findResourceManagerByName(typeInfo.getCompilationUnit().getProject().getName()).getUniqueName() ); final IProject project = typeInfo.getCompilationUnit().getProject().getRawProject(); final IX10PlatformConf platformConf = CppLaunchCore.getInstance().getPlatformConfiguration(project); @@ -100,7 +103,7 @@ protected void setLaunchConfigurationAttributes(final ILaunchConfigurationWorkin } if ((useHostListAttrKey != null) && (hostListAttrKey != null)) { - final IResourceManager resourceManager = PTPConfUtils.findResourceManagerById(platformConf); + final IResourceManager resourceManager = PTPConfUtils.findResourceManagerByName(platformConf.getName()); if (resourceManager != null) { final StringBuilder hostListBuilder = new StringBuilder(); int i = 0; diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ConnectionConfiguration.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ConnectionConfiguration.java index 7d6d6885..8ded9093 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ConnectionConfiguration.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ConnectionConfiguration.java @@ -14,6 +14,7 @@ import org.eclipse.core.runtime.CoreException; import org.eclipse.core.runtime.NullProgressMonitor; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.remote.core.IRemoteConnection; import org.eclipse.ptp.remote.core.IRemoteConnectionManager; @@ -170,7 +171,7 @@ public String toString() { } ConnectionConfiguration(final IResourceManagerConfiguration rmConf) { - IResourceManager rm = PTPCorePlugin.getDefault().getModelManager().getResourceManagerFromUniqueName(rmConf.getUniqueName()); + IResourceManager rm = ModelManager.getInstance().getResourceManagerFromUniqueName(rmConf.getUniqueName()); final IRemoteServices remoteServices = PTPRemoteCorePlugin.getDefault().getRemoteServices(rm.getControlConfiguration().getRemoteServicesId()); final IRemoteConnection rmConn = remoteServices.getConnectionManager().getConnection(rm.getControlConfiguration().getConnectionName()); this.fIsLocal = PTPConstants.LOCAL_CONN_SERVICE_ID.equals(remoteServices.getId()); diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/LoadLevelerConf.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/LoadLevelerConf.java index 58fa805d..0f90f0f4 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/LoadLevelerConf.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/LoadLevelerConf.java @@ -12,7 +12,7 @@ import java.util.Arrays; -import org.eclipse.ptp.remote.core.IRemoteProxyOptions; +import org.eclipse.ptp.rm.core.proxy.IRemoteProxyOptions; import org.eclipse.ptp.rm.ibm.ll.core.rmsystem.IIBMLLResourceManagerConfiguration; import org.eclipse.ptp.rmsystem.IResourceManagerConfiguration; import org.eclipse.ptp.services.core.IServiceProvider; diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/MessagePassingInterfaceConf.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/MessagePassingInterfaceConf.java index be5cfaf3..5989993e 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/MessagePassingInterfaceConf.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/MessagePassingInterfaceConf.java @@ -9,6 +9,7 @@ import java.util.Arrays; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.rm.core.rmsystem.IToolRMConfiguration; import org.eclipse.ptp.rmsystem.IResourceManager; @@ -160,7 +161,7 @@ public String toString() { } MessagePassingInterfaceConf(final IToolRMConfiguration rmConf) { - super(PTPCorePlugin.getDefault().getModelManager().getResourceManagerFromUniqueName(rmConf.getName()).getConfiguration()); //MV - check that rmConf.getName() is a unique name for the resource manager + super(ModelManager.getInstance().getResourceManagerFromUniqueName(rmConf.getName()).getConfiguration()); //MV - check that rmConf.getName() is a unique name for the resource manager //super(rmConf); this.fDefaultToolCmds = rmConf.getUseToolDefaults(); this.fLaunchCmd = rmConf.getLaunchCmd(); diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ParallelEnvironmentConf.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ParallelEnvironmentConf.java index ebee5cde..72f377e5 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ParallelEnvironmentConf.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/platform_conf/ParallelEnvironmentConf.java @@ -7,7 +7,7 @@ *******************************************************************************/ package x10dt.ui.launch.cpp.platform_conf; -import org.eclipse.ptp.remote.core.IRemoteProxyOptions; +import org.eclipse.ptp.rm.core.proxy.IRemoteProxyOptions; import org.eclipse.ptp.rm.ibm.pe.core.PEPreferenceConstants; import org.eclipse.ptp.rm.ibm.pe.core.rmsystem.PEResourceManagerConfiguration; import org.eclipse.ptp.rmsystem.IResourceManagerConfiguration; diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/utils/PTPConfUtils.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/utils/PTPConfUtils.java index 46e15599..da5726da 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/utils/PTPConfUtils.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/utils/PTPConfUtils.java @@ -46,7 +46,7 @@ import org.eclipse.ptp.core.listeners.IResourceManagerListener; import org.eclipse.ptp.remote.core.IRemoteConnection; import org.eclipse.ptp.remote.core.IRemoteConnectionManager; -import org.eclipse.ptp.remote.core.IRemoteProxyOptions; +import org.eclipse.ptp.rm.core.proxy.IRemoteProxyOptions; import org.eclipse.ptp.remote.core.IRemoteServices; import org.eclipse.ptp.remote.core.PTPRemoteCorePlugin; import org.eclipse.ptp.remote.core.exception.RemoteConnectionException; @@ -228,7 +228,7 @@ public static String getCommunicationInterfaceTypeName(final ICommunicationInter * @return A non-null resource manager if we found a match, null otherwise. */ public static IResourceManager findResourceManagerByName(final String platformConfName) { - final IPUniverse universe = (IPUniverse) PTPCorePlugin.getDefault().getModelManager().getUniverse(); + final IPUniverse universe = (IPUniverse) ModelManager.getInstance().getUniverse(); for (final IPResourceManager pResourceManager : universe.getResourceManagers()) { final IResourceManager resourceManager = (IResourceManager) pResourceManager.getAdapter(IResourceManager.class); @@ -248,8 +248,8 @@ public static IResourceManager findResourceManagerByName(final String platformCo * @param platformConf The platform configuration to consider. * @return A non-null resource manager if we found a match, null otherwise. */ - public static IResourceManager findResourceManagerById(final IX10PlatformConf platformConf) { - final IPUniverse universe = (IPUniverse) PTPCorePlugin.getDefault().getModelManager().getUniverse(); + /*public static IResourceManager findResourceManagerById(final IX10PlatformConf platformConf) { + final IPUniverse universe = (IPUniverse) ModelManager.getInstance().getUniverse(); for (final IPResourceManager pResourceManager : universe.getResourceManagers()) { final IResourceManager resourceManager = (IResourceManager) pResourceManager.getAdapter(IResourceManager.class); @@ -260,7 +260,7 @@ public static IResourceManager findResourceManagerById(final IX10PlatformConf pl } return null; - } + }*/ /** * Returns an existing resource manager that is equivalent to information encapsulated in the platform configuration or @@ -276,7 +276,7 @@ public static IResourceManager findResourceManagerById(final IX10PlatformConf pl */ public static IResourceManager getResourceManager(final IX10PlatformConf platformConf) throws RemoteConnectionException, CoreException { - final IPUniverse universe = (IPUniverse) PTPCorePlugin.getDefault().getModelManager().getUniverse(); + final IPUniverse universe = (IPUniverse) ModelManager.getInstance().getUniverse(); for (final IPResourceManager pResourceManager : universe.getResourceManagers()) { final IResourceManager resourceManager = (IResourceManager) pResourceManager.getAdapter(IResourceManager.class); @@ -320,7 +320,7 @@ public static IResourceManager getResourceManager(final IX10PlatformConf platfor * @return A non-null remote connection instance if one has been found, null otherwise. */ public static IRemoteConnection findRemoteConnection(final IConnectionConf connectionConf) { - final IPUniverse universe = (IPUniverse) PTPCorePlugin.getDefault().getModelManager().getUniverse(); + final IPUniverse universe = (IPUniverse) ModelManager.getInstance().getUniverse(); final PTPRemoteCorePlugin plugin = PTPRemoteCorePlugin.getDefault(); final String rmServicesId = connectionConf.isLocal() ? LOCAL_CONN_SERVICE_ID : REMOTE_CONN_SERVICE_ID; From 59f663c1f96add91bd2cd694dc29230312b6d0ed Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Tue, 25 Sep 2012 20:55:13 +0000 Subject: [PATCH 03/10] X10DT - PTP6 - initial commit git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24646 a326200e-df2c-42e0-81fa-597e909af41d --- .../src/x10dt/ui/launch/java/Activator.java | 3 +- .../java/launching/PlacesAndHostsTab.java | 8 +-- .../java/launching/ResourceManagerHelper.java | 2 +- .../launch/java/launching/VMsLocationTab.java | 2 +- .../X10LaunchConfigurationDelegate.java | 57 ++++++++++--------- 5 files changed, 37 insertions(+), 35 deletions(-) diff --git a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/Activator.java b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/Activator.java index f0f3fd0e..bba89dc3 100644 --- a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/Activator.java +++ b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/Activator.java @@ -11,6 +11,7 @@ import org.eclipse.debug.core.ILaunchConfigurationListener; import org.eclipse.debug.core.ILaunchManager; import org.eclipse.imp.preferences.IPreferencesService; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.core.elements.IPResourceManager; import org.eclipse.ptp.core.elements.IPUniverse; @@ -84,7 +85,7 @@ public void launchConfigurationRemoved(final ILaunchConfiguration configuration) if (configuration.getName().equals(serviceConfiguration.getName())) { final IServiceProvider serviceProvider = serviceConfiguration.getServiceProvider(service); //if (serviceProvider instanceof MultiVMResourceManagerConfiguration) { //MV - Check this - final IPUniverse universe = (IPUniverse) PTPCorePlugin.getDefault().getModelManager().getUniverse(); + final IPUniverse universe = (IPUniverse) ModelManager.getInstance().getUniverse(); for (final IPResourceManager pResourceManager : universe.getResourceManagers()) { final IResourceManager resourceManager = (IResourceManager) pResourceManager.getAdapter(IResourceManager.class); final IResourceManagerConfiguration rmConf = resourceManager.getConfiguration(); diff --git a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/PlacesAndHostsTab.java b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/PlacesAndHostsTab.java index 3592ac03..4b97a36f 100644 --- a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/PlacesAndHostsTab.java +++ b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/PlacesAndHostsTab.java @@ -57,11 +57,11 @@ public void setDefaults(final ILaunchConfigurationWorkingCopy configuration) { } public void initializeFrom(final ILaunchConfiguration configuration) { - this.fX10EnvVarsConfTab.initializeFrom(null /* control */, this.fResourceManager, null /* queue */, configuration); + this.fX10EnvVarsConfTab.initializeFrom(configuration); } public void performApply(final ILaunchConfigurationWorkingCopy configuration) { - this.fX10EnvVarsConfTab.performApply(configuration, this.fResourceManager, null /* queue */); + this.fX10EnvVarsConfTab.performApply(configuration); } public String getName() { @@ -80,7 +80,7 @@ public Image getImage() { } public boolean isValid(final ILaunchConfiguration configuration) { - final RMLaunchValidation validation = this.fX10EnvVarsConfTab.isValid(configuration, this.fResourceManager, null); + final RMLaunchValidation validation = this.fX10EnvVarsConfTab.isValid(configuration); setErrorMessage(null); if (validation.isSuccess()) { return true; @@ -95,7 +95,7 @@ public boolean isValid(final ILaunchConfiguration configuration) { private void createX10EnvVarsInfo(final Composite parent) { this.fX10EnvVarsConfTab = new X10PlacesAndHostsDynamicTab(); try { - this.fX10EnvVarsConfTab.createControl(parent, null /* resourceManager */, null /* queue */); + this.fX10EnvVarsConfTab.createControl(parent, null /* control id */); //MV -- REVISE HERE this.fX10EnvVarsConfTab.addContentsChangedListener(new IRMLaunchConfigurationContentsChangedListener() { public void handleContentsChanged(final IRMLaunchConfigurationDynamicTab factory) { diff --git a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/ResourceManagerHelper.java b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/ResourceManagerHelper.java index 2d04eb53..b1c31c78 100644 --- a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/ResourceManagerHelper.java +++ b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/ResourceManagerHelper.java @@ -33,7 +33,7 @@ import org.eclipse.ptp.core.events.IResourceManagerRemovedEvent; import org.eclipse.ptp.core.listeners.IResourceManagerListener; import org.eclipse.ptp.remote.core.IRemoteConnectionManager; -import org.eclipse.ptp.remote.core.IRemoteProxyOptions; +import org.eclipse.ptp.rm.core.proxy.IRemoteProxyOptions; import org.eclipse.ptp.remote.core.PTPRemoteCorePlugin; import org.eclipse.ptp.remote.core.exception.RemoteConnectionException; import org.eclipse.ptp.remote.remotetools.core.RemoteToolsServices; diff --git a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/VMsLocationTab.java b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/VMsLocationTab.java index 69adb758..fe9e940b 100644 --- a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/VMsLocationTab.java +++ b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/VMsLocationTab.java @@ -45,7 +45,7 @@ import org.eclipse.ptp.core.elements.attributes.ResourceManagerAttributes.State; import org.eclipse.ptp.remote.core.IRemoteConnection; import org.eclipse.ptp.remote.core.IRemoteConnectionManager; -import org.eclipse.ptp.remote.core.IRemoteProxyOptions; +import org.eclipse.ptp.rm.core.proxy.IRemoteProxyOptions; import org.eclipse.ptp.remote.core.IRemoteServices; import org.eclipse.ptp.remote.core.PTPRemoteCorePlugin; import org.eclipse.ptp.remote.core.exception.RemoteConnectionException; diff --git a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/X10LaunchConfigurationDelegate.java b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/X10LaunchConfigurationDelegate.java index 24cd6707..054ad276 100644 --- a/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/X10LaunchConfigurationDelegate.java +++ b/x10dt.ui.launch.java/src/x10dt/ui/launch/java/launching/X10LaunchConfigurationDelegate.java @@ -39,6 +39,7 @@ import org.eclipse.jdt.launching.IJavaLaunchConfigurationConstants; import org.eclipse.osgi.util.NLS; import org.eclipse.ptp.core.IPTPLaunchConfigurationConstants; +import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.core.attributes.AttributeManager; import org.eclipse.ptp.core.attributes.StringAttribute; @@ -102,9 +103,9 @@ private void updateAttributes(final ILaunchConfiguration configuration, final St final IProgressMonitor monitor) throws CoreException { final SubMonitor progress = SubMonitor.convert(monitor, 30); try { - final String rmUniqueName = getResourceManagerUniqueName(configuration); + final String rmUniqueName = org.eclipse.ptp.core.util.LaunchUtils.getResourceManagerUniqueName(configuration); IResourceManager resourceManager = null; - for (final IPResourceManager prm : PTPCorePlugin.getDefault().getModelManager().getUniverse().getResourceManagers()) { + for (final IPResourceManager prm : ModelManager.getInstance().getUniverse().getResourceManagers()) { IResourceManager rm= (IResourceManager) prm.getAdapter(IResourceManager.class); if (rm.getUniqueName().equals(rmUniqueName)) { resourceManager = rm; @@ -125,7 +126,7 @@ private void updateAttributes(final ILaunchConfiguration configuration, final St // Make sure there is a queue if (attrMgr.getAttribute(JobAttributes.getQueueIdAttributeDefinition()) == null) { - IPQueue queue = getQueueDefault((IPResourceManager)resourceManager.getAdapter(IPResourceManager.class)); + IPQueue queue = org.eclipse.ptp.core.util.LaunchUtils.getQueueDefault((IPResourceManager)resourceManager.getAdapter(IPResourceManager.class)); if (queue == null) { throw new CoreException(new Status(IStatus.ERROR, PTPLaunchPlugin.getUniqueIdentifier(), Messages.XLCD_NoDefaultQueue)); @@ -194,31 +195,31 @@ protected IPath verifyExecutablePath(final ILaunchConfiguration configuration, } } - protected IPath verifyResource(final String path, final ILaunchConfiguration configuration, - final IProgressMonitor monitor) throws CoreException { - final IResourceManagerConfiguration conf = this.fResourceManager.getConfiguration(); - final IRemoteServices remoteServices = PTPRemoteCorePlugin.getDefault().getRemoteServices(this.fResourceManager.getControlConfiguration().getRemoteServicesId()); - if (remoteServices == null) { - throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoRemoteServices)); - } - final IRemoteConnectionManager connMgr = remoteServices.getConnectionManager(); - if (connMgr == null) { - throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoConnectionMgr)); - } - final IRemoteConnection conn = connMgr.getConnection(this.fResourceManager.getControlConfiguration().getConnectionName()); - if (conn == null) { - throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoConnection)); - } - final IRemoteFileManager fileManager = remoteServices.getFileManager(conn); - if (fileManager == null) { - throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoFileManager)); - } - if (!fileManager.getResource(path).fetchInfo().exists()) { - throw new CoreException(new Status(IStatus.INFO, CppLaunchCore.PLUGIN_ID, NLS.bind(LaunchMessages.CLCD_PathNotFound, - path))); - } - return new Path(path); - } +// protected IPath verifyResource(final String path, final ILaunchConfiguration configuration, +// final IProgressMonitor monitor) throws CoreException { +// final IResourceManagerConfiguration conf = this.fResourceManager.getConfiguration(); +// final IRemoteServices remoteServices = PTPRemoteCorePlugin.getDefault().getRemoteServices(this.fResourceManager.getControlConfiguration().getRemoteServicesId()); +// if (remoteServices == null) { +// throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoRemoteServices)); +// } +// final IRemoteConnectionManager connMgr = remoteServices.getConnectionManager(); +// if (connMgr == null) { +// throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoConnectionMgr)); +// } +// final IRemoteConnection conn = connMgr.getConnection(this.fResourceManager.getControlConfiguration().getConnectionName()); +// if (conn == null) { +// throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoConnection)); +// } +// final IRemoteFileManager fileManager = remoteServices.getFileManager(conn); +// if (fileManager == null) { +// throw new CoreException(new Status(IStatus.ERROR, CppLaunchCore.PLUGIN_ID, LaunchMessages.CLCD_NoFileManager)); +// } +// if (!fileManager.getResource(path).fetchInfo().exists()) { +// throw new CoreException(new Status(IStatus.INFO, CppLaunchCore.PLUGIN_ID, NLS.bind(LaunchMessages.CLCD_PathNotFound, +// path))); +// } +// return new Path(path); +// } // --- Private code From 852f4e31f5dab44cad37515ab7b536f4a639c378 Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Tue, 25 Sep 2012 20:55:42 +0000 Subject: [PATCH 04/10] X10DT - PTP6 - initial commit git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24647 a326200e-df2c-42e0-81fa-597e909af41d --- ...andaloneRMLaunchConfigurationDynamicTab.java | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/x10dt.ui.launch.cpp.rms/src/x10dt/ui/launch/cpp/rms/launch_configuration/StandaloneRMLaunchConfigurationDynamicTab.java b/x10dt.ui.launch.cpp.rms/src/x10dt/ui/launch/cpp/rms/launch_configuration/StandaloneRMLaunchConfigurationDynamicTab.java index ff030317..8ae7b50f 100644 --- a/x10dt.ui.launch.cpp.rms/src/x10dt/ui/launch/cpp/rms/launch_configuration/StandaloneRMLaunchConfigurationDynamicTab.java +++ b/x10dt.ui.launch.cpp.rms/src/x10dt/ui/launch/cpp/rms/launch_configuration/StandaloneRMLaunchConfigurationDynamicTab.java @@ -53,12 +53,11 @@ public void addContentsChangedListener(final IRMLaunchConfigurationContentsChang this.fListeners.add(listener); } - public RMLaunchValidation canSave(final Control control, final IResourceManager resourceManager, final IPQueue queue) { + public RMLaunchValidation canSave(final Control control) { return new RMLaunchValidation(true, null); } - public void createControl(final Composite parent, final IResourceManager resourceManager, - final IPQueue queue) throws CoreException { + public void createControl(final Composite parent, String id) throws CoreException { this.fControl = new Composite(parent, SWT.NONE); this.fControl.setFont(parent.getFont()); this.fControl.setLayout(new GridLayout(1, false)); @@ -103,8 +102,7 @@ public Control getControl() { return this.fControl; } - public RMLaunchValidation initializeFrom(final Control control, final IResourceManager resourceManager, final IPQueue queue, - final ILaunchConfiguration configuration) { + public RMLaunchValidation initializeFrom(final ILaunchConfiguration configuration) { try { this.fNumPlacesSpinner.setSelection(configuration.getAttribute(ATTR_NUM_PLACES, DEFAULT_NUM_PLACES)); } catch (CoreException except) { @@ -113,16 +111,14 @@ public RMLaunchValidation initializeFrom(final Control control, final IResourceM return new RMLaunchValidation(true, null); } - public RMLaunchValidation isValid(final ILaunchConfiguration launchConfig, final IResourceManager resourceManager, - final IPQueue queue) { + public RMLaunchValidation isValid(final ILaunchConfiguration launchConfig) { if (this.fNumPlacesSpinner.getSelection() < 1) { return new RMLaunchValidation(false, Messages.SRMLCDT_AtLeastOnePlaceMsg); } return new RMLaunchValidation(true, null); } - public RMLaunchValidation performApply(final ILaunchConfigurationWorkingCopy configuration, - final IResourceManager resourceManager, final IPQueue queue) { + public RMLaunchValidation performApply(final ILaunchConfigurationWorkingCopy configuration) { configuration.setAttribute(ATTR_NUM_PLACES, this.fNumPlacesSpinner.getSelection()); return new RMLaunchValidation(true, null); } @@ -131,8 +127,7 @@ public void removeContentsChangedListener(final IRMLaunchConfigurationContentsCh this.fListeners.remove(listener); } - public RMLaunchValidation setDefaults(final ILaunchConfigurationWorkingCopy configuration, - final IResourceManager resourceManager, final IPQueue queue) { + public RMLaunchValidation setDefaults(final ILaunchConfigurationWorkingCopy configuration) { return new RMLaunchValidation(true, null); } From 7008c39b6b1b63a0c60fde7f4f030401f2d4d22b Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Tue, 25 Sep 2012 20:57:11 +0000 Subject: [PATCH 05/10] X10DT - PTP6 - initial commit git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24648 a326200e-df2c-42e0-81fa-597e909af41d --- .../X10PlacesAndHostsDynamicTab.java | 22 ++++++++----------- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/x10dt.ui.launch.rms.core/src/x10dt/ui/launch/rms/core/launch_configuration/X10PlacesAndHostsDynamicTab.java b/x10dt.ui.launch.rms.core/src/x10dt/ui/launch/rms/core/launch_configuration/X10PlacesAndHostsDynamicTab.java index 9d7627ea..70aec505 100644 --- a/x10dt.ui.launch.rms.core/src/x10dt/ui/launch/rms/core/launch_configuration/X10PlacesAndHostsDynamicTab.java +++ b/x10dt.ui.launch.rms.core/src/x10dt/ui/launch/rms/core/launch_configuration/X10PlacesAndHostsDynamicTab.java @@ -96,12 +96,11 @@ public void addContentsChangedListener(final IRMLaunchConfigurationContentsChang this.fListeners.add(listener); } - public RMLaunchValidation canSave(final Control control, final IResourceManager resourceManager, final IPQueue queue) { + public RMLaunchValidation canSave(final Control control) { return new RMLaunchValidation(true, null); } - public void createControl(final Composite parent, final IResourceManager resourceManager, - final IPQueue queue) throws CoreException { + public void createControl(final Composite parent, String id) throws CoreException { this.fControl = new Composite(parent, SWT.NONE); this.fControl.setFont(parent.getFont()); this.fControl.setLayout(new GridLayout(1, false)); @@ -352,11 +351,10 @@ public Control getControl() { } @SuppressWarnings("unchecked") - public RMLaunchValidation initializeFrom(final Control control, final IResourceManager resourceManager, final IPQueue queue, - final ILaunchConfiguration configuration) { + public RMLaunchValidation initializeFrom(final ILaunchConfiguration configuration) { final List hosts = new ArrayList(); - if (resourceManager != null) { - for (final IPMachine machine : ((IPResourceManager) resourceManager.getAdapter(IPResourceManager.class)).getMachines()) { + if (fResourceManager != null) { + for (final IPMachine machine : ((IPResourceManager) fResourceManager.getAdapter(IPResourceManager.class)).getMachines()) { for (final IPNode node : machine.getNodes()) { hosts.add(node.getName()); } @@ -378,8 +376,7 @@ public RMLaunchValidation initializeFrom(final Control control, final IResourceM return new RMLaunchValidation(true, null); } - public RMLaunchValidation isValid(final ILaunchConfiguration launchConfig, final IResourceManager resourceManager, - final IPQueue queue) { + public RMLaunchValidation isValid(final ILaunchConfiguration launchConfig){ if (this.fNumPlacesSpinner.getSelection() < 1) { return new RMLaunchValidation(false, Messages.SRMLCDT_AtLeastOnePlaceMsg); } @@ -400,8 +397,7 @@ public RMLaunchValidation isValid(final ILaunchConfiguration launchConfig, final return new RMLaunchValidation(true, null); } - public RMLaunchValidation performApply(final ILaunchConfigurationWorkingCopy configuration, - final IResourceManager resourceManager, final IPQueue queue) { + public RMLaunchValidation performApply(final ILaunchConfigurationWorkingCopy configuration) { if (this.fHostFileBt != null) configuration.setAttribute(ATTR_USE_HOSTFILE, this.fHostFileBt.getSelection()); if (this.fNumPlacesSpinner != null) @@ -409,6 +405,7 @@ public RMLaunchValidation performApply(final ILaunchConfigurationWorkingCopy con if (this.fHostFileText != null) configuration.setAttribute(ATTR_HOSTFILE, this.fHostFileText.getText().trim()); configuration.setAttribute(ATTR_HOSTLIST, (this.fHosts.isEmpty()) ? null : this.fHosts); + return new RMLaunchValidation(true, null); } @@ -416,8 +413,7 @@ public void removeContentsChangedListener(final IRMLaunchConfigurationContentsCh this.fListeners.remove(listener); } - public RMLaunchValidation setDefaults(final ILaunchConfigurationWorkingCopy configuration, - final IResourceManager resourceManager, final IPQueue queue) { + public RMLaunchValidation setDefaults(final ILaunchConfigurationWorkingCopy configuration) { return new RMLaunchValidation(true, null); } From 01e3a5b4201437a2ba530dc53d79f78e85f357d1 Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Wed, 26 Sep 2012 20:04:01 +0000 Subject: [PATCH 06/10] Bug fix (lost resource manager) for X10DT with PTP6. git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24673 a326200e-df2c-42e0-81fa-597e909af41d --- .../cpp/launching/CppLaunchConfigurationDelegate.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java index d483cf07..a501c32c 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java @@ -42,12 +42,14 @@ import org.eclipse.core.runtime.SubProgressMonitor; import org.eclipse.debug.core.ILaunch; import org.eclipse.debug.core.ILaunchConfiguration; +import org.eclipse.debug.core.ILaunchConfigurationWorkingCopy; import org.eclipse.debug.core.ILaunchManager; import org.eclipse.imp.preferences.PreferencesService; import org.eclipse.jdt.core.IJavaProject; import org.eclipse.jdt.core.JavaCore; import org.eclipse.jface.dialogs.ErrorDialog; import org.eclipse.osgi.util.NLS; +import org.eclipse.ptp.core.IPTPLaunchConfigurationConstants; import org.eclipse.ptp.core.ModelManager; import org.eclipse.ptp.core.PTPCorePlugin; import org.eclipse.ptp.core.attributes.ArrayAttribute; @@ -125,6 +127,7 @@ public void launch(final ILaunchConfiguration configuration, final String mode, monitor.beginTask(null, 10); monitor.subTask(LaunchMessages.CLCD_ExecCreationTaskName); + this.fProject = verifyProject(configuration); // The following check shouldn't be necessary, since this ILaunchConfigurationDelegate implementation @@ -139,6 +142,11 @@ public void launch(final ILaunchConfiguration configuration, final String mode, if (!monitor.isCanceled() && shouldProcessToLinkStep(fProject) && createExecutable(configuration, fProject, new SubProgressMonitor(monitor, 5)) == 0) { + + ILaunchConfigurationWorkingCopy workingCopy = configuration.getWorkingCopy(); + workingCopy.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, fResourceManager.getUniqueName()); + workingCopy.doSave(); + // Then, performs the launch. if (!monitor.isCanceled()) { // Hijack launch if it's PAMI with LoadLeveler From d1b134a6293c85f2dbdc7d76cbefa874cdce456e Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Wed, 26 Sep 2012 21:34:01 +0000 Subject: [PATCH 07/10] Code clean up (X10DT + PTP6) git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24674 a326200e-df2c-42e0-81fa-597e909af41d --- .../src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java | 4 +--- .../x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java | 2 -- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java index 4a412e28..e453abc0 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppApplicationTab.java @@ -41,6 +41,7 @@ import org.eclipse.ptp.core.IPTPLaunchConfigurationConstants; import org.eclipse.ptp.launch.ui.LaunchConfigurationTab; import org.eclipse.ptp.launch.ui.LaunchImages; +import org.eclipse.ptp.remote.core.exception.RemoteConnectionException; import org.eclipse.swt.SWT; import org.eclipse.swt.events.ModifyEvent; import org.eclipse.swt.events.ModifyListener; @@ -128,7 +129,6 @@ public void performApply(final ILaunchConfigurationWorkingCopy configuration) { configuration.setAttribute(org.eclipse.jdt.launching.IJavaLaunchConfigurationConstants.ATTR_MAIN_TYPE_NAME, this.fMainTypeText.getText().trim()); configuration.setAttribute(ATTR_X10_MAIN_CLASS, this.fMainTypeText.getText().trim()); - configuration.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, PTPConfUtils.findResourceManagerByName(fX10PlatformConf.getName()).getUniqueName()); final String content = this.fPgrmArgsText.getText().trim(); configuration.setAttribute(ATTR_ARGUMENTS, (content.length() > 0) ? content : null); @@ -138,8 +138,6 @@ public void performApply(final ILaunchConfigurationWorkingCopy configuration) { } public void setDefaults(final ILaunchConfigurationWorkingCopy configuration) { - configuration.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, PTPConfUtils.findResourceManagerByName(fX10PlatformConf.getName()).getUniqueName()); - final IResource context = getCurrentSelectionContext(); if (context == null) { configuration.setAttribute(ATTR_PROJECT_NAME, (String) null); diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java index 358279d4..b1313687 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/X10CppLaunchShortcut.java @@ -76,8 +76,6 @@ protected void setLaunchConfigurationAttributes(final ILaunchConfigurationWorkin workingCopy.setAttribute(ATTR_MAIN_TYPE_NAME, typeInfo.getName()); workingCopy.setAttribute(ATTR_CONSOLE, true); workingCopy.setAttribute(ATTR_X10_MAIN_CLASS, typeInfo.getName()); - //MV: Making the assumption that the platform conf name is the same as the project name. - workingCopy.setAttribute(IPTPLaunchConfigurationConstants.ATTR_RESOURCE_MANAGER_UNIQUENAME, PTPConfUtils.findResourceManagerByName(typeInfo.getCompilationUnit().getProject().getName()).getUniqueName() ); final IProject project = typeInfo.getCompilationUnit().getProject().getRawProject(); final IX10PlatformConf platformConf = CppLaunchCore.getInstance().getPlatformConfiguration(project); From 1c27dc3622d2423bc4f77b5520e330451c54a673 Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Thu, 18 Oct 2012 15:20:19 +0000 Subject: [PATCH 08/10] Changes to make DT more independent wrt PTP - removed PTP output view git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24743 a326200e-df2c-42e0-81fa-597e909af41d --- x10dt.ui.launch.cpp/icons/obj/x10_application.gif | Bin 0 -> 576 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 x10dt.ui.launch.cpp/icons/obj/x10_application.gif diff --git a/x10dt.ui.launch.cpp/icons/obj/x10_application.gif b/x10dt.ui.launch.cpp/icons/obj/x10_application.gif new file mode 100644 index 0000000000000000000000000000000000000000..30fd706fb3c4b6e9456fc992409c536f6fe076f7 GIT binary patch literal 576 zcmZ?wbhEHb6krfwIA+iA>(~F^zyJUG{r|_W|KET8&+XfrKVg4F!1$rWorU!HiXn|3aQ%|)4VmlZF^exuJiXlUwQQ9%g_G~g)7`j zS9_GN@v2zsTeVI*ar*H)pFaNhzvc4#eK$V5`~E*>*4fwJ{(t!Kf6CF9Z5uDmJN0_i z`8RLB|1a3`sC@h5hTTs)4m>|`@9oQP|6hOq|LXhys~`S1*B9)V9kFFb)cR?$D<;J+ z>Q9>8oi?RCv$r|t|Ii1DKUo;L7&IAl7=Qp2Ck*U!8#J4mTUy)NTcu@WrpeZn rh2_o>dFKDNEI(H$IvnViH)N9GIdGv}L0UlBYf8t8MXt_`3=Gx)z65&$ literal 0 HcmV?d00001 From fdaff9f766590c81d0c303244884fadae8cc5373 Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Thu, 18 Oct 2012 15:21:07 +0000 Subject: [PATCH 09/10] Changes to make DT more independent wrt PTP - removed PTP output view git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24744 a326200e-df2c-42e0-81fa-597e909af41d --- x10dt.ui.launch.cpp/plugin.xml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/x10dt.ui.launch.cpp/plugin.xml b/x10dt.ui.launch.cpp/plugin.xml index 46110a6a..9401c8f6 100644 --- a/x10dt.ui.launch.cpp/plugin.xml +++ b/x10dt.ui.launch.cpp/plugin.xml @@ -99,9 +99,9 @@ id="x10dt.ui.cpp.launch.X10LaunchConfigurationTabGroup" type="x10dt.ui.cpp.launch.X10LaunchConfigurationType"> + perspective="x10dt.ui.perspective"> @@ -109,7 +109,7 @@ point="org.eclipse.debug.ui.launchConfigurationTypeImages"> @@ -163,7 +163,7 @@ point="org.eclipse.debug.ui.launchShortcuts"> From de75221a235a3c379029ebcd364d1432bb189952 Mon Sep 17 00:00:00 2001 From: Mandana Vaziri Date: Thu, 18 Oct 2012 15:24:52 +0000 Subject: [PATCH 10/10] Changes to make DT more independent wrt PTP: run launch command remotely without relying on PTP launch git-svn-id: svn://svn.code.sf.net/p/x10/code/x10dt/branches/X10DT-PTP6@24745 a326200e-df2c-42e0-81fa-597e909af41d --- .../CppLaunchConfigurationDelegate.java | 37 ++++++++++++++++++- 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java index a501c32c..aa30a4c6 100644 --- a/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java +++ b/x10dt.ui.launch.cpp/src/x10dt/ui/launch/cpp/launching/CppLaunchConfigurationDelegate.java @@ -157,7 +157,8 @@ public void launch(final ILaunchConfiguration configuration, final String mode, } else { monitor.subTask(LaunchMessages.CLCD_LaunchCreationTaskName); updateAttributes(configuration, mode, monitor); - super.launch(configuration, mode, launch, new SubProgressMonitor(monitor, 5)); + //super.launch(configuration, mode, launch, new SubProgressMonitor(monitor, 5)); + runMyCommand(monitor); } } } @@ -166,6 +167,40 @@ public void launch(final ILaunchConfiguration configuration, final String mode, } } + private void runMyCommand(final IProgressMonitor monitor) { + try { + final MessageConsole messageConsole = UIUtils.findOrCreateX10Console(); + final MessageConsoleStream mcStream = messageConsole.newMessageStream(); + messageConsole.clearConsole(); + final String cmd = this.fExecPath; + final List command = new ArrayList(); + command.add(cmd); + final int returnCode = this.fTargetOpHelper.run(command, this.fWorkspaceDir, new IProcessOuputListener() { + + public void read(final String line) { + mcStream.println(line); + } + + public void readError(final String line) { + if (this.fCounter == 0) { + mcStream.println(NLS.bind(LaunchMessages.CLCD_CmdUsedMsg, cmd)); + this.fCounter = 1; + } + mcStream.println(line); + } + + // --- Fields + + int fCounter; + + }, monitor); + mcStream.flush(); + } catch (InterruptedException e){ + //TODO + } catch (IOException e){ + //TODO + } + } private void launchPAMIwLL(final ILaunchConfiguration configuration, final IProject project, final IProgressMonitor monitor) throws CoreException{