---

yaml
---
r: 104391
b: refs/heads/try
c: fda6716
h: refs/heads/master
i:
  104389: 13fc57c
  104387: 66dba4a
  104383: 4e66141
v: v3

Author: alexcrichton

[refs]

@@ -2,7 +2,7 @@
 refs/heads/master: 62f1d68439dcfd509eaca29887afa97f22938373
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 6e7f170fedd3c526a643c0b2d13863acd982be02
-refs/heads/try: d70f909fa3bdc1c8231f127882cc30f274b263d1
+refs/heads/try: fda671683ec27454416a76a3c2456b0de2f1bfc0
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
 refs/heads/try2: 147ecfdd8221e4a4d4e090486829a06da1e0ca3c

branches/try/mk/crates.mk

@@ -50,7 +50,7 @@
 ################################################################################
 
 TARGET_CRATES := std extra green rustuv native flate arena glob term semver \
-                 uuid serialize sync getopts collections num test
+                 uuid serialize sync getopts collections num
 HOST_CRATES := syntax rustc rustdoc fourcc
 CRATES := $(TARGET_CRATES) $(HOST_CRATES)
 TOOLS := compiletest rustdoc rustc
@@ -63,8 +63,7 @@ DEPS_native := std
 DEPS_syntax := std term serialize collections
 DEPS_rustc := syntax native:rustllvm flate arena serialize sync getopts \
               collections extra
-DEPS_rustdoc := rustc native:sundown serialize sync getopts collections \
-                test
+DEPS_rustdoc := rustc native:sundown serialize sync getopts collections
 DEPS_flate := std native:miniz
 DEPS_arena := std collections
 DEPS_glob := std
@@ -77,9 +76,8 @@ DEPS_getopts := std
 DEPS_collections := std serialize
 DEPS_fourcc := syntax std
 DEPS_num := std extra
-DEPS_test := std extra collections getopts serialize term
 
-TOOL_DEPS_compiletest := test green rustuv getopts
+TOOL_DEPS_compiletest := extra green rustuv getopts
 TOOL_DEPS_rustdoc := rustdoc green rustuv
 TOOL_DEPS_rustc := rustc green rustuv
 TOOL_SOURCE_compiletest := $(S)src/compiletest/compiletest.rs

branches/try/mk/tests.mk

@@ -537,6 +537,10 @@ TEST_SREQ$(1)_T_$(2)_H_$(3) = \
 # remove directive, if present, from CFG_RUSTC_FLAGS (issue #7898).
 CTEST_RUSTC_FLAGS := $$(subst --cfg ndebug,,$$(CFG_RUSTC_FLAGS))
 
+# There's no need our entire test suite to take up gigabytes of space on disk
+# including copies of libstd/libextra all over the place
+CTEST_RUSTC_FLAGS := $$(CTEST_RUSTC_FLAGS) -C prefer-dynamic
+
 # The tests can not be optimized while the rest of the compiler is optimized, so
 # filter out the optimization (if any) from rustc and then figure out if we need
 # to be optimized

branches/try/src/compiletest/compiletest.rs

@@ -13,13 +13,16 @@
 #[allow(non_camel_case_types)];
 #[deny(warnings)];
 
-extern crate test;
+extern crate extra;
 extern crate getopts;
 
 use std::os;
 use std::io;
 use std::io::fs;
+
 use getopts::{optopt, optflag, reqopt};
+use extra::test;
+
 use common::config;
 use common::mode_run_pass;
 use common::mode_run_fail;

branches/try/src/compiletest/header.rs

@@ -32,8 +32,6 @@ pub struct TestProps {
     force_host: bool,
     // Check stdout for error-pattern output as well as stderr
     check_stdout: bool,
-    // Don't force a --crate-type=dylib flag on the command line
-    no_prefer_dynamic: bool,
 }
 
 // Load any test directives embedded in the file
