[
https://issues.apache.org/jira/browse/CB-4189?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13801033#comment-13801033
]
ASF subversion and git services commented on CB-4189:
-----------------------------------------------------
Commit 0596e47f870a13d8658ebb27f01f70fffcec5e23 in branch refs/heads/master
from [~shepheb]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-plugman.git;h=0596e47 ]
Overhaul dependency uninstallation
This fixes several long-standing bugs with plugman deleting
dependencies. The new rules are to uninstall all of the target plugin's
dependencies, provided they are:
- Not installed at the top level.
- Not depended on by any other plugins.
Similarly, plugin directories are only deleted when no plugin depends on
them, and they are not top-level, on any platform.
Fixes CB-5056, CB-5060, CB-4992, and CB-4189.
> Plugman uninstallPlugin plugin repo removal should work bottom-up: dependency
> leaves first, moving up the dependency chain
> --------------------------------------------------------------------------------------------------------------------------
>
> Key: CB-4189
> URL: https://issues.apache.org/jira/browse/CB-4189
> Project: Apache Cordova
> Issue Type: Improvement
> Components: Plugman
> Affects Versions: 2.9.0, 3.0.0
> Reporter: Filip Maj
> Assignee: Mark Koudritsky
> Priority: Minor
>
> Currently, the plugin-repo-removal code (encapsulated as the
> {{uninstallPlugin}} method) blindly removes dependent plugins recursively. It
> should probably do this in a smarter fashion, similarly to how the
> {{uninstallPlatform}} method checks for any other dependents before
> proceeding with recursive uninstalls.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
