commit 0e6bc66b0cb7b174aa81cd2a66ab8e1bba50ad97
author Andrew Grieve <agrieve@chromium.org> Tue Feb 24 10:44:57 2015 -0500
committer Andrew Grieve <agrieve@chromium.org> Tue Feb 24 10:44:57 2015 -0500
tree 0bca723eedc965c0941152222af472a8d4f84c54
parent 420e330eeb8f73f8599f6feb8e6a8b4be67dd3c0

Fix createmobilespec --thirdpartyplugins not working when you don't have --android

createmobilespec/createmobilespec.js

diff --git a/createmobilespec/createmobilespec.js b/createmobilespec/createmobilespec.js
index 6157bd9..5c8ec53 100755
--- a/createmobilespec/createmobilespec.js
+++ b/createmobilespec/createmobilespec.js
@@ -505,8 +505,10 @@
             var fbVars = ' --variable APP_ID=value --variable APP_NAME=value';
             shelljs.exec(cli + " plugin add " + CORDOVA_REGISTRY_PLUGINS.join(' ') + searchpath + browserifyFlag + mapVars + fbVars);
             // Delete duplicate <uses-permission> due to maxSdk (CB-8401)
-            shelljs.sed('-i', /{[^{]*(maxSdk|WRITE_EXTERNAL_STORAGE).*?(maxSdk|WRITE_EXTERNAL_STORAGE)[^}]*},/, '', path.join('plugins', 'android.json'));
-            shelljs.sed('-i', /<.*(maxSdk|WRITE_EXTERNAL_STORAGE).*(maxSdk|WRITE_EXTERNAL_STORAGE).*>/, '', path.join('platforms', 'android', 'AndroidManifest.xml'));
+            if (argv.android) {
+                shelljs.sed('-i', /{[^{]*(maxSdk|WRITE_EXTERNAL_STORAGE).*?(maxSdk|WRITE_EXTERNAL_STORAGE)[^}]*},/, '', path.join('plugins', 'android.json'));
+                shelljs.sed('-i', /<.*(maxSdk|WRITE_EXTERNAL_STORAGE).*(maxSdk|WRITE_EXTERNAL_STORAGE).*>/, '', path.join('platforms', 'android', 'AndroidManifest.xml'));
+            }
         }
         if (argv.thirdpartyplugins || argv.telerikplugins) {
             shelljs.exec(cli + " plugin add " + TELERIK_VERIFIED_PLUGINS.join(' ') + searchpath + browserifyFlag);