Skip to content

Update README local run command to remove syspath - #3593

Merged
bors merged 1 commit into
masterfrom
unknown repository
Dec 30, 2018
Merged

Update README local run command to remove syspath#3593
bors merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Dec 29, 2018

Copy link
Copy Markdown

Since #3257 was reverted, including the sysroot in RUSTFLAGS gives the
error Option 'sysroot' given more than once

Since #3257 was reverted, including the sysroot in RUSTFLAGS gives the
error `Option 'sysroot' given more than once`
@phansch phansch added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 30, 2018
@phansch

phansch commented Dec 30, 2018

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Dec 30, 2018

Copy link
Copy Markdown
Contributor

📌 Commit f4cf82c has been approved by phansch

@phansch

phansch commented Dec 30, 2018

Copy link
Copy Markdown
Contributor

also cc #3594

@bors

bors commented Dec 30, 2018

Copy link
Copy Markdown
Contributor

⌛ Testing commit f4cf82c with merge 61bfb39...

bors added a commit that referenced this pull request Dec 30, 2018
Update README local run command to remove syspath

Since #3257 was reverted, including the sysroot in RUSTFLAGS gives the
error `Option 'sysroot' given more than once`
@o01eg

o01eg commented Dec 30, 2018

Copy link
Copy Markdown
Contributor

It should works after #3594 because rustc --print sysroot was left intact.

@bors

bors commented Dec 30, 2018

Copy link
Copy Markdown
Contributor

☀️ Test successful - status-appveyor, status-travis
Approved by: phansch
Pushing 61bfb39 to master...

@bors
bors merged commit f4cf82c into rust-lang:master Dec 30, 2018
@ghost
ghost deleted the readme-syspath-2 branch January 6, 2019 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants