Compile fails

[sharkdp]

Compile fails

[BurntSushi]

@sharkdp Nah I think 1.16 is fine. I'll probably bump ripgrep's minimum Rust version in the next release to something similar. I just like to make sure that the bump happens purposefully and with an appropriate version bump.

[BurntSushi]

Your rustc is probably too old. What version are you using? The compile error you're getting indicates that it can't find the String::split_off method, which was added in Rust 1.16.

@sharkdp One thing I typically do is put an explicit Rust version in my .travis.yml so that I always know for sure what the minimum Rust version my tool compiles with.

[EricCrosson]

Awesome :) I'll see if I can compile after I update. Thanks for the pointers

[EricCrosson]

I currently see

rustc --version                                                                                                                                                                          
rustc 1.14.0 (e8a012324 2016-12-16)

[EricCrosson]

Updating info: latest update on 2017-06-08, rust version 1.18.0 (03fc9d622 2017-06-06)

Yessir, updating past Rust 1.16 fixed my problem. Do you know of a better way to update rust than running the uninstall script and then the rustup.sh installer again?

I am amused that travis is still building with the minimum version pegged at 1.12

[BurntSushi]

@EricCrosson The link I gave was to my own separate project, not this one. :-) This one just tests on current stable/beta/nightly.

Do you know of a better way to update rust than running the uninstall script and then the rustup.sh installer again?

The preferred method of managing Rust installations for Rust programmers is rustup. rustup itself is a tool that will let you install multiple Rust installations and switch between them easily. (For instances, I have every Rust version from 1.0 on upwards installed locally.)

[sharkdp]

@sharkdp One thing I typically do is put an explicit Rust version in my .travis.yml so that I always know for sure what the minimum Rust version my tool compiles with.

Thank you very much for the suggestion! It's implemented now. Also, there was already a comment in the README that refers to the rust 1.16 requirement.

Do you think it would be worth to backport to 1.12 (or similar)? My (unfounded) impression is, that most people use rustup instead of old versions that come with some distribution package manager (?).