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

Rollup of 6 pull requests #30024

Closed
wants to merge 14 commits into from
Closed

Conversation

Dylan McKay and others added 14 commits October 27, 2015 23:23
Previously the file was not regenrated upon modification of src/rustllvm or others.

Now it will be rebuilt if `src/llvm` or `src/rustllvm` is touched.

Also added *.rs rule to 'clean' rule so that it is removed upon 'make
clean'.
It was introduced with the change that made copy opt-in. The
implementation gives a warning, because the struct contains a raw
pointer.
Previously the file was not regenrated upon modification of `src/rustllvm` or others.

Now it will be rebuilt if `src/llvm` or `src/rustllvm` is touched.

Also added *.rs rule to 'clean' rule so that it is removed upon 'make
clean'.

Fixes rust-lang#28614.
…sakis

It was introduced with the change that made copy opt-in. The
implementation gives a warning, because the struct contains a raw
pointer.
…r, r=alexcrichton

Good candidate for a rollup, this one.
`unit` was in code formatting, which is wrong, since it's not actual code. The correct code is `()`.
@rust-highfive
Copy link
Contributor

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@Manishearth
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Collaborator

bors commented Nov 24, 2015

📌 Commit 0a3b429 has been approved by Manishearth

@Manishearth Manishearth assigned Manishearth and unassigned brson Nov 24, 2015
@bors
Copy link
Collaborator

bors commented Nov 24, 2015

⌛ Testing commit 0a3b429 with merge 41d444b...

@bors
Copy link
Collaborator

bors commented Nov 24, 2015

💔 Test failed - auto-linux-64-nopt-t

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants