[Esbox-commits] r944 - in trunk: org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/api/maemosdk/core/tools org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/maemosdk/core/adapters org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests org.maemo.esbox.scratchbox.core/src/org/maemo/esbox/internal/api/scratchbox/core/sdk org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/core org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/ui/adapters

raul at garage.maemo.org raul at garage.maemo.org
Mon Nov 17 23:07:21 EET 2008


Author: raul
Date: 2008-11-17 23:07:20 +0200 (Mon, 17 Nov 2008)
New Revision: 944

Modified:
   trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/api/maemosdk/core/tools/MaemoLauncher.java
   trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/maemosdk/core/adapters/MaemoTargetEnvironmentModifierAdapter.java
   trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestOProfileValgrindSupport.java
   trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestProcessLauncher.java
   trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB1SDKProvider.java
   trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB2SDKProvider.java
   trunk/org.maemo.esbox.scratchbox.core/src/org/maemo/esbox/internal/api/scratchbox/core/sdk/AbstractScratchboxSDKTarget.java
   trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1EnvironmentProvider.java
   trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1ProcessLauncher.java
   trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/core/Scratchbox2SDK.java
   trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2EnvironmentProvider.java
   trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2ProcessLauncher.java
   trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/ui/adapters/MaemoScratchbox2SDKManageTargetAdapter.java
