cordova-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fil...@apache.org
Subject git commit: Further [CB-4077] work: remove plugin directory for dependent plugins during uninstallation.
Date Tue, 09 Jul 2013 21:27:46 GMT
Updated Branches:
  refs/heads/CB-4077 2b70e6b7d -> 281dba805


Further [CB-4077] work: remove plugin directory for dependent plugins during uninstallation.


Project: http://git-wip-us.apache.org/repos/asf/cordova-plugman/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-plugman/commit/281dba80
Tree: http://git-wip-us.apache.org/repos/asf/cordova-plugman/tree/281dba80
Diff: http://git-wip-us.apache.org/repos/asf/cordova-plugman/diff/281dba80

Branch: refs/heads/CB-4077
Commit: 281dba805b416f8211a6432103484b14754edffd
Parents: 2b70e6b
Author: Fil Maj <maj.fil@gmail.com>
Authored: Tue Jul 9 14:27:41 2013 -0700
Committer: Fil Maj <maj.fil@gmail.com>
Committed: Tue Jul 9 14:27:41 2013 -0700

----------------------------------------------------------------------
 spec/uninstall.spec.js | 5 ++++-
 src/uninstall.js       | 8 +++++++-
 2 files changed, 11 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-plugman/blob/281dba80/spec/uninstall.spec.js
----------------------------------------------------------------------
diff --git a/spec/uninstall.spec.js b/spec/uninstall.spec.js
index aca4e7c..738e941 100644
--- a/spec/uninstall.spec.js
+++ b/spec/uninstall.spec.js
@@ -51,7 +51,10 @@ describe('uninstallPlatform', function() {
         });
 
         describe('with dependencies', function() {
-            it('should uninstall "dangling" dependencies');
+            it('should uninstall "dangling" dependencies', function() {
+            });
+            it('should remove dangling dependent directories', function() {
+            });
             it('should not uninstall any dependencies that are relied on by other plugins');
         });
     });

http://git-wip-us.apache.org/repos/asf/cordova-plugman/blob/281dba80/src/uninstall.js
----------------------------------------------------------------------
diff --git a/src/uninstall.js b/src/uninstall.js
index 6d59e84..0f07774 100644
--- a/src/uninstall.js
+++ b/src/uninstall.js
@@ -93,7 +93,13 @@ function runUninstall(actions, platform, project_dir, plugin_dir, plugins_dir,
o
                 cli_variables: options.cli_variables,
                 is_top_level: false /* TODO: should this "is_top_level" param be false for
dependents? */
             };
-            runUninstall(actions, platform, project_dir, dependent_path, plugins_dir, opts,
end);
+            runUninstall(actions, platform, project_dir, dependent_path, plugins_dir, opts,
function(err) {
+                if (err) {
+                    if (callback) return callback(err);
+                    else throw err;
+                }
+                module.exports.uninstallPlugin(dangler, plugins_dir, end);
+            });
         });
     } else {
         // this plugin can get axed by itself, gogo!


Mime
View raw message