I reviewed and merged it in. Error is fixed and tests were all passing.
On Thu, Apr 10, 2014 at 3:52 PM, Steven Gill <[email protected]> wrote: > Looks like Josh Bavari created a pull request that fixes it. > https://github.com/apache/cordova-plugman/pull/73 > > I can review it when I get home if no one has done it by then (currently > traveling back from ApacheCon). > On Apr 10, 2014 1:42 PM, "David Kemp" <[email protected]> wrote: > >> It appears that the plugman commit >> 57a5eaa1357f1f70d85437b9bb2d8f322bc39bc2 >> from April 8 broke the build for iOS. Android is OK. >> >> If this requires a change to the test procedure, let me know - otherwise >> it >> should be fixed/rolled back. >> >> Details below.. >> >> TypeError: Cannot read property 'parents' of undefined >> at PlatformMunger_apply_file_munge [as apply_file_munge] >> >> (/Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/plugman/src/util/config-changes.js:156:35) >> at PlatformMunger.reapply_global_munge >> >> (/Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/plugman/src/util/config-changes.js:283:14) >> at >> /Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/src/prepare.js:105:24 >> at _fulfilled >> >> (/Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/q/q.js:798:54) >> at self.promiseDispatch.done >> >> (/Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/q/q.js:827:30) >> at Promise.promise.promiseDispatch >> >> (/Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/q/q.js:760:13) >> at >> /Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/q/q.js:821:14 >> at flush >> (/Users/medic/buildbot/slave_ios/IOS_Release/build/cordova-cli/node_modules/q/q.js:108:17) >> at process._tickCallback (node.js:415:13) >> at Function.Module.runMain (module.js:499:11)program finished with >> exit code 1 >> elapsedTime=1.089107 >> >
