<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 22, 2015 at 9:01 AM, Martin Basti <span dir="ltr"><<a href="mailto:mbasti@redhat.com" target="_blank">mbasti@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000"><div><div class="h5">
    <div>On 22/05/15 16:08, Gabe Alford wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">
        <div>Hello,<br>
          <br>
        </div>
        <div>This should fix <a href="https://fedorahosted.org/freeipa/ticket/3584" target="_blank">https://fedorahosted.org/freeipa/ticket/3584</a>,
          and as requested in the ticket, this should also fix <a href="https://fedorahosted.org/freeipa/ticket/2284" target="_blank">https://fedorahosted.org/freeipa/ticket/2284<br>
          </a></div>
        <div><br>
        </div>
        <div>Thanks,<br>
          <br>
        </div>
        <div>Gabe<br>
        </div>
      </div>
      <br>
      <fieldset></fieldset>
      <br>
    </blockquote></div></div>
    Thank you!<br>
    <br>
    IMO your first part of fix only mask issue, not solving it.<br>
    <br>
    This could be way, but I did not test it.<br>
    <br>
                out_encoding = getattr(outfile, 'encoding', None)<br>
                if out_encoding is None:<br>
                    out_encoding = 'utf-8'<br>
                print >> outfile, 
    unicode(string).encode(out_encoding)<br></div></blockquote><div><br></div><div>I'm confused and maybe missing something here. If I run `ipa help dns | bad_command`, shouldn't the command fail with only the following?<br><pre class="">-bash: bad: command not found</pre></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    <br>
    Can you split this patch into 2 separate patches for each ticket
    please?<br></div></blockquote><div><br></div><div>Done<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    Martin^2<span class=""><font color="#888888"><br>
    <br>
    <pre cols="72">-- 
Martin Basti</pre>
  </font></span></div>

</blockquote></div><br></div></div>