<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Hello,<br>
<br>
Seems good :) Thanks for the patch.<br>
I applied it as r196624.<br>
<br>
Sorry about the delay,<br>
Sylvestre<br>
<br>
On 05/12/2013 18:09, Todd Fiala wrote:<br>
</div>
<blockquote
cite="mid:CAFab17LbCjq_im4gQm4gBTzJcy+Rz-HwHhZ4pqp51wub=UNqsg@mail.gmail.com"
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 moz-do-not-send="true"
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
moz-do-not-send="true"
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>
</body>
</html>