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

[Rhys Kidd]

On 2 February 2017 at 02:53, Andres Gomez <agomez at igalia.com> wrote:

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

Reviewed-by: Rhys Kidd <rhyskidd at gmail.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
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170202/ef158761/attachment.html>