poetic / ember-cli-cordova

A tool for creating hybrid apps using a combination of ember-cli and cordova
MIT License
441 stars 77 forks source link

ember cordova:init when fails it causes app/router.js to disappear #43

Closed david-gang closed 10 years ago

david-gang commented 10 years ago

In windows 7 i have to be an administrator in order to create symlinks. If i don't do this i get a failure. The interesting issue here is that when ember-cli-cordova fails, router.js disappears.

ember new foo cd foo npm install --save-dev ember-cli-cordova

Now router.js exist.

After this command the router.js disappears:

C:\Users\davidga\Desktop\ember\foo>ember cordova:init com.exlibris.nextgen --pla tform android version: 0.0.39

Installing npm dependencies...done Removing default ember-cli files...done Copying custom ember files to new ember-cli project...done Update index.html with cordova.js ...done Creating Cordova project...done Copying default cordova hooks to cordova project...done Adding ios platform to cordova... Error thrown while running shell command

Error: Command failed: Error: C:\Users\davidga.cordova\lib\ios\cordova\3.5.0\bi n\create: Command failed with exit code ENOENT at ChildProcess.whenDone (C:\Users\davidga\Desktop\ember\foo\node_modules\em ber-cli-cordova\node_modules\cordova\node_modules\cordova-lib\src\cordova\supers pawn.js:135:23) at ChildProcess.emit (events.js:95:17) at Process.ChildProcess._handle.onexit (child_process.js:807:12)

at ChildProcess.exithandler (child_process.js:647:15) at ChildProcess.emit (events.js:98:17) at maybeClose (child_process.js:755:16) at Process.ChildProcess._handle.onexit (child_process.js:822:5) Update config.xml with your project settings...done Symlinking ember dir to cordova www...done


It may be connected with the issue #37 but i cannot check it because i have not the ability to build ember-cli-cordova from master. May you please make a new release with the fix of the issue #37 and i will check if the issue persists?

jakecraige commented 10 years ago

Just pushed a new release since this was fixed in master. Sorry for the trouble: f5fe0513ee0c73b45787d13f413549ae10380bbe