Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Rust 2016-03-18. #10076

Merged
merged 1 commit into from Mar 20, 2016
Merged

Update to Rust 2016-03-18. #10076

merged 1 commit into from Mar 20, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Mar 18, 2016

This change is Reviewable

@highfive
Copy link

highfive commented Mar 18, 2016

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
  • These commits modify script code, but no tests are modified. Please consider adding a test!
@@ -118,6 +118,11 @@ version = "0.3.3"
source = "registry+https://github.com/rust-lang/crates.io-index"

[[package]]
name = "bitflags"

This comment has been minimized.

@jdm

jdm Mar 18, 2016

Member

Two copies of bitflags?

This comment has been minimized.

@Ms2ger

Ms2ger Mar 18, 2016

Author Contributor

Unfortunately, yes. Worse, 0.5.0 has been released, with a real compat issue (bitflags/bitflags#38). I'd prefer just pushing 0.5 through rather than going through all the deps twice.

@jdm jdm assigned jdm and unassigned nox Mar 18, 2016
@Ms2ger Ms2ger mentioned this pull request Mar 18, 2016
@jdm
Copy link
Member

jdm commented Mar 18, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 18, 2016

📌 Commit 77cdd97 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Mar 18, 2016

Testing commit 77cdd97 with merge 4ccc5fd...

bors-servo added a commit that referenced this pull request Mar 18, 2016
Update to Rust 2016-03-18.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10076)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 18, 2016

💔 Test failed - gonk

@Ms2ger
Copy link
Contributor Author

Ms2ger commented Mar 18, 2016

@Ms2ger
Copy link
Contributor Author

Ms2ger commented Mar 18, 2016

And need to pacify lockfile_changed.sh.

@Ms2ger
Copy link
Contributor Author

Ms2ger commented Mar 18, 2016

@bors-servo r=jdm

@bors-servo
Copy link
Contributor

bors-servo commented Mar 18, 2016

📌 Commit d807fd8 has been approved by jdm

@jdm
Copy link
Member

jdm commented Mar 18, 2016

@bors-servo: r-

$ ./mach test-compiletest

 Downloading compiletest_rs v0.0.11

   Compiling compiletest_rs v0.0.11

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/procsrv.rs:16:5: 16:37 error: unresolved import `std::dynamic_lib::DynamicLibrary`. Could not find `dynamic_lib` in `std` [E0432]

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/procsrv.rs:16 use std::dynamic_lib::DynamicLibrary;

                                                                                                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/procsrv.rs:16:5: 16:37 help: run `rustc --explain E0432` to see a detailed explanation

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/procsrv.rs:23:9: 23:41 error: the type of this value must be known in this context

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/procsrv.rs:23         path.insert(0, PathBuf::from(p));

                                                                                                                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:122:5: 133:6 error: missing field `quiet` in initializer of `test::TestOpts` [E0063]

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:122     test::TestOpts {

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:123         filter: match config.filter {

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:124             None => None,

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:125             Some(ref filter) => Some(filter.clone()),

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:126         },

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:127         run_ignored: config.run_ignored,

                                                                                                                            ...

/home/travis/build/servo/servo/.cargo/registry/src/github.com-88ac128001ac3a9a/compiletest_rs-0.0.11/src/compiletest.rs:122:5: 133:6 help: run `rustc --explain E0063` to see a detailed explanation

error: aborting due to 2 previous errors

Could not compile `compiletest_rs`.
@nox nox force-pushed the rustup branch from d807fd8 to 019f1ec Mar 19, 2016
@nox
Copy link
Member

nox commented Mar 20, 2016

@bors-servo r=larsbergstrom,manishearth

Updated the manifest.

@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2016

📌 Commit 7c22066 has been approved by larsbergstrom,manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2016

Testing commit 7c22066 with merge ea46564...

bors-servo added a commit that referenced this pull request Mar 20, 2016
Update to Rust 2016-03-18.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10076)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2016

💔 Test failed - status-appveyor

@Manishearth
Copy link
Member

Manishearth commented Mar 20, 2016

So it begins...

   Running `rustc C:/projects/servo/.cargo\\registry\\src\\github.com-88ac128001ac3a9a\\winapi-0.2.5\\src\\lib.rs --crate-name winapi --crate-type lib -g -C metadata=a0bf8c84304ddc12 -C extra-filename=-a0bf8c84304ddc12 --out-dir C:/projects/servo/target\\debug\\deps --emit=dep-info,link -L dependency=C:/projects/servo/target\\debug\\deps -L dependency=C:/projects/servo/target\\debug\\deps --cap-lints allow`
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:148:13: 150:14 error: duplicate definitions with name `item`: [E0201] 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:148             pub unsafe fn $variant(&self) -> &$fieldtype { 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:149                 ::std::mem::transmute(&self.$field)
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:150             } 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\commctrl.rs:2496:1: 2496:59 note: in this expansion of UNION! (defined in C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs) 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:148:13: 150:14 help: run `rustc --explain E0201` to see a detailed explanation 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:148:13: 150:14 note: conflicting definition is here: 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:148             pub unsafe fn $variant(&self) -> &$fieldtype { 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:149                 ::std::mem::transmute(&self.$field)
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:150             } 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\commctrl.rs:2503:1: 2503:59 note: in this expansion of UNION! (defined in C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs) 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:152:13: 154:14 error: duplicate definitions with name `item_mut`: [E0201] 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:152             pub unsafe fn $variantmut(&mut self) -> &mut $fieldtype { 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:153                 ::std::mem::transmute(&mut self.$field)
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:154             } 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\commctrl.rs:2496:1: 2496:59 note: in this expansion of UNION! (defined in C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs) 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:152:13: 154:14 help: run `rustc --explain E0201` to see a detailed explanation 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:152:13: 154:14 note: conflicting definition is here: 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:152             pub unsafe fn $variantmut(&mut self) -> &mut $fieldtype { 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:153                 ::std::mem::transmute(&mut self.$field)
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs:154             } 
C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\commctrl.rs:2503:1: 2503:59 note: in this expansion of UNION! (defined in C:/projects/servo/.cargo\registry\src\github.com-88ac128001ac3a9a\winapi-0.2.5\src\macros.rs) 

cc @retep998

@retep998
Copy link

retep998 commented Mar 20, 2016

Just a case of rust-lang/rust#32247
Please update your winapi to avoid this issue.

@nox
Copy link
Member

nox commented Mar 20, 2016

@bors-servo r=larsbergstrom,manishearth

Bumped winapi.

@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2016

📌 Commit 6f679c1 has been approved by larsbergstrom,manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2016

Testing commit 6f679c1 with merge aa35d77...

bors-servo added a commit that referenced this pull request Mar 20, 2016
Update to Rust 2016-03-18.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10076)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2016

@bors-servo bors-servo merged commit 6f679c1 into master Mar 20, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@nox nox deleted the rustup branch Mar 20, 2016
frewsxcv added a commit to servo/webrender that referenced this pull request Mar 21, 2016
bors-servo added a commit to servo/webrender that referenced this pull request Mar 21, 2016
Update nightly Rust version to keep in sync with Servo.

Rustup happened in servo/servo#10076
jdm added a commit to servo/webrender_traits that referenced this pull request Mar 21, 2016
Rustup happened in servo/servo#10076
bors-servo added a commit to servo/webrender_traits that referenced this pull request Mar 21, 2016
Update rust nightly to keep in sync with Servo.

Rustup happened in servo/servo#10076
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

9 participants
You can’t perform that action at this time.