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

falcon fails to build #1351

Closed
endeav0r opened this issue Apr 13, 2021 · 3 comments
Closed

falcon fails to build #1351

endeav0r opened this issue Apr 13, 2021 · 3 comments

Comments

@endeav0r
Copy link

endeav0r commented Apr 13, 2021

Sorry, accidentally hit the create-new-issue button before I was done here. There is already an issue open for this here: #1198

Crate name: Falcon
Build failure link: https://docs.rs/crate/falcon/0.5.1/builds/375762

Additional details:
Falcon does not build correctly against packaged capstone4.

We have a github workflow here: https://github.com/falconre/falcon/blob/master/.github/workflows/tests.yml that successfully builds falcon. In order to do this, we download capstone, build the bindings manually, and install them.

          wget https://github.com/aquynh/capstone/archive/4.0.2.tar.gz
          tar xf 4.0.2.tar.gz
          cd capstone-4.0.2
          make -j 8
          sudo make install

I'm not very familiar with the internal infrastructure of docs.rs, and understand this may be too intensive of a step for a single package. If that's the case, we'll host the docs externally. However, I'm a huge fan of docs.rs, and if we can make it work I'd love to do that.

@jyn514
Copy link
Member

jyn514 commented Apr 13, 2021

This is rust-lang/rust#84162. I answered before the edits.

@jyn514
Copy link
Member

jyn514 commented Apr 14, 2021

@endeav0r we don't allow network access in the build environment. You could either avoid building capstone in the build.rs if DOCS_RS is set in the environment (https://docs.rs/about/builds#detecting-docsrs) or bundle the sources with the crates.io tarball.

@jyn514
Copy link
Member

jyn514 commented Apr 15, 2021

@endeav0r I'm going to close this as a duplicate of #1198, feel free to add more info there.

@jyn514 jyn514 closed this as completed Apr 15, 2021
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

2 participants