On 02/06/15 10:24, Fraser Tweedale wrote:
On Mon, Jun 01, 2015 at 04:47:46PM +0200, Martin Basti wrote:
On 01/06/15 16:14, Rob Crittenden wrote:
Martin Basti wrote:
Fixes an issue caused by the latest installer patches pushed to
The use of globals makes my skin crawl a bit, but since you're making
changes in here you should take a look at this ticket:
this is fix for that ticket, I missed the ticket somehow.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
Fixes the problem for me, but I agree with Rob re globals - a
context manager would be much nicer. Something like (pseudocode):
... stuff currently in init_private_ccache()
... stuff currently in destroy_private_ccache()
Then in ipa-server-install main():
if not options.uninstall:
+ label=_('Failed principals'),
+ label=_('Failed targets'),
+ label=_('principal member'),
Are these names correct?
# ipa servicedelegationrule-find
1 service delegation rule matched
Delegation name: ipa-http-delegation
Allowed Target: ipa-ldap-delegation-targets,
Failed principals: HTTP/vm-093.example....@example.com
+ pattern_errmsg='may only include letters, numbers, _, -,
+ 'and a space inside',
This regex does not allow space inside
In : print re.match(pattern, 'lalalala lalala')
+ yield Str('%s*' % name, cli_name='%ss' % name, doc=doc,
+ label=_('member %s') % name,
+ csv=True, alwaysask=True)
IMHO CSV values should not be supported.
Honza told me, the option doesn't work anyway.
Patch with minor fixes attached.
I removed unused code and PEP8 complains