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

Jonathan Bond commented on CB-6272:
-----------------------------------

Patch should be:
path.join.apply(null, dep.subdir.split('/'));

My bad, I'll write a test for the 'subdir' case, no npm tests for it.


> Fix subdir handling
> -------------------
>
>                 Key: CB-6272
>                 URL: https://issues.apache.org/jira/browse/CB-6272
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugman
>    Affects Versions: 3.5.0
>            Reporter: Josh Soref
>            Assignee: Josh Soref
>            Priority: Blocker
>
> CB-6160 landed [~jbondc]'s ae2ce7ac6325d1c6558365b4be82e5a8ff0983b9
> which broke some portion of install:
> +                if (dep.subdir) {
> +                    dep.subdir = path.join(dep.subdir.split('/'));
> ...
> -                    if (dep_subdir) {
> -                        dep_subdir = path.join.apply(null, 
> dep_subdir.split('/'));



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to