----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35214/#review87046 -----------------------------------------------------------
Ship it! Makes sense to me to avoid requiring a dependency for an optional component and where a reasonable replacement can be provided as it is here. - Gordon Sim On June 8, 2015, 3:24 p.m., Flavio Percoco wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35214/ > ----------------------------------------------------------- > > (Updated June 8, 2015, 3:24 p.m.) > > > Review request for qpid and Rafael Schloming. > > > Repository: qpid-proton-git > > > Description > ------- > > Instead of relying on libuuid for uuid generation, let proton-c have a > built-in uuid4 generator as a fallback when no uuid generator is present in > the system. > > Eventually, proton-c could simply rely on the built-in generator but lets > give this some time to mature and test. > > > Diffs > ----- > > proton-c/CMakeLists.txt b534e86 > proton-c/bindings/python/setup.py b255580 > proton-c/src/platform.c 8f8ac5f > > Diff: https://reviews.apache.org/r/35214/diff/ > > > Testing > ------- > > > Thanks, > > Flavio Percoco > >
