<div dir="ltr">Thanks, Sylvestre!  No worries on the timing, again sorry for breaking you.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 6, 2013 at 3:21 PM, Sylvestre Ledru <span dir="ltr"><<a href="mailto:sylvestre@debian.org" target="_blank">sylvestre@debian.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>Hello,<br>
      <br>
      Seems good :) Thanks for the patch.<br>
      I applied it as r196624.<br>
      <br>
      Sorry about the delay,<br>
      Sylvestre<div><div class="h5"><br>
       <br>
      On 05/12/2013 18:09, Todd Fiala wrote:<br>
    </div></div></div><div><div class="h5">
    <blockquote type="cite">
      <div dir="ltr">Here's a proposed patch for bug 18124 with the
        addition of supporting the optional DESTDIR argument to the make
        command.
        <div><br>
        </div>
        <div>I tested it with the original scenarios, plus the scenario
          of having a DESTDIR specified as the overall root for the
          install.</div>
        <div><br>
        </div>
        <div>Sylvestre, perhaps you can test this on your end to make
          sure it really fixes your scenario, and if it looks good,
          commit it?</div>
        <div><br>
        </div>
        <div>Thanks, and sorry for breaking you!</div>
        <div><br>
        </div>
        <div>Sincerely,</div>
        <div>Todd Fiala<br>
          <div class="gmail_extra"><br>
            <br>
            <div class="gmail_quote">On Wed, Dec 4, 2013 at 12:35 PM,
              Todd Fiala <span dir="ltr"><<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>></span>
              wrote:<br>
              <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
                <div dir="ltr">Hi all,
                  <div><br>
                  </div>
                  <div>This is the patch previously reviewed here for
                    fixing the autotools-based install of python
                    packages to the correct install tree.  Can I get a
                    committer to commit this, please?  Thank you!</div>
                  <div><br>
                  </div>
                  <div>The related bug report is here: <a href="http://llvm.org/bugs/show_bug.cgi?id=18124" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=18124</a></div>
                  <div><br>
                  </div>
                  <div>Sincerely,</div>
                  <div>Todd Fiala</div>
                </div>
              </blockquote>
            </div>
            <br>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
  </div></div></div>

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