apache / cordova-plugman

Apache Cordova Plugman
https://cordova.apache.org/
Apache License 2.0
401 stars 151 forks source link

CB-6414 - fixes the issue where two config.xml munges exists, it will st... #73

Closed jbavari closed 10 years ago

jbavari commented 10 years ago

As seen in this issue:

https://issues.apache.org/jira/browse/CB-6414

Referencing this comment by @kamrik here: https://github.com/apache/cordova-plugman/pull/72

I've corrected the code for both iOS and Android by correcting the code in config_keeper.

revie commented 9 years ago

I see the work-around for Android, but it is still broken for Amazon-FireOS, as reported here:

http://forum.ionicframework.com/t/ionic-1-0-1-amazon-cordova-class-not-found/30542/3

Can the work-around be expanded to be "(platform == 'android' || platform == 'amazon-fireos')" please?