[ 
https://issues.apache.org/jira/browse/CB-7686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14155339#comment-14155339
 ] 

ASF subversion and git services commented on CB-7686:
-----------------------------------------------------

Commit 753e01d40f2374ccb1f9f400f56936e3ea5dbf72 in cordova-coho's branch 
refs/heads/master from [~cmarcelk]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-coho.git;h=753e01d ]

CB-7686 add "npm-link" command to coho


> 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)

Reply via email to