Skip to content
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

musl: use of undeclared type name libc::ucontext_t #325

Closed
posborne opened this issue Mar 20, 2016 · 5 comments
Closed

musl: use of undeclared type name libc::ucontext_t #325

posborne opened this issue Mar 20, 2016 · 5 comments

Comments

@posborne
Copy link
Member

x86_64-unknown-linux-musl should be added to CI. It is included in the posborne/rust-cross:x86 docker image. Discovered this while working on some fixup for the CI stuff. May be a libc issue.

=======================================================
TESTING VERSION: 1.7.0, TARGET: x86_64-unknown-linux-musl
=======================================================
/source/src/ucontext.rs:7:14: 7:30 error: use of undeclared type name `libc::ucontext_t` [E0412]
/source/src/ucontext.rs:7     context: libc::ucontext_t,
                                       ^~~~~~~~~~~~~~~~
/source/src/ucontext.rs:7:14: 7:30 help: run `rustc --explain E0412` to see a detailed explanation
@posborne
Copy link
Member Author

@kamalmarhubi Can you please label this as appropriate. I don't seem to have the ability to create new labels.

@kamalmarhubi
Copy link
Member

@posborne I think you should be able to do this now. I created a "team" and gave it admin access on the repo. Added some labels though!

@posborne
Copy link
Member Author

I started looking at things a bit -- there are some legit test problems in ioctl and a few other places. I started looking more at using ctest to do the types of checks we are doing in some places (some of this need goes away with moving contants to libc, but some will likely need to remain).

I ended up bumping into rust-lang/cargo#1430 and decided to take a break.

@fiveop
Copy link
Contributor

fiveop commented Apr 7, 2016

I made a patch for libc, that adds ucontext for linux-musl ( rust-lang/libc#255 ). That should resolve this.

@fiveop
Copy link
Contributor

fiveop commented Apr 15, 2016

musl does not implement the functions working on ucontext structs (http://wiki.musl-libc.org/wiki/Open_Issues#ucontext.h), so let's see how far the struct gets us.

@homu homu closed this as completed in #381 Jun 30, 2016
homu added a commit that referenced this issue Jun 30, 2016
musl target needs std::mem now

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

No branches or pull requests

3 participants