cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mark Koudritsky" <>
Subject Re: Review Request 15046: "Prepare" should not depend on the ~/.cordova/libs directory
Date Thu, 07 Nov 2013 16:48:58 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 7, 2013, 4:48 p.m.)

Review request for cordova.


Fixed conflicts in tests with this commit

Bugs: CB-5063

Repository: cordova-cli


Currently all platform parsers (src/metadata/*_parser.js) copy
cordova.js from util.libDirectory for each run of prepare. This results
in errors when changing machines since the ~/.cordova/lib directory is
not populated.

With this change all the parsers construct the project www dir by first
copying contents of a new dir named <project_root>/platforms/<platfrom>/platform_www
(which contains cordova.js) and then the www dir from the app.

Until individual platform create scripts will be updated to create the
platform_www dir, prepare.js checks if the platform_www exists and if
not, creates it using cordova.js from ~/.cordova/lib.

Even after the platform create scripts are updated the check will be
required for projects that were created using older version and don't
yet have a platform_www dir.

Diffs (updated)

  spec/metadata/android_parser.spec.js 4afb52d 
  spec/metadata/blackberry_parser.spec.js 3353eb1 
  spec/metadata/ios_parser.spec.js 17206c6 
  spec/metadata/windows8_parser.spec.js 5639317 
  spec/metadata/wp7_parser.spec.js 6b69d09 
  spec/metadata/wp8_parser.spec.js 02372cc 
  spec/prepare.spec.js 8012924 
  src/metadata/android_parser.js df446e1 
  src/metadata/blackberry10_parser.js d9f71f3 
  src/metadata/firefoxos_parser.js c3edd7b 
  src/metadata/ios_parser.js 20e985c 
  src/metadata/windows8_parser.js 68520fc 
  src/metadata/wp7_parser.js b19dc32 
  src/metadata/wp8_parser.js 3bdef16 
  src/prepare.js 4ea22c5 



npm test
corodva platform add android
corodva platform add ios
corodva platform add balckberry10
cordova prepare
verified platfrom_www dir exists and contains the right file.

Did not test for win platforms.


Mark Koudritsky

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message