----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35360/#review87970 -----------------------------------------------------------
This patch breaks my ability to run the python-tox-test correctly. With this patch, when I run 'make test' (or ctest -R python-tox-test), tox attempts to pull down the proton sources from apache. This should not happen when running 'make test or ctest' - tox should use the local sources in those cases. proton-c/bindings/python/setup.py <https://reviews.apache.org/r/35360/#comment140376> I didn't realize this call actually runs swig. Can you drop a comment? - Kenneth Giusti On June 11, 2015, 3:03 p.m., Flavio Percoco wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35360/ > ----------------------------------------------------------- > > (Updated June 11, 2015, 3:03 p.m.) > > > Review request for qpid, Gordon Sim and Rafael Schloming. > > > Repository: qpid-proton-git > > > Description > ------- > > The patch allows python-qpid-proton for generating the `cproton_wrap.c` > file when creating the distribution package. Since this file is already > cross-platform, it should be safe to ship it as part of the > python-qpid-proton package. > > This allows us for building the bindings even when swig is not > available. The setup.py process will try to generate a new > cproton_wrap.c everytime but it'll fallback to the one distributed with > the package if swig is not available. > > > Note that this will be done just in the platforms where bundling proton-c > is supported. The reason being that, in the other platforms, w can't verify > whether the installed qpid-proton version is the same that was used to > generated these files > > > Diffs > ----- > > proton-c/bindings/python/setup.py 84b5a66 > > Diff: https://reviews.apache.org/r/35360/diff/ > > > Testing > ------- > > > Thanks, > > Flavio Percoco > >
