[jira] [Commented] (CB-9163) Make engines error look less nasty

2015-06-17 Thread Steve Gill (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590628#comment-14590628
 ] 

Steve Gill commented on CB-9163:


I think the bigger issue here is that this stops automated plugin install. 
Cordova-lib should leave a nice warning message and skip instead of error and 
exit when the supported platform version isn't available. 

 Make engines error look less nasty
 --

 Key: CB-9163
 URL: https://issues.apache.org/jira/browse/CB-9163
 Project: Apache Cordova
  Issue Type: Bug
  Components: CordovaLib, Plugin Whitelist
 Environment: iOS
Reporter: Shazron Abdullah
  Labels: cordova-ios-4.0.x

 When installing the cordova-plugin-whitelist plugin (from master), and it 
 doesn't fulfill the engine requirement, you get a scary error which is 
 harmless.
 {code}
 $ cordova plugin add 
 https://github.com/apache/cordova-plugin-whitelist.git#master
 Fetching plugin https://github.com/apache/cordova-plugin-whitelist.git; via 
 git clone
 Repository https://github.com/apache/cordova-plugin-whitelist.git; checked 
 out to git ref master.
 Installing cordova-plugin-whitelist for android
 Installing cordova-plugin-whitelist for ios
 Failed to install 'cordova-plugin-whitelist':CordovaError: Plugin doesn't 
 support this project's cordova-ios version. cordova-ios: 3.8.0, failed 
 version requirement: =4.0.0-dev
 at checkEngines 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/src/plugman/install.js:119:29)
 at 
 /Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/src/plugman/install.js:296:16
 at _fulfilled 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:787:54)
 at self.promiseDispatch.done 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:816:30)
 at Promise.promise.promiseDispatch 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:749:13)
 at 
 /Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:557:44
 at flush 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:108:17)
 at process._tickCallback (node.js:419:13)
 Plugin doesn't support this project's cordova-ios version. cordova-ios: 
 3.8.0, failed version requirement: =4.0.0-dev
 {code}
 Change it to not have the stack-trace, and have a friendlier output. If you 
 have the Android platform added, it succeeds in installing it also, but not 
 sure if its because the platform tag appears first in plugin.xml (check).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9163) Make engines error look less nasty

2015-06-17 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590686#comment-14590686
 ] 

ASF subversion and git services commented on CB-9163:
-

Commit d7f8ebdec56a8b92997ba0458eb31139ab608e10 in cordova-lib's branch 
refs/heads/master from [~stevegill]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-lib.git;h=d7f8ebd ]

CB-9163 when engine check isn't satisfied, skip it


 Make engines error look less nasty
 --

 Key: CB-9163
 URL: https://issues.apache.org/jira/browse/CB-9163
 Project: Apache Cordova
  Issue Type: Bug
  Components: CordovaLib, Plugin Whitelist
 Environment: iOS
Reporter: Shazron Abdullah
  Labels: cordova-ios-4.0.x

 When installing the cordova-plugin-whitelist plugin (from master), and it 
 doesn't fulfill the engine requirement, you get a scary error which is 
 harmless.
 {code}
 $ cordova plugin add 
 https://github.com/apache/cordova-plugin-whitelist.git#master
 Fetching plugin https://github.com/apache/cordova-plugin-whitelist.git; via 
 git clone
 Repository https://github.com/apache/cordova-plugin-whitelist.git; checked 
 out to git ref master.
 Installing cordova-plugin-whitelist for android
 Installing cordova-plugin-whitelist for ios
 Failed to install 'cordova-plugin-whitelist':CordovaError: Plugin doesn't 
 support this project's cordova-ios version. cordova-ios: 3.8.0, failed 
 version requirement: =4.0.0-dev
 at checkEngines 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/src/plugman/install.js:119:29)
 at 
 /Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/src/plugman/install.js:296:16
 at _fulfilled 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:787:54)
 at self.promiseDispatch.done 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:816:30)
 at Promise.promise.promiseDispatch 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:749:13)
 at 
 /Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:557:44
 at flush 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:108:17)
 at process._tickCallback (node.js:419:13)
 Plugin doesn't support this project's cordova-ios version. cordova-ios: 
 3.8.0, failed version requirement: =4.0.0-dev
 {code}
 Change it to not have the stack-trace, and have a friendlier output. If you 
 have the Android platform added, it succeeds in installing it also, but not 
 sure if its because the platform tag appears first in plugin.xml (check).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-9163) Make engines error look less nasty

2015-06-17 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590708#comment-14590708
 ] 

ASF subversion and git services commented on CB-9163:
-

Commit b6727ba9e465a733ce07ac222b17b46ab975f714 in cordova-lib's branch 
refs/heads/master from [~stevegill]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-lib.git;h=b6727ba ]

CB-9163 fixed failing tests


 Make engines error look less nasty
 --

 Key: CB-9163
 URL: https://issues.apache.org/jira/browse/CB-9163
 Project: Apache Cordova
  Issue Type: Bug
  Components: CordovaLib, Plugin Whitelist
 Environment: iOS
Reporter: Shazron Abdullah
  Labels: cordova-ios-4.0.x

 When installing the cordova-plugin-whitelist plugin (from master), and it 
 doesn't fulfill the engine requirement, you get a scary error which is 
 harmless.
 {code}
 $ cordova plugin add 
 https://github.com/apache/cordova-plugin-whitelist.git#master
 Fetching plugin https://github.com/apache/cordova-plugin-whitelist.git; via 
 git clone
 Repository https://github.com/apache/cordova-plugin-whitelist.git; checked 
 out to git ref master.
 Installing cordova-plugin-whitelist for android
 Installing cordova-plugin-whitelist for ios
 Failed to install 'cordova-plugin-whitelist':CordovaError: Plugin doesn't 
 support this project's cordova-ios version. cordova-ios: 3.8.0, failed 
 version requirement: =4.0.0-dev
 at checkEngines 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/src/plugman/install.js:119:29)
 at 
 /Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/src/plugman/install.js:296:16
 at _fulfilled 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:787:54)
 at self.promiseDispatch.done 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:816:30)
 at Promise.promise.promiseDispatch 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:749:13)
 at 
 /Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:557:44
 at flush 
 (/Users/shaz/Documents/Git/Apache/cordova-lib/cordova-lib/node_modules/q/q.js:108:17)
 at process._tickCallback (node.js:419:13)
 Plugin doesn't support this project's cordova-ios version. cordova-ios: 
 3.8.0, failed version requirement: =4.0.0-dev
 {code}
 Change it to not have the stack-trace, and have a friendlier output. If you 
 have the Android platform added, it succeeds in installing it also, but not 
 sure if its because the platform tag appears first in plugin.xml (check).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org