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.

Sign up for GitHub

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

Jump to bottom

http-parser dependency improperly specified #19

Closed
tduehr opened this issue Oct 28, 2017 · 2 comments
Closed

http-parser dependency improperly specified #19

tduehr opened this issue Oct 28, 2017 · 2 comments

Comments

@tduehr
Copy link

tduehr commented Oct 28, 2017 •
edited
Loading

http-parser HEAD version is required. In v2.7.1 http_parser.h does not contain HTTP_STATUS_MAP.

upstream bug filed: nodejs/http-parser#394
@tduehr tduehr mentioned this issue Oct 28, 2017
Closed
@npmccallum
Copy link
Contributor

Thanks for this report. We'll fix this as soon as we get a new release of http-parser.

@npmccallum
Copy link
Contributor

@tduehr Where is the version improperly specified?

We check the header for the proper HTTP_STATUS_MAP define in configure.ac. Also, the Fedora and RHEL packages have the correct dependencies. I'm making a patch to clarify the README.md. If I've missed anything, please reopen this ticket.

npmccallum added a commit to npmccallum/tang that referenced this issue Jul 12, 2018
@npmccallum
Update docs to reflect minimum http-parser version
07a6230 
Fixes: latchset#19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tduehr @npmccallum