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

crabtw's bindgen repo has ceased to exist #15

Closed
cillian64 opened this issue Feb 13, 2017 · 7 comments
Closed

crabtw's bindgen repo has ceased to exist #15

cillian64 opened this issue Feb 13, 2017 · 7 comments
Assignees

Comments

@cillian64
Copy link

cillian64 commented Feb 13, 2017

I cannot build current master because it depends on https://github.com/crabtw/rust-bindgen.git?rev=0433980286a5264e3d69214aa42e314c37238cf6. That repository has been deleted and the only crates.io release of rust-bindgen has been yanked.

To replicate this you would need to clear out your cargo crates registry and git checkouts or try to build on a clean install of cargo which does not have rust-bindgen cached.

The bindgen package by servo looks like a newer version of the same base, so I assume rsmpi needs porting to use this.

@bsteinb
Copy link
Collaborator

bsteinb commented Feb 13, 2017

Yes, I am aware. Let me see if I can whip something up.

@bsteinb bsteinb self-assigned this Feb 13, 2017
@bsteinb
Copy link
Collaborator

bsteinb commented Feb 13, 2017

So, I got this working a few days (maybe weeks) ago, but never pushed those changes. In the meantime that version of the libbindgen crate (as it was called back then) has been yanked from crates.io. The current version of bindgen on crates.io again does not work for rsmpi (rust-lang/rust-bindgen#509). I will push a working version using libbindgen from the servo/rust-bindgen git repo into a branch for now.

@bsteinb
Copy link
Collaborator

bsteinb commented Feb 13, 2017

I have added a branch bindgen-hack that can be built for now. Not all CI tests pass at the moment. I cannot reproduce the locally at the moment, will have a look at it later.

@cillian64
Copy link
Author

This branch seems to work correctly for me, thank-you!

@Rufflewind
Copy link
Contributor

The aforementioned bindgen bug has been fixed and they have a new release now. Is there anything else that's holding back this issue?

@bsteinb
Copy link
Collaborator

bsteinb commented Mar 17, 2017

I have updated the bindgen-hack branch to use the current version of bindgen on crates.io. However I currently still have to work around this issue. I will hold back master until this is solved.

@bsteinb
Copy link
Collaborator

bsteinb commented May 23, 2017

It does not look like the bindgen issue is getting fixed soon, so I have marked the workaround with FIXMEs and fast-forwarded the master branch to include these changes.

@bsteinb bsteinb closed this as completed May 23, 2017
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

No branches or pull requests

3 participants