-
Notifications
You must be signed in to change notification settings - Fork 43
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
Fails to build #16
Comments
@jumpwah thanks for reporting this bug, @cburgdorf is gonna take a look as soon as possible! |
Thanks @PascalPrecht, I understand that it's close to holidays, so no rush. |
Similar issues today: Building failed with docopt macro errors (I'm assuming cause it was an older version in
So I added a
I'm using nightly |
There are actually quite a few changes that need to be made to get clog to compile (not that I have got it to compile though) than what's on master here (because obviously rust has updated a lot in the past few months, so like everything, or a lot of things, are super outdated). I've been learning rust in the meantime, and I've been hoping to fix this issue myself, just haven't had the time to focus on this! However, I will hopefully get time to look into this more in the coming weeks, or otherwise at least before rust 1.0. Of course, you can obviously go ahead and fix this yourself if you want. It seems like you know a lot more rust than me anyway. :) |
Also, I forgot to update here, but for any other readers on arch linux, I think the rust-nightly-bin AUR package now provides cargo. |
I can confirm that it does, that's what I've been using for a while now ;) I'll take a look at the code tomorrow and see if can do some updating. I'll
|
Oh btw, I'm not owner nor collaborator here, I'm the one who originally opened the issue. Just to clarify. |
ah cool thanks, its hard to tell since I'm replying via email :P My
|
Hey there, I recently started a branch to port it to latest rust but it's not finished yet and I haven't found the time to continue with it yet. I can push the branch if you are interested to help. |
Yep! :) However, any of my help will only be very basic at the moment. I haven't studied the source code yet.. |
Ok, here it is. But don't expect too much. It's really just a start ;) |
Awesome, you still figured out a lot of the changes, so it's quite a lot already. Does this compile (for you)? I would try myself, but can't right now. I will try in a couple of days. Also, if you don't mind, what rust version are you using (for reference)? |
Nope, not yet. Just went down the rabbit whole for a while but stopped before I got it to compile ;) |
Cool, I'll hopefully at least try it out and report back (with the errors) in a couple of days. If I find time, I maybe could also help with descending into the depths of the rabbit hole but not sure yet. Thanks for your work. :) |
I just submitted a PR that compiles and works in case y'all want to check it out. I ran it on the fork and it took 7ms...since I don't have any prior knowledge I'm not sure if that's a |
Cool I won't bother with the update branch then. :) |
My versions, installed about a day (24 hours) ago:
If it matters, I'm on arch linux, using the rust-nightly-bin and cargo-nightly-bin AUR packages.
cargo build
in this repository gives (what looks like) syntax errors when attempting to build docopt. Their latest commits and release (0.6.16) fixes this.So
cargo update -p docopt
, and thencargo build --verbose
results in the following error:So add in a semicolon at the end of line 41 in src/main.rs, and then
cargo build --verbose
results in the following error:This is where I decided to stop and open this issue, because I have little to no knowledge of rust yet (just started).
The text was updated successfully, but these errors were encountered: