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

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

Commit d95130a8897a09f44cecbe1cbd73ab7b88be55a0 in cordova-plugman's branch 
refs/heads/master from [~jsoref]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-plugman.git;h=d95130a ]

CB-6124 Make `cordova plugin remove` resilient to a missing plugin directory

This breaks a couple prepare tests that Andrew deletes in the next commit.
github: close #57


> cordova plugin remove x fails if there's a plugin whose directory was deleted
> -----------------------------------------------------------------------------
>
>                 Key: CB-6124
>                 URL: https://issues.apache.org/jira/browse/CB-6124
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugman
>    Affects Versions: 3.4.0
>            Reporter: Josh Soref
>            Assignee: Josh Soref
>
> $ cordova plugin remove org.apache.cordova.battery-status
> Uninstalling org.apache.cordova.battery-status from android
> Error: ENOENT, no such file or directory 
> 'mobilespec/plugins/com.blackberry.pim.lib/plugin.xml'
>     at Object.fs.openSync (fs.js:427:18)
>     at Object.fs.readFileSync (fs.js:284:15)
>     at Object.module.exports.parseElementtreeSync 
> (cordova-plugman/src/util/xml-helpers.js:119:27)
>     at cordova-plugman/src/util/dependencies.js:22:35
>     at Array.forEach (native)
>     at Object.module.exports.generate_dependency_info 
> (cordova-plugman/src/util/dependencies.js:21:45)
>     at Object.module.exports.dependents 
> (cordova-plugman/src/util/dependencies.js:38:46)
>     at runUninstall (cordova-plugman/src/uninstall.js:105:35)
>     at Function.module.exports.uninstallPlatform 
> (cordova-plugman/src/uninstall.js:39:12)
>     at cordova-cli/src/plugin.js:164:58
> mobilespec $ ls plugins/com.blackberry.pim.lib
> ls: plugins/com.blackberry.pim.lib: No such file or directory
> Expected results: Just because a plugin doesn't exist doesn't mean you can't 
> build a graph. Especially if the goal is to delete stuff.
> Note: I'm only using cordova plugin remove to delete things (and cordova 
> plugin add to add), so whomever either added it incompletely or failed to 
> properly remove it is partially to blame, but my goal right now is safe 
> handling of bad states, not investigating how I got to the bad state.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to