> On April 17, 2015, 2:16 p.m., Alan Conway wrote: > > Looks good (surprised there wasn't more). Run the entire test set before > > committing, we have bits of python in surprising places so best be sure.
Actually you shoulld run demos too. We badly, badly need to automate the demos. - Alan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33304/#review80455 ----------------------------------------------------------- On April 17, 2015, 1:17 p.m., Kenneth Giusti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33304/ > ----------------------------------------------------------- > > (Updated April 17, 2015, 1:17 p.m.) > > > Review request for qpid, Alan Conway, Gordon Sim, Rafael Schloming, and > Robbie Gemmell. > > > Bugs: PROTON-490 > https://issues.apache.org/jira/browse/PROTON-490 > > > Repository: qpid-proton-git > > > Description > ------- > > First of a series of patches, this patch shows the changes made by the > futurize tool when run on the proton-c/bindings directory. > > These changes preserve python2 compatibility while avoiding language features > deprecated by python3. These changes don't enable python3 support, they > merely eliminate the syntax/features that won't port. > > > Diffs > ----- > > proton-c/bindings/python/proton/__init__.py eeb7768 > proton-c/bindings/python/proton/reactor.py 467bb76 > proton-c/bindings/python/proton/utils.py b5ecdf7 > > Diff: https://reviews.apache.org/r/33304/diff/ > > > Testing > ------- > > Ran the unit tests - all pass including Jython-based tests. > > > Thanks, > > Kenneth Giusti > >
