search

nodejs / http-parser

http request/response parser for c
MIT License
6.32k stars 1.53k forks source link

Fix 5 dependency issues in Makefile #488

Open vemakereporter opened 5 years ago

vemakereporter commented 5 years ago

Hi, I've fixed 4 dependencies missing and 1 dependency excessive reported. Those issues can cause incorrect results when http-parser is incrementally built. For example, any changes in "http-parser.h" will not cause "url_parser" to be rebuilt, which is incorrect. The changes in "http-parser.h" will cause an unnecessary rebuild of "test_fast". I've tested it on my computer, the fixed version worked as expected. Looking forward to your confirmation.

Thanks Vemake