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

2017-02-02 Thread Rhys Kidd 
On 2 February 2017 at 02:53, Andres Gomez  wrote:

> LGTM
>
> Reviewed-by: Andres Gomez 
>

Reviewed-by: Rhys Kidd 


>
> On Wed, 2017-02-01 at 22:30 +, Emil Velikov wrote:
> > From: Emil Velikov 
> >
> > 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 
> > Cc: Eric Anholt 
> > Signed-off-by: Emil Velikov 
> > ---
> >  .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
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

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

2017-02-01 Thread Andres Gomez 
LGTM

Reviewed-by: Andres Gomez 

On Wed, 2017-02-01 at 22:30 +, Emil Velikov wrote:
> From: Emil Velikov 
> 
> 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 
> Cc: Eric Anholt 
> Signed-off-by: Emil Velikov 
> ---
>  .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
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

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

2017-02-01 Thread Emil Velikov 
From: Emil Velikov 

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 
Cc: Eric Anholt 
Signed-off-by: Emil Velikov 
---
 .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
-- 
2.11.0

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev