On Thu, 30 Mar 2023, Rodrigo Vivi <rodrigo.v...@intel.com> wrote:
> On Wed, Mar 29, 2023 at 10:53:53PM +0300, Ville Syrjälä wrote:
>> On Wed, Feb 08, 2023 at 08:21:51AM -0500, Rodrigo Vivi wrote:
>> > On Wed, Feb 08, 2023 at 03:32:43AM +0200, Ville Syrjala wrote:
>> > > From: Ville Syrjälä <ville.syrj...@linux.intel.com>
>> > > 
>> > > GNU grep 3.8 started to complain about backslashes where
>> > > they're not needed:
>> > >  grep: warning: stray \ before /
>> > > Get rid of them.
>> > > 
>> > > Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
>> > > ---
>> > >  dim | 6 +++---
>> > >  1 file changed, 3 insertions(+), 3 deletions(-)
>> > > 
>> > > diff --git a/dim b/dim
>> > > index b87b54259f8b..296feb9dd314 100755
>> > > --- a/dim
>> > > +++ b/dim
>> > > @@ -503,7 +503,7 @@ function dim_retip
>> > >          if repo_to_remote drm-tip &> /dev/null ; then
>> > >                  new_upstream=$(repo_to_remote drm-tip)/drm-tip
>> > >          else
>> > > -                new_upstream=$(git branch -r | grep '\/drm-tip$')
>> > > +                new_upstream=$(git branch -r | grep '/drm-tip$')
>> > 
>> > this I knew it would work... because of ' '
>> > 
>> > >          fi
>> > >          upstream=$(git_find_tip "$branch")
>> > >  
>> > > @@ -1124,12 +1124,12 @@ function check_diff_prefix
>> > >          local rv patch msg patch_recoded
>> > >          patch="$1"
>> > >  
>> > > -        rv=$(grep -q -E "^diff --git a\/.+ b\/.+$" $patch)
>> > > +        rv=$(grep -q -E "^diff --git a/.+ b/.+$" $patch)
>> > 
>> > but here with  " "
>> > I had my doubts... but confirmed that it does work with previous
>> > grep 3.7.
>> > 
>> > I hope it doesn't go badly with older grep versions...
>> > 
>> > with this hope
>> > 
>> > Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com>
>> 
>> I can't push this btw. -EPERM or something.
>
> per rules we need Jani's or Daniel's ack, then I can help on
> pushing it...

Acked-by: Jani Nikula <jani.nik...@intel.com>

Daniel, is it time we switched the whole thing to merge request process
btw?

BR,
Jani.



>
>> 
>> > 
>> > 
>> > >          if [ -z "$rv" ]; then
>> > >                  msg=$(mktemp --tmpdir dim-msg.XXXXXX)
>> > >                  patch_recoded=$(mktemp --tmpdir dim-patch.XXXXXX)
>> > >                  git mailinfo "$msg" "$patch_recoded" < $patch >/dev/null
>> > > -                rv=$(grep -q -E "^diff --git a\/.+ b\/.+$" 
>> > > $patch_recoded)
>> > > +                rv=$(grep -q -E "^diff --git a/.+ b/.+$" $patch_recoded)
>> > >                  rm "$msg" "$patch_recoded"
>> > >          fi
>> > >  
>> > > -- 
>> > > 2.39.1
>> > > 
>> 
>> -- 
>> Ville Syrjälä
>> Intel

-- 
Jani Nikula, Intel Open Source Graphics Center

Reply via email to