Tom Tromey <[EMAIL PROTECTED]> writes:
> > "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> As a first step, I can apply this patch *but* with this part and
> Akim> the --Werror part in defs.
>
> I don't see a `defs' change here.
Hm, I suppose I'm still using `but' incorrectly
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
Akim> As a first step, I can apply this patch *but* with this part and
Akim> the --Werror part in defs.
I don't see a `defs' change here.
Akim> * automake.in: Use -w.
Akim> Normalize all use of `$lang . '-foo'' into `"$lang-foo"'.
Ak
Akim Demaille <[EMAIL PROTECTED]> writes:
> As a first step, I can apply this patch *but* with this part and the
> --Werror part in defs. Then we have -w in automake, the bad uses are
> fixed (at least those triggered by the test suite and the fileutils),
> the only thing is that the test suite
Tom Tromey <[EMAIL PROTECTED]> writes:
> >> Sometimes this error is really an error. That is why we save and
> >> optionally restore the exit status around the message printing. Let's
> >> make this work correctly instead of breaking it.
>
> Akim> I believe my patch is correct, note the $exit_
>> Sometimes this error is really an error. That is why we save and
>> optionally restore the exit status around the message printing. Let's
>> make this work correctly instead of breaking it.
Akim> I believe my patch is correct, note the $exit_status = 1.
I was just responding to your comment
Tom Tromey <[EMAIL PROTECTED]> writes:
> >> (&require_file_internal): Don't warn about installed files, just
> >> output on STDERR.
>
> Akim> I'm fine with making this a warning again some day, but then it
> Akim> should be something like --warning=installing or something, set
> Akim> by default
>> (&require_file_internal): Don't warn about installed files, just
>> output on STDERR.
Akim> I'm fine with making this a warning again some day, but then it
Akim> should be something like --warning=installing or something, set
Akim> by default, but we need to be able to switch it off when runni
Akim Demaille <[EMAIL PROTECTED]> writes:
> (&require_file_internal): Don't warn about installed files, just
> output on STDERR.
I'm fine with making this a warning again some day, but then it should
be something like --warning=installing or something, set by default,
but we need to
The goal is to use -w *and* to have the test suite fail when there are
-w-warnings.
Note that I could catch the `$_ needs be saved' issue because I could
see the errors from sinclude.test, i.e., before that I realized that
--Wno-error was needed by this test, I had to observe the many
complaints
