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

options: Remove the linking-related options. #1288

Merged
merged 1 commit into from
Apr 2, 2018

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Mar 31, 2018

They do nothing, and are effectively superseded by --raw-line and friends.

They also tend to confuse people.

Closes #104

They do nothing, and are effectively superseded by --raw-line and friends.

They also tend to confuse people.

Closes rust-lang#104
@emilio
Copy link
Contributor Author

emilio commented Mar 31, 2018

r? @fitzgen or @pepyakin

@pepyakin
Copy link
Contributor

pepyakin commented Apr 2, 2018

@bors-servo r+

@bors-servo
Copy link

📌 Commit 8e3e585 has been approved by pepyakin

@bors-servo
Copy link

⌛ Testing commit 8e3e585 with merge 0fd5e9e...

bors-servo pushed a commit that referenced this pull request Apr 2, 2018
options: Remove the linking-related options.

They do nothing, and are effectively superseded by --raw-line and friends.

They also tend to confuse people.

Closes #104
@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: pepyakin
Pushing 0fd5e9e to master...

@bors-servo bors-servo merged commit 8e3e585 into rust-lang:master Apr 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants