[lvm-devel] [PATCH 0/5] lvm2app & python bindings bug fixes

[Tony Asleson]

This patch set should resolve the following issues:

* Handle signed properties better

I added another flag on the properties structure to indicate if the property is
signed and added a int64_t signed_integer to the union.  Updated python binding
to build the correct representation.  I didn't get any feedback on this
approach in the bz, not sure if another approach would work better.

ref. https://bugzilla.redhat.com/show_bug.cgi?id=838257


* Unable to retrieve specific properties:

Any property that internally was defined as a string list was broken, ended up
being a missing flag.

ref. https://bugzilla.redhat.com/show_bug.cgi?id=1139920

Would appreciate someone taking a peek at these changes.

Thanks,
Tony

Tony Asleson (5):
  lvm2app: Handle property values which are signed
  python: Build correct python value for numerical property
  python: Check for value before constructing string property
  lvm2app: Correct missing string properties
  Python: Improve lv property test coverage

 lib/properties/prop_common.h |    5 +++-
 lib/report/columns.h         |    8 +++---
 lib/report/properties.c      |    3 +-
 lib/report/report.c          |    2 +
 liblvm/lvm2app.h             |    4 ++-
 liblvm/lvm_misc.c            |    1 +
 liblvm/lvm_prop.c            |    2 +-
 python/liblvm.c              |   17 +++++++++---
 test/api/python_lvm_unit.py  |   56 +++++++++++++++++++++++++++++++++++++++--
 9 files changed, 83 insertions(+), 15 deletions(-)