Message ID | 878sjz6r0u.fsf@guixSD.i-did-not-set--mail-host-address--so-tickle-me |
---|---|
State | Accepted |
Headers | show |
Series | [bug#38546,v5] Julia: Update to 1.3.1 | expand |
Context | Check | Description |
---|---|---|
cbaines/applying patch | fail | View Laminar job |
Nicolò Balzarotti <anothersms@gmail.com> writes: > Hi, > > I looked at it. THe patch is literally a one-line change[1] they make > to support RFC 3986. According to [2], the patch has been ported to > fedora http-parser. Our version is different form the one bundled in > libgit2, but deleting the corresponding line (see snippet) libgit2 tests > are now passing. However, this http-parser test is failing: > > *** http_parser_parse_url("http://hostname:/") "proxy empty port" test failed, unexpected rv 0 *** > > I opened an issue upstream here [3] so that if they fix it we can > continue unbundling it without having to patch it ourselves. > > Hope they decide to upstream it > > I'll send updates if there are news on the issue, Awesome, thank you very much for looking into it. I have no objections to taking the patches from the upstream PR [0] if that helps updating Julia. :-) 0: https://github.com/nodejs/http-parser/pull/483
On Thu, Mar 19, 2020 at 01:28:09PM +0100, Marius Bakke wrote: > Nicolò Balzarotti <anothersms@gmail.com> writes: > > > Hi, > > > > I looked at it. THe patch is literally a one-line change[1] they make > > to support RFC 3986. According to [2], the patch has been ported to > > fedora http-parser. Our version is different form the one bundled in > > libgit2, but deleting the corresponding line (see snippet) libgit2 tests > > are now passing. However, this http-parser test is failing: > > > > *** http_parser_parse_url("http://hostname:/") "proxy empty port" test failed, unexpected rv 0 *** > > > > I opened an issue upstream here [3] so that if they fix it we can > > continue unbundling it without having to patch it ourselves. > > > > Hope they decide to upstream it > > > > I'll send updates if there are news on the issue, > > Awesome, thank you very much for looking into it. I have no objections > to taking the patches from the upstream PR [0] if that helps updating > Julia. :-) > > 0: https://github.com/nodejs/http-parser/pull/483 Any news from upstream?
diff -u a/http_parser.c b/http_parser.c --- a/http_parser.c 2020-03-17 14:05:53.542832960 +0100 +++ b/http_parser.c 2020-03-17 14:05:21.810382406 +0100 @@ -2394,7 +2394,6 @@ case s_http_host_v6_zone_start: case s_http_host_v6_zone: - case s_http_host_port_start: case s_http_userinfo: case s_http_userinfo_start: return 1; default: