Hi David,
> > I thought the test should change to handle no iconv, iconv, and
> > ice-age iconv
>
> As in, now it doesn't handle the no-iconv case? I went that route in
> anticipation of removing support for that, with the benefit of not
> having to make the test more complicated.
Right, there's
Ralph wrote:
> I thought the test should change to handle no iconv, iconv, and ice-age
> iconv
As in, now it doesn't handle the no-iconv case? I went that route in
anticipation of removing support for that, with the benefit of not
having to make the test more complicated.
David
--
Nmh-workers
Hi Ken,
> David wrote:
> > restores the old behavior of the test in that respect.
I thought the test should change to handle no iconv, iconv, and ice-age
iconv, but...
> Should I just spin another 1.7.1 release candidate?
...yes, please. 1.8 can revisit what tests are doing something useful
an
Ken wrote:
> Should I just spin another 1.7.1 release candidate?
Please!
David
--
Nmh-workers
https://lists.nongnu.org/mailman/listinfo/nmh-workers
>> But even though we are using GNU iconv, it turns out that in the old
>> code iconv_elides_question_marks was set to 0.
>
>Right. I just committed 8ed8876c6, which basically restores the old
>behavior of the test in that respect. It works on Mac OS X. On all
>platforms, it skips the tests that
Ken wrote:
> But even though we are using GNU iconv, it turns out that in the old
> code iconv_elides_question_marks was set to 0.
Right. I just committed 8ed8876c6, which basically restores the old
behavior of the test in that respect. It works on Mac OS X. On all
platforms, it skips the test
>GNU iconv is also used here, where I did d4814561e, but the commit does
>not remove support for eliding the `?'.
>
>The before, 79e8d527, says
>
>iconv_elides_question_marks=0
>if test "$ICONV_ENABLED" -eq 0; then
> text_size=10
>else
> text_size=11
> The GNU ico
Hi Ken,
> http://lists.nongnu.org/archive/html/nmh-workers/2018-02/msg00017.html
>
> You said you had a fix to a broken test ... was that pushed to master?
> If that was commit d4814561e
It was.
> then that is a BIT of a problem, since that commit broke test-charset
> on MacOS X
...
> MacOS X