mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Review Request 59993: Removed need for patched http_parser version.
Date Mon, 12 Jun 2017 09:31:07 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Greg Mann and Vinod Kone.

Repository: mesos


'http_parser.h' defines a C-style enum 'flags' which clashes with e.g.,
a namespace name in stout. In '7c6a750a85becabbaa5c118c3735020400f11c39'
we attempted to fix this by directly patching 'http_parser.h'. This fix
broken support for building with unbundled http_parser dependency.

In this patch we remove the requirement to patch http_parser and instead
directly rename http_parser's 'flags' variable when the defining header
is included. This currently is just a single location.


  3rdparty/libprocess/src/decoder.hpp 31e8851d6882648937f3e0c37192131f8b057a89 



Tested on internal CI on various configurations.


Benjamin Bannier

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message