Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-13 Thread Stefano Lattarini
Hello Ralf. There's still a wart to be fixed before this patch series can be committed ... On Thursday 13 January 2011, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Jan 12, 2011 at 09:11:41PM CET: On Wednesday 12 January 2011, Ralf Wildenhues wrote: So, now with that said, I'm

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-13 Thread Stefano Lattarini
[Ralf Wildenhues] If some code later calls it like process_option_list (first-set-of-options); process_option_list (second-set-of-options); then things will go wrong again. I suspect that it will mean that AM_INIT_AUTOMAKE([foreign -Wno-portability])

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-13 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Thu, Jan 13, 2011 at 10:27:48PM CET: On Thursday 13 January 2011, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Jan 12, 2011 at 09:11:41PM CET: On Wednesday 12 January 2011, Ralf Wildenhues wrote: So, now with that said, I'm not sure whether I

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-13 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Fri, Jan 14, 2011 at 12:11:27AM CET: [Ralf Wildenhues] If some code later calls it like process_option_list (first-set-of-options); process_option_list (second-set-of-options); then things will go wrong again. I suspect that it will mean

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-12 Thread Ralf Wildenhues
Here comes my (belated, as always) re-review; I am not repeating nits already dealt with before. * Stefano Lattarini wrote on Wed, Jan 05, 2011 at 08:22:13PM CET: On Wednesday 05 January 2011, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Tue, Jan 04, 2011 at 06:38:04PM CET: This is

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-12 Thread Stefano Lattarini
On Wednesday 12 January 2011, Ralf Wildenhues wrote: Here comes my (belated, as always) re-review; I am not repeating nits already dealt with before. * Stefano Lattarini wrote on Wed, Jan 05, 2011 at 08:22:13PM CET: On Wednesday 05 January 2011, Ralf Wildenhues wrote: * Stefano Lattarini

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-12 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Wed, Jan 12, 2011 at 09:11:41PM CET: On Wednesday 12 January 2011, Ralf Wildenhues wrote: So, now with that said, I'm not sure whether I should approve this patch. What do you think? I think that you should, provided that I add the sanity check you suggested

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-06 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Wed, Jan 05, 2011 at 08:22:13PM CET: On Wednesday 05 January 2011, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Tue, Jan 04, 2011 at 06:38:04PM CET: + # will take precedence over warning settings defined implicitly by the + # strictness. Well,

Re: [PATCH 4/9] Warnings win over strictness in AM_INIT_AUTOMAKE.

2011-01-05 Thread Stefano Lattarini
On Wednesday 05 January 2011, Ralf Wildenhues wrote: Hi Stefano, * Stefano Lattarini wrote on Tue, Jan 04, 2011 at 06:38:04PM CET: This is derived from [PATCH 07/10] of the older series. It requires a review. Thank you for rewriting the patch. Really you should thank git rebase -i ;-)