Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

rust 1.3.0 #44120

Closed
wants to merge 1 commit into from
Closed

rust 1.3.0 #44120

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 18, 2015

No description provided.

@ghost ghost mentioned this pull request Sep 18, 2015
@ghost
Copy link
Author

ghost commented Sep 18, 2015

Both Travis CI builds failed with "The job exceeded the maximum time limit for jobs, and has been terminated."

@stouset
Copy link
Contributor

stouset commented Sep 18, 2015

Rust takes a long time to compile. How would one circumvent that?

@DomT4
Copy link
Contributor

DomT4 commented Sep 19, 2015

There will be a Jenkins CI build done soonish which doesn't have such a limited timeout.

@MikeMcQuaid
Copy link
Member

Also if there's any way to speed up the build that would be handy too.

@DomT4
Copy link
Contributor

DomT4 commented Sep 19, 2015

Merged in aea0f1d. Thank you for your contribution to Homebrew @dtrebbien; we appreciate it! 😺

@DomT4 DomT4 closed this in aea0f1d Sep 19, 2015
@ghost
Copy link
Author

ghost commented Sep 19, 2015

Just to follow up on reducing build time, I asked on #rust-internals IRC and was told that build time is a known issue, but there is currently no master plan for fixing it.

One suggestion was to use an external LLVM rather than having rust build its own LLVM. Testing this approach out this morning, I see that the build time is actually slightly longer on my MacBook Pro when using Homebrew's LLVM (i.e. external LLVM):
32.0 minutes with in-tree LLVM (https://github.com/rust-lang/llvm) versus 32.3, 33.7 minutes using Homebrew's LLVM 3.6.2.

As noted at rust-lang/llvm#21, Rust's LLVM has some additional optimizations.

@ghost
Copy link
Author

ghost commented Sep 19, 2015

PR #44138 adds a --with-llvm option

@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants