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ä <[email protected]> > > > > 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ä <[email protected]> > > --- > > 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 ' '
Shell doesn't do anything with / so ' vs " shouldn't matter I don't think. > > > 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 " " That "..$" stuff scares me a bit. I guess it doesn't do anything because there's nothing else after the $. But maybe that should be changed to single quotes? > 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 <[email protected]> > > > > 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
