<div class="gmail_quote">On Sun, Oct 10, 2010 at 7:39 PM, Sean McGovern <span dir="ltr"><<a href="mailto:gseanmcg@gmail.com">gseanmcg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><div class="gmail_quote">On Sun, Oct 10, 2010 at 7:29 PM, Sean McGovern <span dir="ltr"><<a href="mailto:gseanmcg@gmail.com" target="_blank">gseanmcg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Submitted for approval, here is a patch that fixes build of x264 on SPARC Solaris 10.<div><br></div><div>Some notes about the patch:</div><div><ol><li>As before, this still assumes you have the GNU compiler & toolchain (except Sun's assembler), I have successfully built gcc 4.5.1 myself for SPARC and have been using that. I have not made any attempt to build with Sun Studio, although I do have it installed.</li>


<li>On Solaris, neither /usr/xpg4/bin/grep nor /usr/bin/grep are capable of handling binary files directly, so I have instead piped the conftest.o through /usr/bin/strings, which also works on many other *ix platforms including Linux and MacOS X.</li>


<li>SPARC is an acronym (<a href="http://www.sparc.org/standards/SPARCV9.pdf" target="_blank">http://www.sparc.org/standards/SPARCV9.pdf</a> , section 0.1) -- all references have been changed to uppercase (there weren't many).</li>

<li>
sun4v's are 64-bit capable as well, so catch this in the configure script.</li></ol></div><div>Additionally, although it's largely harmless, builders may want to add the LARGEFILE arguments to CFLAGS to squelch a warning. My build script (bash-syntax) does it thus:</div>


<div><br></div><div>LFS_CFLAGS=$(getconf LFS_CFLAGS)</div><div>CFLAGS="${CFLAGS} ${LFS_CFLAGS}"</div><div><br></div><div>I would be more than happy to share my build script if anyone wants a copy.</div><div><br>


</div><div>-- Sean McG</div>
</blockquote></div><br></div></div><div>It helps a bunch when the patch is um... actually attached. Yeah.</div><div><br></div><div>-- Sean McG</div>
</blockquote></div><br><div>Ping?</div>