Skip to content

Commit

Permalink
rustc: Use --extern to always override
Browse files Browse the repository at this point in the history
Previously if --extern was specified it would not override crates in the
standard distribution, leading to issues like #21771. This commit alters the
behavior such that if --extern is passed then it will always override any other
choice of crates and no previous match will be used (unless it is the same path
as --extern).

Closes #21771
  • Loading branch information
alexcrichton committed Jan 30, 2015
1 parent 1a51eb9 commit cf535a3
Show file tree
Hide file tree
Showing 4 changed files with 35 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/librustc/metadata/creader.rs
Original file line number Diff line number Diff line change
Expand Up @@ -330,6 +330,7 @@ impl<'a> CrateReader<'a> {
if found {
ret = Some(cnum);
}
return
}

// Alright, so we've gotten this far which means that `data` has the
Expand Down
5 changes: 5 additions & 0 deletions src/test/run-make/extern-overrides-distribution/Makefile
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
-include ../tools.mk

all:
$(RUSTC) libc.rs
$(RUSTC) main.rs --extern libc=$(TMPDIR)/liblibc.rlib
13 changes: 13 additions & 0 deletions src/test/run-make/extern-overrides-distribution/libc.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![crate_type = "lib"]

pub fn foo() {}
16 changes: 16 additions & 0 deletions src/test/run-make/extern-overrides-distribution/main.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

extern crate libc;

fn main() {
libc::foo();
}

0 comments on commit cf535a3

Please sign in to comment.