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

Bump to 1.22.0 and update boostrap compiler #44154

Merged
merged 5 commits into from
Sep 1, 2017

Conversation

alexcrichton
Copy link
Member

Time to get a new nightly!

fn clone(&self) -> [T; $N] {
*self
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Oh. This is going to make the reversal at #44045 more difficult (if the decision turns out to be "not close").

@@ -323,13 +323,9 @@
// `force_alloc_system` is *only* intended as a workaround for local rebuilds
// with a rustc without jemalloc.
// The not(stage0+msvc) gates will only last until the next stage0 bump
Copy link
Member

Choose a reason for hiding this comment

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

This comment can be deleted

@steveklabnik
Copy link
Member

@alexcrichton
Copy link
Member Author

r? @Mark-Simulacrum

@Mark-Simulacrum
Copy link
Member

Aside from the concern raised by @kennytm (#44154 (comment)) r=me.

@alexcrichton
Copy link
Member Author

Ok the current disposition is to close so it sounds like there's not going to be need for a revert, and if so I'll volunteer to do the revert myself.

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Aug 29, 2017

📌 Commit 1768563 has been approved by Mark-Simulacrum

@@ -320,16 +320,9 @@
// Always use alloc_system during stage0 since we don't know if the alloc_*
// crate the stage0 compiler will pick by default is enabled (e.g.
// if the user has disabled jemalloc in `./configure`).
// `force_alloc_system` is *only* intended as a workaround for local rebuilds
// with a rustc without jemalloc.
Copy link
Member

Choose a reason for hiding this comment

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

Please leave this comment in! Though if you feel like it, it could be corrected to "...for local rebuilds using a rustc with jemalloc to create a rustc without jemalloc."

Copy link
Member

Choose a reason for hiding this comment

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

Just to clarify my previous comment, only the // The not(stage0+msvc) gates will only last until the next stage0 bump line should be deleted 🙂

@aidanhs
Copy link
Member

aidanhs commented Aug 29, 2017

@bors r-

since

[00:35:28] error[E0599]: no method named `set_dest` found for type `mdbook::MDBook` in the current scope
[00:35:28]   --> /checkout/src/tools/rustbook/src/main.rs:59:32
[00:35:28]    |
[00:35:28] 59 |         Some(dest_dir) => book.set_dest(Path::new(dest_dir)),

and other bits and pieces

@arielb1 arielb1 added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Aug 29, 2017
@alexcrichton alexcrichton force-pushed the bump-bootstrap branch 2 times, most recently from 95a6b4f to 45d7224 Compare August 29, 2017 22:30
@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Aug 29, 2017

📌 Commit 45d7224 has been approved by Mark-Simulacrum

@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Aug 29, 2017

📌 Commit 076485f has been approved by Mark-Simulacrum

@kennytm
Copy link
Member

kennytm commented Aug 30, 2017

cc #43975 (that PR depends on it)

@steveklabnik
Copy link
Member

Given this has been sitting for almost a day, and some other PRs depend on it, and IIRC we need it for a nightly...

@bors: p=1

@steveklabnik
Copy link
Member

Oh wait, this is in the rollup. My bad! Not that it harms anything.

bors added a commit that referenced this pull request Aug 31, 2017
Rollup of 8 pull requests

- Successful merges: #44044, #44089, #44116, #44125, #44154, #44157, #44160, #44172
- Failed merges: #44162
@bors
Copy link
Contributor

bors commented Aug 31, 2017

⌛ Testing commit 076485f with merge 8368ca641567e6d41f7e22b406c1ba86e74c80c6...

@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Aug 31, 2017

📌 Commit 678f6da has been approved by Mark-Simulacrum

@kennytm
Copy link
Member

kennytm commented Aug 31, 2017

@alexcrichton 678f6da failed the CI with tidy error.

[00:03:09] tidy error: /checkout/src/libstd/lib.rs:330: platform-specific cfg: cfg(all(
[00:03:09]     not(all(stage0, target_env = "msvc")),
[00:03:09]     any(stage0, feature = "force_alloc_system")))
[00:03:09] some tidy checks failed

@alexcrichton
Copy link
Member Author

Simulacrum

@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Aug 31, 2017

📌 Commit 731c095 has been approved by Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Aug 31, 2017

⌛ Testing commit 731c09578ef8a720aa8a3fa1fdb9e988ceac2e66 with merge 57059b7a2f8739e50ac459eee4916881b279034d...

@bors
Copy link
Contributor

bors commented Aug 31, 2017

💔 Test failed - status-appveyor

@steveklabnik
Copy link
Member

error: linking with `link.exe` failed: exit code: 1120
  |
  = note: "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\bin\\amd64\\link.exe" "/NOLOGO" "/NXCOMPAT" "/LIBPATH:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-sysroot\\lib\\rustlib\\x86_64-pc-windows-msvc\\lib" "C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\x86_64-pc-windows-msvc\\release\\deps\\term-91aff61c22a4f407.0.o" "/OUT:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\x86_64-pc-windows-msvc\\release\\deps\\term-91aff61c22a4f407.dll" "/DEF:C:\\Users\\appveyor\\AppData\\Local\\Temp\\1\\rustc.U0oBp9mzWqg8\\lib.def" "C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\x86_64-pc-windows-msvc\\release\\deps\\term-91aff61c22a4f407.crate.metadata.o" "C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\x86_64-pc-windows-msvc\\release\\deps\\term-91aff61c22a4f407.crate.allocator.o" "/OPT:REF,ICF" "/DEBUG" "/LIBPATH:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\x86_64-pc-windows-msvc\\release\\deps" "/LIBPATH:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\release\\deps" "/LIBPATH:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-sysroot\\lib\\rustlib\\x86_64-pc-windows-msvc\\lib" "kernel32.lib" "/LIBPATH:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-sysroot\\lib\\rustlib\\x86_64-pc-windows-msvc\\lib" "std-488c7c40c2087fcf.dll.lib" "C:\\Users\\appveyor\\AppData\\Local\\Temp\\1\\rustc.U0oBp9mzWqg8\\libcompiler_builtins-2f642fd8dbd70770.rlib" "advapi32.lib" "ws2_32.lib" "userenv.lib" "shell32.lib" "libcmt.lib" "/DLL" "/IMPLIB:C:\\projects\\rust\\build\\x86_64-pc-windows-msvc\\stage0-test\\x86_64-pc-windows-msvc\\release\\deps\\term-91aff61c22a4f407.dll.lib"
  = note:    Creating library C:\projects\rust\build\x86_64-pc-windows-msvc\stage0-test\x86_64-pc-windows-msvc\release\deps\term-91aff61c22a4f407.dll.lib and object C:\projects\rust\build\x86_64-pc-windows-msvc\stage0-test\x86_64-pc-windows-msvc\release\deps\term-91aff61c22a4f407.dll.exp
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_alloc referenced in function __rust_alloc
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_oom referenced in function __rust_oom
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_dealloc referenced in function __rust_dealloc
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_usable_size referenced in function __rust_usable_size
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_realloc referenced in function __rust_realloc
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_alloc_zeroed referenced in function __rust_alloc_zeroed
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_alloc_excess referenced in function __rust_alloc_excess
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_realloc_excess referenced in function __rust_realloc_excess
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_grow_in_place referenced in function __rust_grow_in_place
          term-91aff61c22a4f407.crate.allocator.o : error LNK2019: unresolved external symbol __rg_shrink_in_place referenced in function __rust_shrink_in_place
          C:\projects\rust\build\x86_64-pc-windows-msvc\stage0-test\x86_64-pc-windows-msvc\release\deps\term-91aff61c22a4f407.dll : fatal error LNK1120: 10 unresolved externals

I think there may still be bugs preventing its removal..
@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Sep 1, 2017

📌 Commit 9e5a7e9 has been approved by Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Sep 1, 2017

⌛ Testing commit 9e5a7e9 with merge 5289948b56acb02c9ce5f202f6798e5750405092...

@bors
Copy link
Contributor

bors commented Sep 1, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Sep 1, 2017

dist-powerpc-linux failed when compiling cargo, legit (libgit2 version mismatch?).

[01:12:37]    Compiling git2 v0.6.7
[01:12:39] error[E0308]: mismatched types
[01:12:39]    --> /cargo/registry/src/github.com-1ecc6299db9ec823/git2-0.6.7/src/blob.rs:117:53
[01:12:39]     |
[01:12:39] 117 |             let res = ((*self.raw).write)(self.raw, buf.as_ptr() as *const i8, buf.len());
[01:12:39]     |                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^ expected u8, found i8
[01:12:39]     |
[01:12:39]     = note: expected type `*const u8`
[01:12:39]                found type `*const i8`
[01:12:39] 
[01:12:40] error: aborting due to previous error
[01:12:40] 
[01:12:40] error: Could not compile `git2`.

@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Sep 1, 2017

📌 Commit 271c63c has been approved by Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Sep 1, 2017

⌛ Testing commit 271c63c with merge f861b6e...

bors added a commit that referenced this pull request Sep 1, 2017
Bump to 1.22.0 and update boostrap compiler

Time to get a new nightly!
@bors
Copy link
Contributor

bors commented Sep 1, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing f861b6e to master...

@bors bors merged commit 271c63c into rust-lang:master Sep 1, 2017
@alexcrichton alexcrichton deleted the bump-bootstrap branch September 1, 2017 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants