On 28/07/15 14:11, Martin Basti wrote:
On 28/07/15 13:33, Martin Babinsky wrote:
On 07/27/2015 05:10 PM, Martin Basti wrote:
On 23/07/15 16:06, Martin Babinsky wrote:
This is a quick fix for https://fedorahosted.org/freeipa/ticket/5037



NACK

I do not like your change in first regexp too much.

Can you try this instead?

PermPat = re.compile(r'(\w+)\s*\(([^()]*)\)\s*(.*)', re.UNICODE)

This just removes '(' and ') ' from pattern and accept all other characters.

--
Martin Basti


Attaching updated patch.

ACK

Pushed to:
master: a2ba9373070b19c158be8be78f7fbeee5ccab081
ipa-4-2: d85f92c0e75f3b389edac353de2cf08105b33cc4


--
Martin Basti

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to