GNU bug report logs -
#43117
[PATCH] Add .git-blame-ignore-revs file
Previous Next
Reported by: Brian Leung <leungbk <at> mailfence.com>
Date: Sun, 30 Aug 2020 17:47:01 UTC
Severity: normal
Tags: patch, wontfix
Done: Stefan Kangas <stefan <at> marxist.se>
Bug is archived. No further changes may be made.
Full log
Message #26 received at 43117 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
OK, please see attached. Right now there are no extraneous whitespace changes, and the only changes I've made change if -> when in the eshell files.
> Pure re-indentation patches I thought were not acceptable unless
> accompanied by an actual code change near the re-indented lines.
With a .git-blame-ignore file of some sort, shouldn't this not be a hindrance?
> ----------------------------------------
> From: Dmitry Gutov <dgutov <at> yandex.ru>
> Sent: Mon Aug 31 23:03:23 CEST 2020
> To: Robert Pluim <rpluim <at> gmail.com>
> Cc: Brian Leung <leungbk <at> mailfence.com>, <43117 <at> debbugs.gnu.org>
> Subject: Re: bug#43117: [PATCH] Add .git-blame-ignore-revs file
>
>
> On 31.08.2020 23:57, Robert Pluim wrote:
>
> > Dmitry> Do we accept reindentation patches, though? Ones where the effective
> > Dmitry> indentation does change.
> >
> > Didnʼt we just have this thread? :-)
>
> I might have missed it. :-(
>
> > Pure re-indentation patches I thought were not acceptable unless
> > accompanied by an actual code change near the re-indented lines.
>
> Okay.
>
> Brian, could you re-submit patches without extra re-indentation?
--
Sent with https://mailfence.com
Secure and private email
[0001-eshell-Replace-single-clause-if-statements-with-when.patch (text/x-diff, attachment)]
[0002-Add-.git-blame-ignore-revs.patch (text/x-diff, attachment)]
This bug report was last modified 4 years and 71 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.