On July 22, 2014, 1:34 p.m., Miloslav Trmac wrote:
> Misses the non-hasattr(… "property") case (would end up forwarding this
into the RoleBase class and failing AFAICS)
This is correct, but given our time constraints (and the fact that this will work on the
target platform of Fedora 21), I'd suggest opening a ticket at
https://fedorahosted.org/rolekit and making sure we fix it later.
- Stephen
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/22/#review71
-----------------------------------------------------------
On July 22, 2014, 10:23 a.m., Thomas Woerner wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/22/
-----------------------------------------------------------
(Updated July 22, 2014, 10:23 a.m.)
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
This property shows the name of the role in D-Bus
Diffs
-----
src/rolekit/server/dbusrole.py 6cef7cfeba6a7bbe396601c7ab31fe034757bc6a
Diff:
http://reviewboard-fedoraserver.rhcloud.com/r/22/diff/
Testing
-------
Thanks,
Thomas Woerner