[Mesa-dev] [PATCH 3/5] travis: correct libdrm required regex to also track libdrm itself

Andres Gomez agomez at igalia.com
Thu Feb 2 07:53:15 UTC 2017


LGTM

Reviewed-by: Andres Gomez <agomez at igalia.com>

On Wed, 2017-02-01 at 22:30 +0000, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> The current regex was tracking only the libdrm_foo packages, while with
> recent changed we bumped only (and rightfully so) libdrm.
> 
> Fix the regex to track any libdrm package.
> 
> Cc: Rhys Kidd <rhyskidd at gmail.com>
> Cc: Eric Anholt <eric at anholt.net>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
>  .travis.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index dfbc052a7d..e8426e96a2 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -47,7 +47,7 @@ install:
>  
>    # Since libdrm gets updated in configure.ac regularly, try to pick up the
>    # latest version from there.
> -  - for line in `grep "^LIBDRM_.*_REQUIRED=" configure.ac`; do
> +  - for line in `grep "^LIBDRM.*_REQUIRED=" configure.ac`; do
>        old_ver=`echo $LIBDRM_VERSION | sed 's/libdrm-//'`;
>        new_ver=`echo $line | sed 's/.*REQUIRED=//'`;
>        if `echo "$old_ver,$new_ver" | tr ',' '\n' | sort -Vc 2> /dev/null`; then
-- 
Br,

Andres


More information about the mesa-dev mailing list