[x265] [Patch] [Master, Release 3.5] Fix: avoids unnecessary lexicographic order checks on git changesets

Srikanth Kurapati srikanth.kurapati at multicorewareinc.com
Tue Dec 22 08:41:11 UTC 2020


I have an update for this patch wrt to the correctness in the version
string. I am sending a cumulative patch altogether.

On Sat, Dec 19, 2020 at 10:05 PM Srikanth Kurapati <
srikanth.kurapati at multicorewareinc.com> wrote:

> From bda79a8f23f52bbfc9eca663c04ce831d39c81f2 Mon Sep 17 00:00:00 2001
> From: Srikanth Kurapati <srikanth.kurapati at multicorewareinc.com>
> Date: Wed, 16 Dec 2020 10:52:33 +0530
> Subject: [PATCH] fix: avoids unnecessary lexicographic order checks on git
>  changesets
>
> ---
>  source/cmake/Version.cmake | 20 ++++----------------
>  1 file changed, 4 insertions(+), 16 deletions(-)
>
> diff --git a/source/cmake/Version.cmake b/source/cmake/Version.cmake
> index 37c759268..09e642577 100644
> --- a/source/cmake/Version.cmake
> +++ b/source/cmake/Version.cmake
> @@ -168,25 +168,13 @@ endif()
>
>  # formatting based on positive or negative distance from tag
>  if(X265_TAG_DISTANCE STREQUAL "0")
> -    if(X265_REVISION_ID STREQUAL X265_REPO_ID)
> -        set(X265_VERSION "${X265_LATEST_TAG}")
> -    else()
> -        message(WARNING "REPO AND RELEASE CHANGESETS NOT MATCHING")
> -    endif()
> +    set(X265_VERSION "${X265_LATEST_TAG}")
>  elseif(X265_TAG_DISTANCE STRGREATER "0")
> -    if(X265_REVISION_ID STRLESS X265_REPO_ID)
> -        set(X265_VERSION
> "${X265_LATEST_TAG}+${X265_TAG_DISTANCE}-${X265_REVISION_ID}")
> -    else()
> -        message(WARNING "ARCCHIVE TIP CHANGESET TO BE GREATER THAN
> REVISION ID")
> -    endif()
> +    set(X265_VERSION
> "${X265_LATEST_TAG}+${X265_TAG_DISTANCE}-${X265_REVISION_ID}")
>  elseif(X265_TAG_DISTANCE STRLESS "0")
> -    if(X265_REVISION_ID STRGREATER X265_REPO_ID)
> -       set(X265_VERSION
> "${X265_LATEST_TAG}${X265_TAG_DISTANCE}+${X265_REPO_ID}")
> -    else()
> -        message(WARNING "REVISION ID EXPECTED TO BE LARGER THAN ARCHIVE
> TIP CHANGESET")
> -    endif()
> +    set(X265_VERSION
> "${X265_LATEST_TAG}${X265_TAG_DISTANCE}+${X265_REPO_ID}")
>  else()
> -    message(ERROR "Inappropriate set of version information")
> +    message(ERROR "Inappropriate version information")
>  endif()
>
>  #will always be printed in its entirety based on version file
> configuration to avail revision monitoring by repo owners
> --
> 2.20.1.windows.1
>
>
> --
> *With Regards,*
> *Srikanth Kurapati.*
>


-- 
*With Regards,*
*Srikanth Kurapati.*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20201222/0e2fada4/attachment.html>


More information about the x265-devel mailing list