cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From martincgg <>
Subject [GitHub] cordova-mobile-spec pull request: CB-6437[Improvements & support f...
Date Mon, 21 Apr 2014 20:09:52 GMT
Github user martincgg commented on a diff in the pull request:
    --- Diff: createmobilespec/createmobilespec.js ---
    @@ -17,99 +16,197 @@
         KIND, either express or implied.  See the License for the
         specific language governing permissions and limitations
         under the License.
    +     */
    -var fs = require('fs'),
    -    path = require('path'),
    +var fs            = require("fs"),
    +    path          = require("path"),
    +    child_process = require("child_process"),
    +// Dependencies requirements check
     try {
    -    shelljs = require('shelljs');
    -    optimist = require('optimist');
    +    shelljs = require("shelljs");
    +    optimist = require("optimist");
     } catch (e) {
    -    console.error('Missing module. Please run "npm install" from this directory:\n\t'
    +    console.error('Missing module. Please run \"npm install\" from this directory:\n\t'
    -var tokens = process.argv.slice(2);
    -var argv = optimist(tokens)
    -           .default('android', false)
    -           .default('ios', false)
    -           .usage('Usage: $0 [--android] [--ios]\nDefault is to use Android and iOS.')
    -           .argv;
    -// preserve the original behavior when there are no args
    -if (tokens.length === 0) {
    - = true;
    -    argv.ios = true;
    +// Print relevant information
    +console.log("Creating \"mobilespec\" project. If you have any errors, it may be from
missing repositories.");
    +console.log("To clone needed repositories:");
    +console.log("  ./cordova-coho/coho repo-clone -r plugins -r mobile-spec -r android -r
ios -r cli");
    +console.log("To update all repositories:");
    +console.log("  ./cordova-coho/coho repo-update");
    +// Setting up vars, folders and libraries, to ensure full compatibility cross platform,
absolute paths are used instead of relative paths
    +// Cordova Coho dir, it should contain all libraries and required repositories
    +// [cordova-cli, cordova-android, cordova-blackberry, cordova-ios, cordova-windows, cordova-windows8,
all plugins libraries, cordova-mobile-spec, cordova-js]
    +// searchDir function it was added, to look for cordova-coho folder backwards, for cases
like absolute/path/cordova-coho/cordova-coho/...All libraries
    +// This is to make sure that cordova-coho exists and it's the right one.
    +var mainModDir     = process.cwd(),
    +    coho_dir       = searchDir(mainModDir, "cordova-coho"),
    --- End diff --
    I just wanted to make sure that we were in the right directory, avoid fail if a error
is found, something like a wrong directory or so. Make the module work no matter where's located
inside the mobile spec library or any other library.
    No worries, I got rid of it. It will always look for the coho directory with 3 jumps back.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message