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 5 pull requests #47225

Merged
merged 12 commits into from
Jan 6, 2018
Merged

Rollup of 5 pull requests #47225

merged 12 commits into from
Jan 6, 2018

Conversation

frewsxcv and others added 12 commits December 24, 2017 11:29
- Update example in ‘security’ section to use hard links, like the
  linked securityvulns.com example.
- Weaken language on symbolic links – indicate behavior is
  platform-specific

Fixes rust-lang#43617.
The function was added as a wrapper to handle compatibility with older
LLVM versions that we no longer support, so it can be removed.

Refs rust-lang#46437
…ietMisdreavus

Minor rewrite of env::current_exe docs; clarify symlinks.

- Update example in ‘security’ section to use hard links, like the
  linked securityvulns.com example.
- Weaken language on symbolic links – indicate behavior is
  platform-specific

Fixes rust-lang#43617.
[unix] Don't clone command-line args on startup

Fixes part of rust-lang#47164 and simplifies the `args` code on non-Apple Unix platforms.

Note: This could change behavior for programs that use both `std::env::args` *and* unsafe code that mutates `argv` directly.  However, these programs already behave differently on different platforms.  The new behavior on non-Apple platforms is closer to the existing behavior on Apple platforms.
Remove some outdated LLVM-related code

Ticks two boxes on rust-lang#46437
…acrum

Clarify appending behavior of 'io::Read::read_to_string()'.

A simple doc clarification.
@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@rust-highfive
Copy link
Contributor

Some changes occurred in HTML/CSS.

cc @GuillaumeGomez

@bors
Copy link
Collaborator

bors commented Jan 6, 2018

📌 Commit 35d1555 has been approved by GuillaumeGomez

@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@shepmaster shepmaster added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 6, 2018
@bors
Copy link
Collaborator

bors commented Jan 6, 2018

⌛ Testing commit 35d1555 with merge 8d370ec...

bors added a commit that referenced this pull request Jan 6, 2018
Rollup of 5 pull requests

- Successful merges: #46987, #47165, #47173, #47202, #47216
- Failed merges:
@bors
Copy link
Collaborator

bors commented Jan 6, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: GuillaumeGomez
Pushing 8d370ec to master...

@bors bors merged commit 35d1555 into rust-lang:master Jan 6, 2018
@GuillaumeGomez GuillaumeGomez deleted the rollup branch January 6, 2018 14:11
@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 S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.