<div dir="ltr">2013/4/11 Carl Eugen Hoyos <span dir="ltr"><<a href="mailto:cehoyos@ag.or.at" target="_blank">cehoyos@ag.or.at</a>></span><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im">Lars Hammarstrand <lars.hammarstrand@...> writes:<br>  <br></div><div class="im">
> Like a git submodule or what do you suggest?<br>
<br>
</div>A link to github as on<br>
<a href="http://ffmpeg.org/platform.html#Darwin-_0028Mac-OS-X_002c-iPhone_0029" target="_blank">http://ffmpeg.org/platform.html#Darwin-_0028Mac-OS-X_002c-iPhone_0029</a><br>
<div class=""><div class="h5"><br></div></div></blockquote><div> </div><div style><div>Well, I'm not sure a that just a link would make it more useful than having our own copy of gas-processor during the automatic configure process that xbmc conducts :) But read on...</div>
<div><br></div></div><div><div>The link explains:</div><div><br></div><div><i>"Mac OS X on PowerPC or ARM (iPhone) requires a preprocessor from <a href="http://github.com/yuvi/gas-preprocessor">http://github.com/yuvi/gas-preprocessor</a> to build the optimized assembler functions. Just download the Perl script and <b><u>put it </u></b><u><b>somewhere in your PATH</b></u>,  FFmpeg’s configure will pick it up automatically." </i></div>
<div><br></div><div>The line "put it somewhere in your PATH" is the reason why xbmc introduced a patch to make configure point out the gas-preprocessor with an absolute path thus make it able to override any potential erroneous or outdated versions of gas-preprocessor in the search path. I would rather see a different solution in the future and get rid of this tedious but still necessary xbmc patch so I'm open to any suggestions! </div>
<div><div><br></div><div>One idea could be to add an "gas" option to configure , like "<font face="courier new, monospace">--gas=/abosolute/path/to/gas-preprocessor</font>" (pls note that this <span style="font-family:arial,helvetica,sans-serif">does not work:</span>  "<font face="courier new, monospace">--as=/absolute/path/to/<a href="http://gas-preprocessor.pl">gas-preprocessor.pl</a> /path/to/as..."</font><font face="arial, helvetica, sans-serif"> ).</font></div>
</div><div><br></div><div style>As an addition to the above suggestion, a version control of <a href="http://gas-processor.pl">gas-processor.pl</a> during the configure process of ffmpeg would be a rather nice complement.<br>
</div><div style><br></div><div style>Any ideas?</div><div style><br></div></div><div style>Regard, Lars</div><div style>--</div><div><br></div><div><br></div><div style>Ps..</div><div style><br></div><div style><div>1. xbmc gas patch: <a href="https://github.com/xbmc/xbmc/blob/master/lib/ffmpeg/patches/0029-stop-forcing-gas-preprocessor-for-darwin.patch">https://github.com/xbmc/xbmc/blob/master/lib/ffmpeg/patches/0029-stop-forcing-gas-preprocessor-for-darwin.patch</a></div>
<div><br></div></div><div style>2.  not important but I would be most grateful if you were are able to answer some of my previous questions as well:</div><div style><div><br></div></div><div style><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div>> 1) Distribute a prepackaged version of <a href="http://gas-processor.pl">gas-processor.pl</a><br>> that works, together with ffmpeg.<br><br></div>Very, very definitely:<br>No.<br><div><br></div></blockquote><div>
<br></div></div><div>Ok, I take that as a No ;-) May I ask why? Another alternative solution could be using git submodule in the ffmpeg distro that points to a working version of gas-processor, would that be ok?</div><div class="im">
<div style="font-family:arial,sans-serif;font-size:13px"><br></div></div></div><div style><div class="im" style="font-family:arial,sans-serif;font-size:13px"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br class="">Note that I originally wanted to write that I find it an incredibly bad idea that xbmc contains its own gas-preprocessor script.<br></blockquote><div><br></div></div><div style="font-family:arial,sans-serif;font-size:13px">
Well, first of all it's not xbmc's own gas-preprocessor but rather a copy of a version that (at that time) was compatible with ffmpeg 0.10.2. If you believe there is a better way, what do you suggest? </div><div><br>
</div><div><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">To be clear on this one:  </span><font color="#500050" face="arial, sans-serif">how du you suggest we should automatic configure and compile ffmpeg </font><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">if xbmc should not contain it's own gas-preprocessor? </span></div>
<div>--</div><div><br></div><div class="im" style="font-family:arial,sans-serif;font-size:13px"></div></div></div></div></div>