Skip to content

Rollup of 5 pull requests #28820

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

Merged
merged 23 commits into from
Oct 3, 2015
Merged

Rollup of 5 pull requests #28820

merged 23 commits into from
Oct 3, 2015

Conversation

Ariel Ben-Yehuda and others added 23 commits September 24, 2015 17:02
Currently the compiler supports the ability to emit multiple output types as
part of one compilation (e.g. asm, LLVM IR, bytecode, link, dep-info, etc). It
does not, however, support the ability to customize the output filename for each
of these output types. The `-o` flag is ignored if multiple emit types are
specified (and the compiler emits a warning about this).

Normally this doesn't matter too much, but in the case of `dep-info` it can lead
to a number of problems (e.g. see rust-lang#28716). By allowing customization of the
output filename for each emit type we're able to solve the problems in that
issue.

This commit adds support for the `--emit` option to the compiler to look like:

    rustc foo.rs --emit dep-info=.deps/foo.d,link

This indicates that the `dep-info` output type will be placed at `.deps/foo.d`
and the `link` output type will otherwise be determined via the `--out-dir` and
`-o` flags.

Closes rust-lang#28716
Previously only keyup event was looked at, which meant that pasting, cutting and
otherwise changing the input without typing would not catch any updates to the
search query.
This helps protect against files being deleted to ensure that `make` won't emit
errors.

Closes rust-lang#28735
This PR closes out rust-lang#28716 and rust-lang#28735 by making two changes to the compiler:

1. The `--emit` flag to the compiler now supports the ability to specify the output file name of a partuclar emit type. For example `--emit dep-info=bar.d,asm=foo.s,link` is now accepted.
2. The dep-info emission now emits a dummy target for all input file names to protect against deleted files.
It's not really clear here, since the example is rendered, where it
starts and ends. So let's use <hr>s to split it up.
…docs, r=brson

Our docs were very basic for the various versions of from_utf8, so
this commit beefs them up.

It also improves docs for the &str variant's error, Utf8Error.
It's not really clear here, since the example is rendered, where it
starts and ends. So let's use <hr>s to split it up.
@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Oct 3, 2015

📌 Commit 92cff9f has been approved by steveklabnik

@bors
Copy link
Collaborator

bors commented Oct 3, 2015

⌛ Testing commit 92cff9f with merge 16331f1...

bors added a commit that referenced this pull request Oct 3, 2015
@bors bors merged commit 92cff9f into rust-lang:master Oct 3, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 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.

10 participants