On Čec. 25, 2014, 1:06 odp., Peter Schiffer wrote:
> src/software-dbus/sw-utils.c, lines 1015-1016
>
<
http://reviewboard-openlmi.rhcloud.com/r/1934/diff/1/?file=11341#file1134...
>
> Missing possiblity of listing disabled repos.
Good catch!
On Čec. 25, 2014, 1:06 odp., Peter Schiffer wrote:
> src/software-dbus/sw-utils.c, lines 1022-1023
>
<
http://reviewboard-openlmi.rhcloud.com/r/1934/diff/1/?file=11341#file1134...
>
> Why 2 lines instead of 1? This looks to me less intuitive.
Thanks for noticing.
- Michal
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1934/#review3162
-----------------------------------------------------------
On Čec. 25, 2014, 12:44 odp., Michal Minar wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1934/
-----------------------------------------------------------
(Updated Čec. 25, 2014, 12:44 odp.)
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
-------
software-dbus: enhanced repository listing functions
Allow to filter repositories by their state (enabled/disabled).
Diffs
-----
src/software-dbus/LMI_HostedSoftwareIdentityResourceProvider.c
ed0a44f8c24ba26211fbf4aa0d420852967bb011
src/software-dbus/LMI_ResourceForSoftwareIdentityProvider.c
731c5c902e033cc3ec91acf652e739d1b6c26c85
src/software-dbus/LMI_SoftwareIdentityResourceProvider.c
607660e80ba3b27a2d28be61a8aee8ebea6cbea6
src/software-dbus/sw-utils.h 8ce90ba240097c05db8f9b083aae01147ab21c5f
src/software-dbus/sw-utils.c 211f669539e1086fa8235d7dac3d6f3df26b4034
Diff:
http://reviewboard-openlmi.rhcloud.com/r/1934/diff/
Testing
-------
Thanks,
Michal Minar