-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1566/#review2386
-----------------------------------------------------------
src/software-dbus/LMI_SoftwareIdentityResourceProvider.c
<
http://reviewboard-openlmi.rhcloud.com/r/1566/#comment1291>
I'm concerned about error message reporting. PackageKit may offer detailed
description why some task failed. This is one of the major issues of python software
provider - it does not provide any information in error cases. Please do not avoid using
last *error* argument.
src/software-dbus/LMI_SoftwareIdentityResourceProvider.c
<
http://reviewboard-openlmi.rhcloud.com/r/1566/#comment1292>
Let's check for errors (last argument).
- Michal Minar
On March 27, 2014, 4:30 p.m., Peter Schiffer wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1566/
-----------------------------------------------------------
(Updated March 27, 2014, 4:30 p.m.)
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
-------
Software-dbus: Implemented SoftwareIdentityResourceProvider
Diffs
-----
src/software-dbus/LMI_Software.h 916ca5f1c95646bf7969ef74b8aff42ecee281b2
src/software-dbus/LMI_SoftwareIdentityResourceProvider.c
d173b484d3745455d507d1557a17f3191635d03c
Diff:
http://reviewboard-openlmi.rhcloud.com/r/1566/diff/
Testing
-------
Thanks,
Peter Schiffer