<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 4, 2013 at 5:37 PM, BugMaster <span dir="ltr"><<a href="mailto:BugMaster@narod.ru" target="_blank">BugMaster@narod.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, 4 Sep 2013 17:11:53 -0400, Sean McGovern wrote:<br>
<br>
<br>
<br>
> On Mon, Jun 17, 2013 at 10:12 AM, Sean McGovern <<a href="mailto:gseanmcg@gmail.com">gseanmcg@gmail.com</a>> wrote:<br>
<br>
<br>
> On Monday, June 17, 2013, Jason Garrett-Glaser <<a href="mailto:jason@x264.com">jason@x264.com</a>> wrote:<br>
>> On Mon, Jun 17, 2013 at 6:50 AM, Sean McGovern <<a href="mailto:gseanmcg@gmail.com">gseanmcg@gmail.com</a>> wrote:<br>
>>><br>
>>><br>
>>> On Monday, June 17, 2013, Jason Garrett-Glaser <<a href="mailto:jason@x264.com">jason@x264.com</a>> wrote:<br>
>>>> On Mon, Jun 17, 2013 at 12:42 AM, Sean McGovern <<a href="mailto:gseanmcg@gmail.com">gseanmcg@gmail.com</a>><br>
>>>> wrote:<br>
>>>>> ---<br>
>>>>>  version.sh |   11 +++++++----<br>
>>>>>  1 file changed, 7 insertions(+), 4 deletions(-)<br>
>>>><br>
>>>> Could you explain this?<br>
>>>><br>
>>>> Jason<br>
>>><br>
>>> Sorry, my bad.<br>
>>><br>
>>> For projects that use tarsnaps of the x264 codebase (for example,<br>
>>> HandBrake), this script fails hard due to lack of .git folders.<br>
>><br>
>> Yes, but what does the patch *do*?<br>
>><br>
>> Jason<br>
<br>
> It generates a version number from git if and only if the .git folder is present.<br>
<br>
> -- Sean McG.<br>
<br>
<br>
> PING!<br>
<br>
> -- Sean McG.<br>
<br>
</div></div>First, it didn't really failed before patch (at least for me) if .git<br>
wasn't present. The only side effect was:<br>
"fatal: Not a git repository (or any of the parent directories): .git"<br>
but version information was generated in x264_config.h<br>
But after this patch X264_VERSION wouldn't be generated at all if you<br>
don't have git installed but have .git directory. So compilation will<br>
fail until you remove .git directory.<br></blockquote><div><br></div><div>OK, I will fix it to probe for git first. Thanks for spotting that.<br><br></div><div>-- Sean McG. <br></div></div></div></div>