
<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><DIV>I think we need to modify logic in hg path, because HG_REVISION will write empty string to X265_VERSION in this path</DIV>

<DIV> </DIV>

<DIV>在 2013-10-06 12:06:24，"Steve Borho" <steve@borho.org> 写道：<BR></DIV>

<BLOCKQUOTE id="isReplyContent" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">

<DIV dir="ltr">

<DIV class="gmail_extra"><BR>

<DIV class="gmail_quote">On Sat, Oct 5, 2013 at 9:26 PM, Min Chen <SPAN dir="ltr"><<A href="mailto:chenm003@163.com" target="_blank">chenm003@163.com</A>></SPAN> wrote:<BR>

<BLOCKQUOTE class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: rgb(204,204,204) 1px solid"># HG changeset patch<BR># User Min Chen <<A href="mailto:chenm003@163.com">chenm003@163.com</A>><BR># Date 1381026364 -28800<BR># Node ID 82d688c22244139df4fc986df60196c1e9541b3d<BR># Parent  4ee217cd64cc18a1f62fb8ef75c46240c123d1d9<BR>cmake: fix X265_VERSION not defined problem<BR><BR>Some system install hg but the source tree isn't managed by HG,<BR>In this time, we will build fail with error X265_VERSION macro<BR><BR>diff -r 4ee217cd64cc -r 82d688c22244 source/cmake/version.cmake<BR>--- a/source/cmake/version.cmake        Fri Oct 04 15:30:31 2013 -0500<BR>+++ b/source/cmake/version.cmake        Sun Oct 06 10:26:04 2013 +0800<BR>@@ -28,7 +28,7 @@<BR>     endif()<BR> elseif(DEFINED ENV{X265_VERSION})<BR>     set(X265_VERSION $ENV{X265_REVISION})<BR>-elseif(HG_EXECUTABLE)<BR>+elseif(HG_EXECUTABLE AND EXISTS ${CMAKE_SOURCE_DIR}/.hg)<BR></BLOCKQUOTE>

<DIV><BR></DIV>

<DIV>I see the problem you're trying to solve but this probably isn't the best solution.</DIV>

<DIV><BR></DIV>

<DIV>Can we just give X265_VERSION a value "unknown" at the top of the file in case the later checks all fail?</DIV>

<DIV> </DIV>

<BLOCKQUOTE class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: rgb(204,204,204) 1px solid">     execute_process(COMMAND<BR>         ${HG_EXECUTABLE} log -r. --template "{latesttag}"<BR>         WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}<BR></BLOCKQUOTE>

<DIV><BR></DIV>

<DIV>--</DIV>

<DIV>Steve Borho </DIV></DIV></DIV></DIV></BLOCKQUOTE></div>