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

Compile deps with optimizations for benches #734

Merged
merged 1 commit into from Oct 27, 2014

Conversation

Projects
None yet
4 participants
@alexcrichton
Copy link
Member

alexcrichton commented Oct 21, 2014

Previously only the local package was erroneously compiled with optimizations.

Closes #728

Compile deps with optimizations for benches
Previously only the local package was erroneously compiled with optimizations.

Closes #728
@IvanUkhov

This comment has been minimized.

Copy link
Contributor

IvanUkhov commented Oct 25, 2014

@alexcrichton, is there anything I can help with to get it into the nightly builds?

@brson

This comment has been minimized.

Copy link

brson commented on 579971e Oct 27, 2014

r+

This comment has been minimized.

Copy link
Owner Author

alexcrichton replied Oct 27, 2014

@bors: retry

This comment has been minimized.

Copy link
Owner Author

alexcrichton replied Oct 27, 2014

@bors: retry

@bors

This comment has been minimized.

Copy link
Contributor

bors commented on 579971e Oct 27, 2014

saw approval from brson
at alexcrichton@579971e

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

merging alexcrichton/cargo/issue-728 = 579971e into auto-cargo

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

alexcrichton/cargo/issue-728 = 579971e merged ok, testing candidate = c3c6116

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

saw approval from brson
at alexcrichton@579971e

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

saw approval from brson
at alexcrichton@579971e

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

merging alexcrichton/cargo/issue-728 = 579971e into auto-cargo

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

alexcrichton/cargo/issue-728 = 579971e merged ok, testing candidate = 9d3a7bd

This comment has been minimized.

Copy link
Contributor

bors replied Oct 27, 2014

fast-forwarding master to auto-cargo = 9d3a7bd

bors added a commit that referenced this pull request Oct 27, 2014

auto merge of #734 : alexcrichton/cargo/issue-728, r=brson
Previously only the local package was erroneously compiled with optimizations.

Closes #728

bors added a commit that referenced this pull request Oct 27, 2014

auto merge of #734 : alexcrichton/cargo/issue-728, r=brson
Previously only the local package was erroneously compiled with optimizations.

Closes #728

bors added a commit that referenced this pull request Oct 27, 2014

auto merge of #734 : alexcrichton/cargo/issue-728, r=brson
Previously only the local package was erroneously compiled with optimizations.

Closes #728

@bors bors merged commit 579971e into rust-lang:master Oct 27, 2014

1 of 2 checks passed

continuous-integration/travis-ci The Travis CI build failed
Details
default all tests passed

@alexcrichton alexcrichton deleted the alexcrichton:issue-728 branch Oct 28, 2014

@IvanUkhov

This comment has been minimized.

Copy link
Contributor

IvanUkhov commented Oct 28, 2014

Thank you, @alexcrichton! Works beautifully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.