Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-14 Thread Jonathan Wakely via Gcc-patches
On Sun, 13 Jun 2021 at 06:51, Tobias Burnus wrote: > > On 11.06.21 09:43, Martin Liška wrote: > > I support the patch, please install it (except the not intentional change > > in gcc/c/c-parser.c). > Committed as r12-1408-gd554f43c98eb07f222afef5e90b5582d65519f7e > > Then, please ask Jonathan for u

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-12 Thread Tobias Burnus
On 11.06.21 09:43, Martin Liška wrote: I support the patch, please install it (except the not intentional change in gcc/c/c-parser.c). Committed as r12-1408-gd554f43c98eb07f222afef5e90b5582d65519f7e Then, please ask Jonathan for updating of the server hook. @Jonathan: ↑ - thanks. Tobais ---

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-11 Thread Martin Liška
On 6/10/21 5:32 PM, Tobias Burnus wrote: On 10.06.21 16:46, Martin Liška wrote: Note that flake8 has "plugins". At openSUSE, I install: ... None of those are available on Ubuntu – I probably should nag doko or start using my private computer for the tests ... I support the patch, please inst

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-10 Thread Jonathan Wakely via Gcc-patches
On Thu, 10 Jun 2021 at 15:08, Tobias Burnus wrote: > > (Moved to gcc-patches, missed this when I replied to the initial email) > > Regarding patch at: https://gcc.gnu.org/pipermail/gcc/2021-June/236357.html > > On 10.06.21 14:45, Jonathan Wakely wrote: > > > As well as the "contrig" typo that Flori

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-10 Thread Tobias Burnus
On 10.06.21 16:46, Martin Liška wrote: Note that flake8 has "plugins". At openSUSE, I install: ... None of those are available on Ubuntu – I probably should nag doko or start using my private computer for the tests ... Updated as suggested and with you flake8-fix patch applied on top. Tobias

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-10 Thread Martin Liška
On 6/10/21 4:42 PM, Tobias Burnus wrote: On 10.06.21 16:24, Martin Liška wrote: I'm sending a small update that handles some flake8 issues and as defined in setup.cfg, line limit for the file is 120 characters. Aha, the known issue that Ubuntu 20.04 has a too old flake8 such that I only see a

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-10 Thread Tobias Burnus
On 10.06.21 16:24, Martin Liška wrote: I'm sending a small update that handles some flake8 issues and as defined in setup.cfg, line limit for the file is 120 characters. Aha, the known issue that Ubuntu 20.04 has a too old flake8 such that I only see a subset of the errors ... I have one quest

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-10 Thread Martin Liška
On 6/10/21 4:07 PM, Tobias Burnus wrote: (Moved to gcc-patches, missed this when I replied to the initial email) Regarding patch at: https://gcc.gnu.org/pipermail/gcc/2021-June/236357.html On 10.06.21 14:45, Jonathan Wakely wrote: As well as the "contrig" typo that Florian noticed, the subjec

Re: [Patch] contrib/gcc-changelog: Check that PR in subject in in changelog

2021-06-10 Thread Tobias Burnus
(Moved to gcc-patches, missed this when I replied to the initial email) Regarding patch at: https://gcc.gnu.org/pipermail/gcc/2021-June/236357.html On 10.06.21 14:45, Jonathan Wakely wrote: As well as the "contrig" typo that Florian noticed, the subject says "in in" which should be "is in". An