Skip to content
This repository has been archived by the owner on Oct 18, 2019. It is now read-only.

Rust 1.2 build fix #17

Merged
merged 1 commit into from
Sep 2, 2015
Merged

Rust 1.2 build fix #17

merged 1 commit into from
Sep 2, 2015

Conversation

potatosalad
Copy link
Contributor

From best I can tell about the error mentioned in #14, cargo didn't like that we were trying to build a release version of the application prior to building a debug version. I actually don't know, but that appears to have fixed the issue for rust 1.2. It may not be necessary, but I also completely removed ~/.cargo after upgrading to 1.2 just to make sure I was starting with a clean slate.

image

As an interesting side note, rust 1.2 has a much, much faster regex implementation (from 19.1 seconds in version 1.0 down to 1.7 seconds in version 1.2).

Rust w/ Substring 2.4s 2.3s
Rust w/ Regular Expression 19.1s 1.7s

@dimroc
Copy link
Owner

dimroc commented Sep 2, 2015

My issue was related to faulty CA certificates not playing well with cargo:
rust-lang/cargo#1420

I've now got it running.

dimroc added a commit that referenced this pull request Sep 2, 2015
@dimroc dimroc merged commit 39346d6 into dimroc:master Sep 2, 2015
@potatosalad potatosalad mentioned this pull request Sep 7, 2015
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.

2 participants