Skip to content

Fix build failure on FreeBSD, due to useless annotations. #32188

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

Closed
wants to merge 1 commit into from
Closed

Fix build failure on FreeBSD, due to useless annotations. #32188

wants to merge 1 commit into from

Conversation

dcci
Copy link

@dcci dcci commented Mar 11, 2016

This patch tries to fix a failure I hit while trying to build rust on FreeBSD.

Example:

src/libcore/result.rs:253:54: 253:56 error: This stability annotation is useless
src/libcore/result.rs:253     Ok(#[stable(feature = "rust1", since = "1.0.0")] T),
                                                                               ^~
src/libcore/result.rs:257:55: 257:57 error: This stability annotation is useless
src/libcore/result.rs:257     Err(#[stable(feature = "rust1", since = "1.0.0")] E)
                                                                                ^~

I'm not terribly sure this is correct as this is my first contribution to rust, but still i thougth it was worth a try. Also, isn't the annotation highlighting the wrong piece? (T and E instead of #[stable ...]).

Thanks!

Davide

This stability annotation is useless
src/libcore/result.rs:257     Err(#[stable(feature = "rust1", since =
"1.0.0")] E)
@dcci dcci changed the title Fix build failure on FreeBSD, due to useless allocations. Fix build failure on FreeBSD, due to useless annotations. Mar 11, 2016
@sfackler
Copy link
Member

Those annotations are supposed to be there. You may be building with an out of date snapshot?

@dcci
Copy link
Author

dcci commented Mar 11, 2016

Hmm, not really, c9629d6

@sfackler
Copy link
Member

It looks like the most recent snapshot hasn't been built for freebsd, which would cause this error: https://github.com/rust-lang/rust/blob/master/src/snapshots.txt#L1

@dcci
Copy link
Author

dcci commented Mar 11, 2016

Thanks! I'm not terribly familiar with that -- is there something that can be done to get this moving forward?

@alexcrichton
Copy link
Member

Ah yeah these are currently required and we just need some new FreeBSD snapshots. It should be possible to check out the snapshot revision, and then run make snap-stage3, and then we can upload the resulting tarball.

@dcci
Copy link
Author

dcci commented Mar 11, 2016

@alexcrichton OK, I'm going to run that. Should I upload them somewhere and then open another pull request, or something else? Thanks!

@alexcrichton
Copy link
Member

Yeah feel free to just send me a link to the snapshot when it's done and I can upload it.

@alexcrichton
Copy link
Member

Thanks @dcci! This actually also ended up being independently produced by @andoriyu in #32197, and I've uploaded that snapshot so once the PR lands FreeBSD should be good to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants