Re: [PATCH] issues with configure --enable-checking option

2020-03-10 Thread Roman Zhuykov
Hi! 25.02.2020 13:36, Roman Zhuykov wrote: > So, IMHO the best next step is to improve the behavior rather then docs :) I want to add one more point into this discussion.  I have recently decided to test some stuff on old branches, e.q gcc-4.9, 5 and 6. On modern systems there are some issues wi

Re: [PATCH] issues with configure --enable-checking option

2020-02-25 Thread Roman Zhuykov
Hi, Jakub! Jakub Jelinek writes: >> +When the option is not specified, the active set of checks depends on >> context. >> +Namely, bootstrap stage 1 defaults to @samp{--enable-checking=yes}, builds >> +from release archives default to @samp{--enable-checking=release}, and > Is archives the right w

Re: [PATCH] issues with configure --enable-checking option

2020-02-25 Thread Jakub Jelinek
On Tue, Feb 25, 2020 at 01:36:13PM +0300, Roman Zhuykov wrote: > Not sure how to formulate it better, just "builds from release branch or > release archives default to ..." ? "from a release branch or" or "from release branches or", otherwise LGTM. Jakub

Re: [PATCH] issues with configure --enable-checking option

2020-02-25 Thread Jakub Jelinek
On Tue, Feb 25, 2020 at 11:20:58AM +0300, Roman Zhuykov wrote: > Sandra and Richard, thank you for review! > > Since 'types' checks are included into 'yes' and I addressed all other > hints, I have pushed updated patch as r10-6832. > > Jakub, Richard B, can I apply it to 8 and 9 ? Yes. > +When

Re: [PATCH] issues with configure --enable-checking option

2020-02-25 Thread Roman Zhuykov
Hi all! 22.02.2020 6:25, Sandra Loosemore wrote: > On 2/11/20 7:46 AM, Roman Zhuykov wrote: >> Since I have to ask again about backports, I've decided to make few more >> steps and with Alexander's help created new patch which rewords the >> whole option description and covers items (3), (4) and (

Re: [PATCH] issues with configure --enable-checking option

2020-02-21 Thread Sandra Loosemore
On 2/11/20 7:46 AM, Roman Zhuykov wrote: [snip] Since I have to ask again about backports, I've decided to make few more steps and with Alexander's help created new patch which rewords the whole option description and covers items (3), (4) and (8).  CCing Jakub and Richard as release managers, al

Re: [PATCH] issues with configure --enable-checking option

2020-02-17 Thread Roman Zhuykov
Ping.  Proposed patch is docs-only (install.texi), and IMHO it's better to push it into 8.4 and 9.3. 11.02.2020 17:46, Roman Zhuykov wrote: > 07.02.2020 20:20, Richard Sandiford writes: >> Roman Zhuykov writes: >>> Hi! >>> I've investigated a bit, because some of the following confused me while

Re: [PATCH] issues with configure --enable-checking option

2020-02-11 Thread Roman Zhuykov
07.02.2020 20:20, Richard Sandiford writes: > Roman Zhuykov writes: >> Hi! >> I've investigated a bit, because some of the following confused me while >> working with some local 9.2-based branch. >> >> Documentation issues: >> (0) See patch for install.texi at the bottom, two possible values are

Re: [PATCH] issues with configure --enable-checking option

2020-02-07 Thread Richard Sandiford
Roman Zhuykov writes: > Hi! > I've investigated a bit, because some of the following confused me while > working with some local 9.2-based branch. > > Documentation issues: > (0) See patch for install.texi at the bottom, two possible values are > not documented. Ok for master? Backports? > (1) F

Re: [PATCH] issues with configure --enable-checking option

2020-02-03 Thread Roman Zhuykov
Ping with CCing Jeff and Joseph. I think patch is rather obvious, but hopefully other items will also get some attention. 29.01.2020 16:32, Roman Zhuykov wrote: > Hi! > I've investigated a bit, because some of the following confused me > while working with some local 9.2-based branch. > > Documen