Changes committed to git://oss.sgi.com/pcp/pcp.git dev
debian/changelog | 1 +
debian/control | 2 +-
debian/rules | 3 +++
qa/737 | 29 +++++++++++++++++++++++++++++
qa/737.out | 5 +++++
qa/group | 1 +
qa/src/GNUlocaldefs | 3 ++-
qa/src/test_pcp_time.python | 32 ++++++++++++++++++++++++++++++++
src/python/pcp/pmapi.py | 41 ++++++++++++++++++++++++++---------------
9 files changed, 100 insertions(+), 17 deletions(-)
commit 004d8a3e46fe4c56b0ea4db27c2c618b86b54f76
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date: Mon Feb 3 17:13:12 2014 +1100
Fix and test python wrappers for pmLocaltime and pmCtime
Several problems uncovered in the time-processing wrappers
of the python PMAPI module after attempting to use 'em for
the first time:
- size of 'struct tm' did not match modern reality (missing
the final two fields for Linux and BSD - sigsegv results);
- system time.mktime python method does not take 9 arguments
just one - fix the string-printing method for struct tm;
- system time.mktime takes year field as an actual year, and
not the number of years since 1900 - code didn't cater for
this difference to struct tm form returned from localtime.
- definition of the pmLocaltime return code was not correct;
- definition of first pmCtime parameter was not correct;
- code didn't cope with numeric input being passed in either
float/integer format - pythonic time-in-seconds is float.
- no memory allocated to hold the struct tm that pmLocaltime
returns - pointer passed in pointing off into lala-land;
- need to ensure a pointer to a ctypes c_long is passed into
both of these routines - caller might well pass int (e.g.
from a pmResult timestamp (struct timeval, tv_sec field).
Fixed the above and added qa test (boeing) 737 to verify it.
Some small cleanups as well, in the affected code - fixed a
couple of typos in comments, and removed an unused import
of struct_time from the system time module.
commit 5550ecb6fd843466df5e998ee406d684e6fa5cdd
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date: Mon Feb 3 14:00:21 2014 +1100
Make a changelog note about deb bug# fixed in next release
commit efc5d9fb0b1ba80165a57dfdff1faa2036b4c219
Author: Logan Rosen <logan@xxxxxxxxxx>
Date: Mon Feb 3 13:09:50 2014 +1100
Use autotools-dev to update config.{sub,guess} for new architectures
For example, we needed these updates in Ubuntu for the new arm64
and ppc64el architectures.
In Ubuntu, the attached patch was applied to achieve the following:
* Use autotools-dev to update config.{sub,guess} for new arches.
|