From: Daniel Shahaf <d.s@daniel.shahaf.name>
To: zsh-workers@zsh.org
Subject: Re: [PATCH] Make --disable-multibyte warn, since the test suite fails in that configuration. (was: Re: Test failures in --disable-multibyte)
Date: Sun, 5 Jan 2020 19:13:22 +0000 [thread overview]
Message-ID: <20200105191322.s6yfdtpbpkvtkczk@tarpaulin.shahaf.local2> (raw)
In-Reply-To: <fad99155a576d89e7fb91288a0d6adde047db7a0.camel@ntlworld.com>
Peter Stephenson wrote on Sun, Jan 05, 2020 at 18:20:30 +0000:
> On Fri, 2020-01-03 at 20:11 +0000, Daniel Shahaf wrote:
> > Here's a draft. Probably needs wordsmithing.
>
> Sorry to be slow --- Definitely in the target area; I'm wondering about
No worries; I wasn't going to commit this until after the weekend anyway.
> > + # Some requisite functions are missing.
> > + AC_MSG_WARN([Multibyte support disabled due to missing functions: $zfuncs_absent])
> > + AC_MSG_WARN([Building without multibyte support is strongly discouraged and may cause errors in 'make test'.])
>
> where the user doesn't have a lot of choice about what they're building,
> so discouraging it is a bit pointless. I'd be tempted to omit that bit
> here.
Fair point. How about:
diff --git a/configure.ac b/configure.ac
index 256584538..cd42a789e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2553,6 +2553,7 @@ wmemcpy wmemmove wmemset; do
AC_MSG_NOTICE([all functions found, multibyte support enabled])
zsh_cv_c_unicode_support=yes
else
+ # Warns at the end of configure
AC_MSG_NOTICE([missing functions, multibyte support disabled])
zsh_cv_c_unicode_support=no
fi
@@ -3299,4 +3300,18 @@ fi
echo "See config.modules for installed modules and functions.
"
+if test x$zsh_cv_c_unicode_support != xyes; then
+ if test "x$zfuncs_absent" = x; then
+ # The user opted out.
+ AC_MSG_WARN([You have chosen to build without multibyte support.])
+ AC_MSG_WARN([This configuration may not be suitable for production use. It is known to cause errors in 'make test'. We strongly recommend to re-run configure with --enable-multibyte.])
+ else
+ # Some requisite functions are missing.
+ AC_MSG_WARN([Multibyte support cannot be enabled: some standard library functions are missing: $zfuncs_absent])
+ AC_MSG_WARN([This configuration may not be suitable for production use. It is known to cause errors in 'make test'. If your system provides those functions, we recommend to re-run configure appropriately.])
+ # If your system doesn't have those functions, consider patching the
+ # test suite and sending the patch to zsh-workers@ for inclusion.
+ fi
+fi
+
exit 0
(The "if" branch is unchanged.)
Cheers,
Daniel
next prev parent reply other threads:[~2020-01-05 19:14 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-31 18:39 Test failures in --disable-multibyte Daniel Shahaf
2019-12-31 19:46 ` Peter Stephenson
2020-01-01 13:44 ` Daniel Shahaf
2020-01-01 18:01 ` Peter Stephenson
2020-01-02 11:30 ` Daniel Shahaf
2020-01-03 10:15 ` Peter Stephenson
2020-01-03 20:11 ` [PATCH] Make --disable-multibyte warn, since the test suite fails in that configuration. (was: Re: Test failures in --disable-multibyte) Daniel Shahaf
2020-01-05 18:20 ` Peter Stephenson
2020-01-05 19:13 ` Daniel Shahaf [this message]
2020-01-06 9:44 ` Peter Stephenson
2019-12-31 20:23 ` Test failures in --disable-multibyte dana
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200105191322.s6yfdtpbpkvtkczk@tarpaulin.shahaf.local2 \
--to=d.s@daniel.shahaf.name \
--cc=zsh-workers@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).