Skip to content

Commit

Permalink
Ignore regex tests (regular, cfail and benchmark) on Windows (for now).
Browse files Browse the repository at this point in the history
  • Loading branch information
BurntSushi committed Apr 25, 2014
1 parent 09a8b38 commit 7269bc7
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/libregex/lib.rs
Expand Up @@ -386,7 +386,8 @@ mod parse;
mod re;
mod vm;

#[cfg(test)]
// FIXME(#13725) windows needs fixing.
#[cfg(test, not(windows))]
mod test;

/// The `program` module exists to support the `regex!` macro. Do not use.
Expand Down
2 changes: 2 additions & 0 deletions src/test/bench/shootout-regex-dna.rs
Expand Up @@ -8,6 +8,8 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// FIXME(#13725) windows needs fixing.
// ignore-win32
// ignore-stage1
// ignore-cross-compile #12102

Expand Down
2 changes: 2 additions & 0 deletions src/test/compile-fail/syntax-extension-regex-invalid.rs
Expand Up @@ -8,6 +8,8 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// FIXME(#13725) windows needs fixing.
// ignore-win32
// ignore-stage1

#![feature(phase)]
Expand Down

5 comments on commit 7269bc7

@bors
Copy link
Contributor

@bors bors commented on 7269bc7 Apr 25, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at BurntSushi@7269bc7

@bors
Copy link
Contributor

@bors bors commented on 7269bc7 Apr 25, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging BurntSushi/rust/regexp = 7269bc7 into auto

@bors
Copy link
Contributor

@bors bors commented on 7269bc7 Apr 25, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BurntSushi/rust/regexp = 7269bc7 merged ok, testing candidate = eea4909

@bors
Copy link
Contributor

@bors bors commented on 7269bc7 Apr 25, 2014

@bors
Copy link
Contributor

@bors bors commented on 7269bc7 Apr 25, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = eea4909

Please sign in to comment.