<html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On 07/29/2016 11:41 AM, Lenka Doudova wrote: </div> <blockquote cite="mid:b5aca3c1-60a9-d7cb-73a6-8ad924dc7309@redhat.com" type="cite"> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> <div class="moz-cite-prefix">On 07/28/2016 01:35 PM, Peter Lacko wrote: </div> <blockquote cite="mid:529785673.9646711.1469705707399.JavaMail.zimbra@redhat.com" type="cite"> <pre wrap="">Hops, fixed. Peter ----- Original Message ----- From: "Lenka Doudova" <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:ldoudova@redhat.com"><ldoudova@redhat.com></a> To: <a moz-do-not-send="true" class="moz-txt-link-abbreviated" href="mailto:freeipa-devel@redhat.com">freeipa-devel@redhat.com</a> Sent: Thursday, July 28, 2016 1:32:25 PM Subject: Re: [Freeipa-devel] [PATCH 0003] Test validity of URIs in certificate Hi, I cannot find any attached patch :) Lenka On 07/28/2016 01:30 PM, Peter Lacko wrote: </pre> <blockquote type="cite"> <pre wrap="">Attached you can find a patch adding test for URIs in generated certificate ipatests/test_xmlrpc/test_cert_plugin.py Since I'm leaving Red Hat in end of July, I won't be able to modify this patch anymore. Regards, Peter </pre> </blockquote> <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Hi, NACK. Code looks fine and works well on master branch, but patch does not apply on 4-3 and 4-2 branches. Peter left the company but claimed he can fix the patch if necessary, I'll communicate it with him or fix it myself. Lenka <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Oh, and forgot this one - PEP8 error: ./ipatests/test_xmlrpc/test_cert_plugin.py:191:80: E501 line too long (105 > 79 characters) Lenka </body> </html>