diff --git a/changelog/install_uninstall.dd b/changelog/install_uninstall.dd new file mode 100644 index 0000000..445c64b --- /dev/null +++ b/changelog/install_uninstall.dd @@ -0,0 +1,5 @@ +'install' and 'uninstall' commands have been removed + +Those commands were long-deprecated aliases to `fetch` and `remove`, +respectively, and usage of them triggered a warning. +They are no longer listed as command in help and dub will no longer recognize them. diff --git a/source/dub/commandline.d b/source/dub/commandline.d index 9465c01..a9e00d4 100644 --- a/source/dub/commandline.d +++ b/source/dub/commandline.d @@ -58,10 +58,8 @@ ), CommandGroup("Package management", new FetchCommand, - new InstallCommand, new AddCommand, new RemoveCommand, - new UninstallCommand, new UpgradeCommand, new AddPathCommand, new RemovePathCommand, @@ -279,7 +277,7 @@ handler.commandGroups = getCommands(); assert(handler.commandNames == ["init", "run", "build", "test", "lint", "generate", - "describe", "clean", "dustmite", "fetch", "install", "add", "remove", "uninstall", + "describe", "clean", "dustmite", "fetch", "add", "remove", "upgrade", "add-path", "remove-path", "add-local", "remove-local", "list", "search", "add-override", "remove-override", "list-overrides", "clean-caches", "convert"]); } @@ -1935,20 +1933,6 @@ } } -class InstallCommand : FetchCommand { - this() @safe pure nothrow - { - this.name = "install"; - this.hidden = true; - } - override void prepare(scope CommandArgs args) { super.prepare(args); } - override int execute(Dub dub, string[] free_args, string[] app_args) - { - warnRenamed("install", "fetch"); - return super.execute(dub, free_args, app_args); - } -} - class RemoveCommand : FetchRemoveCommand { private { bool m_nonInteractive; @@ -2021,21 +2005,6 @@ } } -class UninstallCommand : RemoveCommand { - this() @safe pure nothrow - { - this.name = "uninstall"; - this.hidden = true; - } - override void prepare(scope CommandArgs args) { super.prepare(args); } - override int execute(Dub dub, string[] free_args, string[] app_args) - { - warnRenamed("uninstall", "remove"); - return super.execute(dub, free_args, app_args); - } -} - - /******************************************************************************/ /* ADD/REMOVE PATH/LOCAL */ /******************************************************************************/ @@ -2785,11 +2754,6 @@ } } -private void warnRenamed(string prev, string curr) -{ - logWarn("The '%s' Command was renamed to '%s'. Please update your scripts.", prev, curr); -} - private bool addDependency(Dub dub, ref PackageRecipe recipe, string depspec) { Dependency dep;