Log:
Merged with changes sent by Ed as a patch. These changes provide a machine for ISDKTarget (ISDKTarget#getMachine()). Those places where ISDK#getMachine() were changed to use ISDKTarget#getMachine() instead.

Modified: trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/api/maemosdk/core/tools/MaemoLauncher.java
===================================================================
--- trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/api/maemosdk/core/tools/MaemoLauncher.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/api/maemosdk/core/tools/MaemoLauncher.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -62,7 +62,7 @@
 			IProcessLauncherFactory processLauncherFactory = sdkTarget.getProcessLauncherFactory();
 			
 			IEnvironmentModifierBlock envBlock = processLauncherFactory.defaultEnvironmentModifierBlock();
-			MaemoEnvironmentUtils.defineEmulatorXDisplayVariable(sdkTarget.getSDK().getMachine(), envBlock);
+			MaemoEnvironmentUtils.defineEmulatorXDisplayVariable(sdkTarget.getMachine(), envBlock);
 			
 			IProcessLauncher processLauncher = ProcessLauncherCreator.createProcessLauncher(processLauncherFactory, null,
 					Collections.singletonList(maemoCommand), envBlock);
@@ -172,8 +172,7 @@
 	 */
 	private boolean testMaemoLauncher(ISDKTarget sdkTarget)
 			throws MicaException {
-		List<IProcess> runningProcesses = getMaemoLaunchers(sdkTarget.getSDK()
-				.getMachine());
+		List<IProcess> runningProcesses = getMaemoLaunchers(sdkTarget.getMachine());
 		return runningProcesses.size() > 0;
 	}
 
@@ -248,7 +247,7 @@
 	 * @throws MicaException
 	 */
 	public void killMaemo(ISDKTarget sdkTarget) throws MicaException {
-		List<IProcess> pids = getMaemoLaunchers(sdkTarget.getSDK().getMachine());
+		List<IProcess> pids = getMaemoLaunchers(sdkTarget.getMachine());
 		if (pids.size() == 0)
 			return;
 
@@ -261,7 +260,7 @@
 		}
 
 		// try again
-		pids = getMaemoLaunchers(sdkTarget.getSDK().getMachine());
+		pids = getMaemoLaunchers(sdkTarget.getMachine());
 		if (pids.size() == 0)
 			return;
 

Modified: trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/maemosdk/core/adapters/MaemoTargetEnvironmentModifierAdapter.java
===================================================================
--- trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/maemosdk/core/adapters/MaemoTargetEnvironmentModifierAdapter.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.maemosdk.core/src/org/maemo/esbox/internal/maemosdk/core/adapters/MaemoTargetEnvironmentModifierAdapter.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -45,7 +45,7 @@
 		envBlock.define("DBUS_SESSION_BUS_ADDRESS", "unix:path=/tmp/session_bus_socket");
 		
 		if (launchProxy instanceof AbstractLocalLaunchProxy) {
-			MaemoEnvironmentUtils.defineEmulatorXDisplayVariable(sdkTarget.getSDK().getMachine(), envBlock);
+			MaemoEnvironmentUtils.defineEmulatorXDisplayVariable(sdkTarget.getMachine(), envBlock);
 			
 		} else {
 			// remote launches need DISPLAY for its own machine

Modified: trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestOProfileValgrindSupport.java
===================================================================
--- trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestOProfileValgrindSupport.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestOProfileValgrindSupport.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -43,7 +43,7 @@
 				assertNotNull(sdkTarget.getName(), oprofileSupport);
 				assertNotNull(sdkTarget.getName(), valgrindSupportAdapter);
 				
-				IMachine machine = sdkTarget.getSDK().getMachine();
+				IMachine machine = sdkTarget.getMachine();
 				if (sdkTarget.getArchitecture().equals(IScratchboxSDKTarget.ARCHITECTURE_ARMEL)) {
 					armelTarget = sdkTarget;
 					assertNotNull(oprofileSupport.validateSupport(machine));	// not a device

Modified: trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestProcessLauncher.java
===================================================================
--- trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestProcessLauncher.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestProcessLauncher.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -71,7 +71,7 @@
 						|| target.getName().contains("REMOVE")
 						|| target.getName().contains("bora"))
 					continue;
-				if (!validateMachine(target.getSDK().getMachine()))
+				if (!validateMachine(target.getMachine()))
 					continue;
 				if (--count < 0)
 					break;
@@ -232,7 +232,7 @@
 	private IPath createStockScript(ISDKTarget target, String cmdText)
 			throws Exception {
 		
-		IPath scriptPath = target.getSDK().getMachine().getUserHome().append(".tmp").append("foo.sh");
+		IPath scriptPath = target.getMachine().getUserHome().append(".tmp").append("foo.sh");
 		IPath targetPath = target.getMachineToSDKTargetFileSystemMapping().convertTargetToHostPath(scriptPath);
 		target.getTargetFileSystemAccess().getFileStore(targetPath.removeLastSegments(1)).mkdir(0, null);
 		
@@ -430,7 +430,7 @@
 				// make an overridden environment that has the script directory in its PATH
 				String scriptDir = scriptPath.removeLastSegments(1).toPortableString();
 				IEnvironmentModifierBlock envVarBlock =
-					ProcessLauncherUtils.addToPATH(target.getSDK().getMachine(),
+					ProcessLauncherUtils.addToPATH(target.getMachine(),
 							target.getProcessLauncherFactory(), 
 							scriptDir, true);
 				

Modified: trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB1SDKProvider.java
===================================================================
--- trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB1SDKProvider.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB1SDKProvider.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -146,7 +146,7 @@
 				assertTrue(target + " @" + sbPath.toString(), store.fetchInfo().exists());
 				
 				// if we happen to be running locally, verify sanity
-				if (target.getSDK().getMachine() instanceof ILocalMachine) {
+				if (target.getMachine() instanceof ILocalMachine) {
 					try {
 						hostPath = target.convertTargetToHostPath(sbPath);
 						assertTrue(hostPath.toFile().toString(),

Modified: trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB2SDKProvider.java
===================================================================
--- trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB2SDKProvider.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.maemosdk.tests/src/org/maemo/esbox/scratchbox/tests/TestSB2SDKProvider.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -120,7 +120,7 @@
 		
 		// these tests only work on machine
 		IPath hostPath = new Path("/");
-		if (target.getSDK().getMachine() instanceof ILocalMachine) {
+		if (target.getMachine() instanceof ILocalMachine) {
 			try {
 				sbPath = target.convertHostToTargetPath(hostPath);
 			} catch (MicaException e) {

Modified: trunk/org.maemo.esbox.scratchbox.core/src/org/maemo/esbox/internal/api/scratchbox/core/sdk/AbstractScratchboxSDKTarget.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.core/src/org/maemo/esbox/internal/api/scratchbox/core/sdk/AbstractScratchboxSDKTarget.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.core/src/org/maemo/esbox/internal/api/scratchbox/core/sdk/AbstractScratchboxSDKTarget.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -50,7 +50,7 @@
 	 */
 	public ICommandAbstractor getCommandAbstractor() throws MicaException {
 		// ensure machine is alive
-		MachineUtils.acquireMachine(this.getSDK().getMachine());
+		MachineUtils.acquireMachine(this.getMachine());
 
 		return new ICommandAbstractor() {
 

Modified: trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1EnvironmentProvider.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1EnvironmentProvider.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1EnvironmentProvider.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -65,8 +65,7 @@
 						.logError(
 								"Failed to read scratchbox environment; using system environment",
 								e);
-				standardEnv = sdkTarget.getSDK().getMachine()
-						.getStandardEnvironment();
+				standardEnv = sdkTarget.getMachine().getStandardEnvironment();
 			}
 
 			cachedStdEnvMap.put(sdkTarget, standardEnv);

Modified: trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1ProcessLauncher.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1ProcessLauncher.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.sb1/src/org/maemo/esbox/internal/scratchbox/sb1/launcher/Scratchbox1ProcessLauncher.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -215,7 +215,7 @@
 	 */
 	@Override
 	protected Process doCreateProcess() throws Exception {
-		return sdkTarget.getSDK().getMachine().createProcess(
+		return sdkTarget.getMachine().createProcess(
 				getLaunchCurrentWorkingDirectory(),
 				getLaunchCommandArguments(),
 				getLaunchEnvironmentModifierBlock(), isUsePTY());

Modified: trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/core/Scratchbox2SDK.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/core/Scratchbox2SDK.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/core/Scratchbox2SDK.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -355,8 +355,7 @@
 			IScratchboxSDKTarget sdkTarget, List<IPath> hostPaths)
 			throws MicaException {
 		// find cached value
-		CacheUtils.getInstance().ensureCurrentMachine(SB2_MACHINE, sdkTarget.getSDK()
-				.getMachine());
+		CacheUtils.getInstance().ensureCurrentMachine(SB2_MACHINE, sdkTarget.getMachine());
 		Map<String, Map<String, String>> targetToPathMappingsMap = null;
 		Map<String, String> storedPathMappings = null;
 		targetToPathMappingsMap = (Map<String, Map<String, String>>) CacheUtils
@@ -464,8 +463,7 @@
 	private synchronized IPath getMaemoRootstrapPath(
 			Scratchbox2SDKTarget sdkTarget) throws MicaException {
 		// find cached value
-		CacheUtils.getInstance().ensureCurrentMachine(SB_MACHINE, sdkTarget.getSDK()
-				.getMachine());
+		CacheUtils.getInstance().ensureCurrentMachine(SB_MACHINE, sdkTarget.getMachine());
 		Map<String, String> targetToRootstrapMap = null;
 		targetToRootstrapMap = (Map<String, String>) CacheUtils.getInstance()
 				.getCachedValue(SB2_TARGET_TO_ROOTSTRAP_MAP);

Modified: trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2EnvironmentProvider.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2EnvironmentProvider.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2EnvironmentProvider.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -75,8 +75,7 @@
 						.logError(
 								"Failed to read scratchbox environment; using system environment",
 								e);
-				standardEnv = sdkTarget.getSDK().getMachine()
-						.getStandardEnvironment();
+				standardEnv = sdkTarget.getMachine().getStandardEnvironment();
 			}
 
 			cachedStdEnvMap.put(sdkTarget, standardEnv);

Modified: trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2ProcessLauncher.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2ProcessLauncher.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/launcher/Scratchbox2ProcessLauncher.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -107,7 +107,7 @@
 	 */
 	@Override
 	protected Process doCreateProcess() throws Exception {
-		return sdkTarget.getSDK().getMachine().createProcess(
+		return sdkTarget.getMachine().createProcess(
 				getLaunchCurrentWorkingDirectory(),
 				getLaunchCommandArguments(),
 				getLaunchEnvironmentModifierBlock(), isUsePTY());

Modified: trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/ui/adapters/MaemoScratchbox2SDKManageTargetAdapter.java
===================================================================
--- trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/ui/adapters/MaemoScratchbox2SDKManageTargetAdapter.java	2008-11-17 21:02:53 UTC (rev 943)
+++ trunk/org.maemo.esbox.scratchbox.sb2/src/org/maemo/esbox/internal/scratchbox/sb2/ui/adapters/MaemoScratchbox2SDKManageTargetAdapter.java	2008-11-17 21:07:20 UTC (rev 944)
@@ -99,8 +99,7 @@
 				// see if a rootstrap -- possibly disconnected from the target --
 				// exists
 				List<MaemoRootstrap> rootstraps = sdk
-						.getAvailableMaemoRootstraps(selectedTarget.getSDK()
-								.getMachine());
+						.getAvailableMaemoRootstraps(selectedTarget.getMachine());
 				for (MaemoRootstrap rootstrap : rootstraps) {
 					if (rootstrap.getName().equals(selectedTarget.getName())) {
 						rootstrapUsed_ = true;



More information about the Esbox-commits mailing list