<div class="gmail_quote">On Thu, May 31, 2012 at 8:55 AM, Óscar Fuentes <span dir="ltr"><<a href="mailto:ofv@wanadoo.es" target="_blank">ofv@wanadoo.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Justin Holewinski <<a href="mailto:jholewinski@nvidia.com">jholewinski@nvidia.com</a>> writes:<br>
<br>
> The attached patch modifies the CMake/LLVMBuild scripts to build<br>
> llvm-config on Windows if sed is found on the user's PATH.  It also<br>
> teaches LLVMBuild about MSVC library naming so llvm-config will<br>
> produce the proper lists of library names.<br>
><br>
> Any comments?  Okay to commit?<br>
<br>
</div>The presence of sed does not mean that it is usable. Think on a<br>
Cygwinized `sed' while building for MSVC. I'm not sure that on that case<br>
it can handle the pathnames as issued by cmake.<br></blockquote><div><br></div><div>I'm not sure why it would matter if its cygwin-ized or not.  All sed sees are strings, it's still up to CMake to write out the proper paths to the sed script... am I missing something?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
IMO the right fix is to get rid of sed altogether. Cmake has<br>
text-manipulation capabilities that should be enough for the job, IIRC.<br></blockquote><div><br></div><div>Fair enough.  I'll see if I can whip something up.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
[snip]<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div><br>