On 07/10/14 06:50, David Smith wrote:
...
On the new pcpfans/dsmith/dev branch, there are 2 new commits:
- 44cb24d: This adds a new test (called '843') that can catch this bug.
Note that I based this test on an existing test, but I'm still not 100%
sure I updated the testsuite machinery properly. Also note that this new
test only tests the changes to src/python/pmda.c. I"m not really sure
how to test the changes to src/python/pmapi.c.
- 8e62465: This is the the python fix itself (that I posted to the list
earlier).
Thanks David.
I've cherry picked and confirmed ...
- qa/843 fails before the code change
- after the code change, debian builds are ok
- after the code change, qa/843 passes
- after the code change qa -g python passes
Integration of qa pieces looks OK to me also, except 843 probably should
be in the python group as well (in qa/group) and I'll fix this in my
next batch of commits.
Cheers, Ken.
|