Skip to content

Commit

Permalink
Auto merge of rust-lang#781 - shinglyu:linux32, r=emilio
Browse files Browse the repository at this point in the history
Passing additional clang arguments for Linux 32 cross compiling

This is for https://bugzilla.mozilla.org/show_bug.cgi?id=1366050

But it has to wait for KyleMayes/clang-sys#57 to land and version number bumped.

(I'm making up the new version numbers for now.)

r? @emilio
  • Loading branch information
bors-servo authored Jul 4, 2017
2 parents 1cbb5d3 + ee478a1 commit d0df1b9
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 6 deletions.
14 changes: 10 additions & 4 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ quasi_codegen = "0.32"
[dependencies]
cexpr = "0.2"
cfg-if = "0.1.0"
clang-sys = { version = "0.18.0", features = ["runtime", "clang_3_9"] }
clang-sys = { version = "0.19.0", features = ["runtime", "clang_3_9"] }
lazy_static = "0.2.1"
peeking_take_while = "0.1.2"
syntex_syntax = "0.58"
Expand Down
2 changes: 1 addition & 1 deletion src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1063,7 +1063,7 @@ impl<'ctx> Bindings<'ctx> {
options.build();

// TODO: Make this path fixup configurable?
if let Some(clang) = clang_sys::support::Clang::find(None) {
if let Some(clang) = clang_sys::support::Clang::find(None, &options.clang_args) {
// If --target is specified, assume caller knows what they're doing
// and don't mess with include paths for them
let has_target_arg = options.clang_args
Expand Down

0 comments on commit d0df1b9

Please sign in to comment.