@@ -47,7 +45,6 @@ pub fn load_props(testfile: &Path) -> TestProps {
     let mut check_lines = ~[];
     let mut force_host = false;
     let mut check_stdout = false;
-    let mut no_prefer_dynamic = false;
     iter_header(testfile, |ln| {
         match parse_error_pattern(ln) {
           Some(ep) => error_patterns.push(ep),
@@ -70,10 +67,6 @@ pub fn load_props(testfile: &Path) -> TestProps {
             check_stdout = parse_check_stdout(ln);
         }
 
-        if !no_prefer_dynamic {
-            no_prefer_dynamic = parse_no_prefer_dynamic(ln);
-        }
-
         match parse_aux_build(ln) {
             Some(ab) => { aux_builds.push(ab); }
             None => {}
@@ -106,7 +99,6 @@ pub fn load_props(testfile: &Path) -> TestProps {
         check_lines: check_lines,
         force_host: force_host,
         check_stdout: check_stdout,
-        no_prefer_dynamic: no_prefer_dynamic,
     };
 }
 
@@ -175,10 +167,6 @@ fn parse_check_stdout(line: &str) -> bool {
     parse_name_directive(line, "check-stdout")
 }
 
-fn parse_no_prefer_dynamic(line: &str) -> bool {
-    parse_name_directive(line, "no-prefer-dynamic")
-}
-
 fn parse_exec_env(line: &str) -> Option<(~str, ~str)> {
     parse_name_value_directive(line, ~"exec-env").map(|nv| {
         // nv is either FOO or FOO=BAR

branches/try/src/compiletest/runtest.rs

@@ -34,7 +34,7 @@ use std::str;
 use std::task;
 use std::vec;
 
-use test::MetricMap;
+use extra::test::MetricMap;
 
 pub fn run(config: config, testfile: ~str) {
 
@@ -258,12 +258,15 @@ actual:\n\
 }
 
 fn run_debuginfo_test(config: &config, props: &TestProps, testfile: &Path) {
-    let mut config = config {
-        target_rustcflags: cleanup_debug_info_options(&config.target_rustcflags),
-        host_rustcflags: cleanup_debug_info_options(&config.host_rustcflags),
-        .. config.clone()
-    };
 
+    // do not optimize debuginfo tests
+    let mut config = match config.target_rustcflags {
+        Some(ref flags) => config {
+            target_rustcflags: Some(flags.replace("-O", "")),
+            .. (*config).clone()
+        },
+        None => (*config).clone()
+    };
     let config = &mut config;
     let check_lines = &props.check_lines;
     let mut cmds = props.debugger_cmds.connect("\n");
@@ -433,20 +436,6 @@ fn run_debuginfo_test(config: &config, props: &TestProps, testfile: &Path) {
                                   check_lines[i]), &ProcRes);
         }
     }
-
-    fn cleanup_debug_info_options(options: &Option<~str>) -> Option<~str> {
-        if options.is_none() {
-            return None;
-        }
-
-        // Remove options that are either unwanted (-O) or may lead to duplicates due to RUSTFLAGS.
-        let options_to_remove = [~"-O", ~"-g", ~"--debuginfo"];
-        let new_options = split_maybe_args(options).move_iter()
-                                                   .filter(|x| !options_to_remove.contains(x))
-                                                   .to_owned_vec()
-                                                   .connect(" ");
-        Some(new_options)
-    }
 }
 
 fn check_error_patterns(props: &TestProps,
@@ -715,13 +704,9 @@ fn compose_and_run_compiler(
     for rel_ab in props.aux_builds.iter() {
         let abs_ab = config.aux_base.join(rel_ab.as_slice());
         let aux_props = load_props(&abs_ab);
-        let crate_type = if aux_props.no_prefer_dynamic {
-            ~[]
-        } else {
-            ~[~"--crate-type=dylib"]
-        };
         let aux_args =
-            make_compile_args(config, &aux_props, crate_type + extra_link_args,
+            make_compile_args(config, &aux_props, ~[~"--crate-type=dylib"]
+                                                  + extra_link_args,
                               |a,b| {
                                   let f = make_lib_name(a, b, testfile);
                                   ThisDirectory(f.dir_path())
@@ -785,10 +770,6 @@ fn make_compile_args(config: &config,
                      ~"-L", config.build_base.as_str().unwrap().to_owned(),
                      ~"--target=" + target]
         + extras;
-    if !props.no_prefer_dynamic {
-        args.push(~"-C");
-        args.push(~"prefer-dynamic");
-    }
     let path = match xform_file {
         ThisFile(path) => { args.push(~"-o"); path }
         ThisDirectory(path) => { args.push(~"--out-dir"); path }

branches/try/src/doc/guide-testing.md

@@ -170,7 +170,7 @@ runner.
 
 The type signature of a benchmark function differs from a unit test:
 it takes a mutable reference to type
-`test::BenchHarness`. Inside the benchmark function, any
+`extra::test::BenchHarness`. Inside the benchmark function, any
 time-variable or "setup" code should execute first, followed by a call
 to `iter` on the benchmark harness, passing a closure that contains
 the portion of the benchmark you wish to actually measure the
@@ -185,10 +185,9 @@ amount.
 For example:
 
 ~~~
-extern crate test;
-
+extern crate extra;
 use std::vec;
-use test::BenchHarness;
+use extra::test::BenchHarness;
 
 #[bench]
 fn bench_sum_1024_ints(b: &mut BenchHarness) {
@@ -244,8 +243,8 @@ recognize that some calculation has no external effects and remove
 it entirely.
 
 ~~~
-extern crate test;
-use test::BenchHarness;
+extern crate extra;
+use extra::test::BenchHarness;
 
 #[bench]
 fn bench_xor_1000_ints(bh: &mut BenchHarness) {
@@ -274,15 +273,15 @@ example above by adjusting the `bh.iter` call to
 bh.iter(|| range(0, 1000).fold(0, |old, new| old ^ new))
 ~~~
 
-Or, the other option is to call the generic `test::black_box`
+Or, the other option is to call the generic `extra::test::black_box`
 function, which is an opaque "black box" to the optimizer and so
 forces it to consider any argument as used.
 
 ~~~
-extern crate test;
+use extra::test::black_box
 
 bh.iter(|| {
-        test::black_box(range(0, 1000).fold(0, |old, new| old ^ new));
+        black_box(range(0, 1000).fold(0, |old, new| old ^ new));
     });
 ~~~
 

branches/try/src/doc/rustdoc.md

@@ -154,7 +154,7 @@ testing this code, the `fib` function will be included (so it can compile).
 
 Running tests often requires some special configuration to filter tests, find
 libraries, or try running ignored examples. The testing framework that rustdoc
-uses is build on crate `test`, which is also used when you compile crates with
+uses is build on `extra::test`, which is also used when you compile crates with
 rustc's `--test` flag. Extra arguments can be passed to rustdoc's test harness
 with the `--test-args` flag.
 

branches/try/src/doc/tutorial.md

@@ -582,32 +582,6 @@ loop {
 This code prints out a weird sequence of numbers and stops as soon as
 it finds one that can be divided by five.
 
-There is also a for-loop that can be used to iterate over a range of numbers:
-
-~~~~
-for n in range(0, 5) {
-    println!("{}", n);
-}
-~~~~
-
-The snippet above prints integer numbers under 5 starting at 0.
-
-More generally, a for loop works with anything implementing the `Iterator` trait.
-Data structures can provide one or more methods that return iterators over
-their contents. For example, strings support iteration over their contents in
-various ways:
-
-~~~~
-let s = "Hello";
-for c in s.chars() {
-    println!("{}", c);
-}
-~~~~
-
-The snippet above prints the characters in "Hello" vertically, adding a new
-line after each character.
-
-
 # Data structures
 
 ## Structs

branches/try/src/libarena/lib.rs

@@ -503,10 +503,10 @@ impl<T> Drop for TypedArena<T> {
 }
 
 #[cfg(test)]
-mod tests {
-    extern crate test;
-    use self::test::BenchHarness;
+mod test {
+    extern crate extra;
     use super::{Arena, TypedArena};
+    use self::extra::test::BenchHarness;
 
     struct Point {
         x: int,

branches/try/src/libcollections/bitv.rs

@@ -938,8 +938,7 @@ impl<'a> Iterator<uint> for BitPositions<'a> {
 
 #[cfg(test)]
 mod tests {
-    extern crate test;
-    use self::test::BenchHarness;
+    use extra::test::BenchHarness;
 
     use bitv::{Bitv, SmallBitv, BigBitv, BitvSet, from_bools, from_fn,
                from_bytes};

branches/try/src/libcollections/deque.rs

@@ -41,11 +41,10 @@ pub trait Deque<T> : Mutable {
 
 #[cfg(test)]
 pub mod bench {
-    extern crate test;
-    use self::test::BenchHarness;
     use std::container::MutableMap;
     use std::{vec, rand};
     use std::rand::Rng;
+    use extra::test::BenchHarness;
 
     pub fn insert_rand_n<M:MutableMap<uint,uint>>(n: uint,
                                                   map: &mut M,

branches/try/src/libcollections/dlist.rs

@@ -657,9 +657,8 @@ impl<D:Decoder,T:Decodable<D>> Decodable<D> for DList<T> {
 
 #[cfg(test)]
 mod tests {
-    extern crate test;
-    use self::test::BenchHarness;
     use deque::Deque;
+    use extra::test;
     use std::rand;
     use super::{DList, Node, ListInsertion};
 

branches/try/src/libcollections/lib.rs

@@ -20,7 +20,7 @@
 #[feature(macro_rules, managed_boxes)];
 
 extern crate serialize;
-#[cfg(test)] extern crate test;
+#[cfg(test)] extern crate extra; // benchmark tests need this
 
 pub use bitv::Bitv;
 pub use btree::BTree;

branches/try/src/libcollections/ringbuf.rs

@@ -431,9 +431,8 @@ impl<D:Decoder,T:Decodable<D>> Decodable<D> for RingBuf<T> {
 
 #[cfg(test)]
 mod tests {
-    extern crate test;
-    use self::test::BenchHarness;
     use deque::Deque;
+    use extra::test;
     use std::clone::Clone;
     use std::cmp::Eq;
     use super::RingBuf;

branches/try/src/libcollections/smallintmap.rs

@@ -470,9 +470,9 @@ mod test_map {
 
 #[cfg(test)]
 mod bench {
-    extern crate test;
-    use self::test::BenchHarness;
+
     use super::SmallIntMap;
+    use extra::test::BenchHarness;
     use deque::bench::{insert_rand_n, insert_seq_n, find_rand_n, find_seq_n};
 
     // Find seq

branches/try/src/libcollections/treemap.rs

@@ -1494,9 +1494,9 @@ mod test_treemap {
 
 #[cfg(test)]
 mod bench {
-    extern crate test;
-    use self::test::BenchHarness;
+
     use super::TreeMap;
+    use extra::test::BenchHarness;
     use deque::bench::{insert_rand_n, insert_seq_n, find_rand_n, find_seq_n};
 
     // Find seq