On Fri, 2019-01-11 at 14:56 +0000, Eric Engestrom wrote:
> On Friday, 2019-01-11 16:42:25 +0200, Andres Gomez wrote:
> > "--summary" will also print extended header information such as
> > creations, renames and mode changes.
> >
> > Let's just use "-s", which suppresses the diff output.
> >
> > Fixes: 559c32d2412 ("bin/get-pick-list.sh: simplify git oneline printing")
> > Cc: Juan A. Suarez <jasua...@igalia.com>
> > Cc: Eric Engestrom <eric.engest...@intel.com>
> > Cc: Dylan Baker <dy...@pnwbakers.com>
> > Cc: Emil Velikov <emil.veli...@collabora.com>
> > Signed-off-by: Andres Gomez <ago...@igalia.com>
> > ---
> > bin/get-pick-list.sh | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/bin/get-pick-list.sh b/bin/get-pick-list.sh
> > index 3099fc69413..d71ff4a708a 100755
> > --- a/bin/get-pick-list.sh
> > +++ b/bin/get-pick-list.sh
> > @@ -143,7 +143,7 @@ do
> > esac
> >
> > printf "[ %8s ] " "$tag"
> > - git --no-pager show --summary --oneline $sha
> > + git --no-pager show -s --oneline $sha
>
> Small preference for the explicit `--no-patch` instead of `-s`, but:
> Reviewed-by: Eric Engestrom <eric.engest...@intel.com>
Changed locally and pushed.
Thanks for the review!
>
> > done
> >
> > rm -f already_picked
> > --
> > 2.18.0
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>
--
Br,
Andres
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
