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

configure: detect an incorrectly-built Hyper #8001

Closed
jsha opened this issue Nov 13, 2021 · 0 comments
Closed

configure: detect an incorrectly-built Hyper #8001

jsha opened this issue Nov 13, 2021 · 0 comments

Comments

@jsha
Copy link
Contributor

@jsha jsha commented Nov 13, 2021

I did this

In Hyper directory:

cargo build

In curl directory:

./configure --with-hyper=$HOME/rust/hyper/

I expected the following

Success!

I got the following

configure: error: --with-hyper but hyper was not found. See docs/HYPER.md.

The problem is that I didn't build Hyper with the appropriate feature flags. The message is confusing because Hyper did exist and was in the right place, but didn't have the correct symbols. We should add a check to configure that the file (libhyper.a) exists in the given directory. If the file exists but the build fails, we can specifically say "hyper was found but it was built with the wrong flags. See docs/HYPER.md"

/cc @kevinburke

bagder added a commit that referenced this issue Nov 13, 2021
If hyper is indeed present in the specified directory but couldn't be
used to find the correct symbol, then offer a different error message to
better help the user understand the issue.

Sugggested-by: Jacob Hoffman-Andrews
Fixes #8001
@bagder bagder self-assigned this Nov 13, 2021
@bagder bagder closed this in 9492d33 Nov 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants