Support external subcommands

src/bin/cargo.rs

@@ -223,8 +223,8 @@ fn execute(flags: Flags, config: &mut Config) -> CliResult {
         }
         None => args,
     };
-
-    execute_external_subcommand(config, &args[1], &args)
+    unimplemented!()
+//    execute_external_subcommand(config, &args[1], &args)
 }
 
 fn try_execute_builtin_command(config: &mut Config, args: &[String]) -> Option<CliResult> {
@@ -283,7 +283,7 @@ fn find_closest(config: &Config, cmd: &str) -> Option<String> {
     filtered.get(0).map(|slot| slot.1.clone())
 }
 
-fn execute_external_subcommand(config: &Config, cmd: &str, args: &[String]) -> CliResult {
+fn execute_external_subcommand(config: &Config, cmd: &str, args: &[&str]) -> CliResult {
     let command_exe = format!("cargo-{}{}", cmd, env::consts::EXE_SUFFIX);
     let path = search_directories(config)
         .iter()

src/bin/cli/mod.rs

@@ -506,6 +506,11 @@ pub fn do_main(config: &mut Config) -> Result<(), CliError> {
                       registry)?;
             Ok(())
         }
+        (external, Some(args)) => {
+            let mut ext_args: Vec<&str> = vec![external];
+            ext_args.extend(args.values_of("").unwrap_or_default());
+            super::execute_external_subcommand(config, external, &ext_args)
+        }
         _ => Ok(())
     }
 }
@@ -518,6 +523,7 @@ fn cli() -> App {
             AppSettings::UnifiedHelpMessage,
             AppSettings::DeriveDisplayOrder,
             AppSettings::VersionlessSubcommands,
+            AppSettings::AllowExternalSubcommands,
         ])
         .about("Rust's package manager")
         .arg(

tests/testsuite/cargo_command.rs

@@ -164,7 +164,6 @@ fn override_cargo_home() {
 }
 
 #[test]
-#[ignore]
 fn cargo_subcommand_env() {
     use cargotest::support::cargo_exe;