Generalize envs() and args() to iterators.

 * Command::envs() now takes anything that is IntoIterator<Item=(K, V)>
   where both K and V are AsRef<OsStr>.
 * Since we're not 100% sure that's the right signature, envs() is
   now marked unstable.  (You can use envs() with HashMap<str, str> but
   not Vec<(str, str)>, for instance.)
 * Update the test to match.

 * By analogy, args() now takes any IntoIterator<Item=S>, S: AsRef<OsStr>.
   This should be uncontroversial.

1 files changed, 4 insertions, 1 deletions

diff --git a/src/test/run-pass/process-envs.rs b/src/test/run-pass/process-envs.rs
index 80ff16aadb2..a131dcbe4dd 100644
--- a/src/test/run-pass/process-envs.rs
+++ b/src/test/run-pass/process-envs.rs
@@ -10,8 +10,11 @@
 
 // ignore-emscripten
 
+#![feature(command_envs)]
+
 use std::process::Command;
 use std::env;
+use std::collections::HashMap;
 
 #[cfg(all(unix, not(target_os="android")))]
 pub fn env_cmd() -> Command {
@@ -38,7 +41,7 @@ fn main() {
     env::set_var("RUN_TEST_NEW_ENV", "123");
 
     // create filtered environment vector
-    let filtered_env : Vec<(String, String)> =
+    let filtered_env : HashMap<String, String> =
         env::vars().filter(|&(ref k, _)| k == "PATH").collect();
 
     let mut cmd = env_cmd();