cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Grebnov (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (CB-5192) Plugman engine check fails on Windows
Date Thu, 24 Oct 2013 06:32:01 GMT

     [ https://issues.apache.org/jira/browse/CB-5192?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Sergey Grebnov resolved CB-5192.
--------------------------------

    Resolution: Fixed
      Assignee:     (was: Sergey Grebnov)

Fixed by pull request below. Please review and merge
https://github.com/apache/cordova-plugman/pull/29

> Plugman engine check fails on Windows
> -------------------------------------
>
>                 Key: CB-5192
>                 URL: https://issues.apache.org/jira/browse/CB-5192
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugman
>         Environment: Windows
>            Reporter: Sergey Grebnov
>
> Right now the simplest (and also the most correct IMO) way to specify plugin restrictions
to specific cordova version is the following:
> plugin.xml:
>  <engines>
>    <engine name="cordova" version=">=2.7.0" />  
> </engines>
> But in this case as per plugman current implementation and engines definition(plugman/src/util/default-engines.js)
plugman won't be able to resove script because on Windows the script named 'version.bat',
not just 'version'.
> module.exports = function(project_dir){
>     return {
>         'cordova': 
>             { 'platform':'*', 'scriptSrc': path.join(project_dir,'cordova','version')
}, <- works in general, but NOT for WP7/8 
> 	 ...
>         'cordova-wp8': 
>             { 'platform':'wp8', 'scriptSrc': path.join(project_dir,'cordova','version.bat')
}, <- correct location, not used in case of example above
> child_process.exec should correctly resolve this case (missing cmd file extension) but
plugman check for file location so it fails to find it.
> This should affect all platforms which are supported on Windows and use .bat files (WP7/8,
BlackBerry, Android)



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message