cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (CB-7686) check if npm-link is present when tools are on master
Date Wed, 01 Oct 2014 19:27:34 GMT


ASF subversion and git services commented on CB-7686:

Commit e0f7530366e945b232ada3bf7bc189d5ec0090db in cordova-mobile-spec's branch refs/heads/master
from [~cmarcelk]
[;h=e0f7530 ]

CB-7686 add check of npm-link when using master of tools

> check if npm-link is present when tools are on master
> -----------------------------------------------------
>                 Key: CB-7686
>                 URL:
>             Project: Apache Cordova
>          Issue Type: New Feature
>          Components: Coho, mobile-spec
>            Reporter: Marcel Kinard
>            Assignee: Marcel Kinard
>            Priority: Minor
> Per CB-7677, we may be in the habit of not testing the latest cordova-lib when we are
on master. Since by default, plugman and cli will download the latest release from the npm
registry unless we explicitly do an "npm link" on our workstations, we may be testing the
wrong version of lib by default.
> So the proposal here is:
> (1) add a check to createmobilespec that if all of [js, lib, plugman, cli] are checked
out at master, that our own dependencies be npm-linked to each other instead of downloaded
directly. This check will not happen if any of those repos are on a branch other than master.
And if for some reason you want to bypass the check even when they all are on master, there
is a --skiplink option in createmobilespec. If this check runs and fails, it will not make
any changes to your workspace, it will simply cause createmobilespec to fail.
> (2) since it should be best practice to do an "npm link" of our own dependencies, this
should be a common task. So add the command "npm-link" to coho to unbuild any directly-installed
dependencies of our own, and instead do an npm-link of them.

This message was sent by Atlassian JIRA

View raw message