[
https://issues.apache.org/jira/browse/CB-7686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14155333#comment-14155333
]
ASF subversion and git services commented on CB-7686:
-----------------------------------------------------
Commit e0f7530366e945b232ada3bf7bc189d5ec0090db in cordova-mobile-spec's branch
refs/heads/master from [~cmarcelk]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-mobile-spec.git;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: https://issues.apache.org/jira/browse/CB-7686
> 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
(v6.3.4#6332)
