zsh-workers
 help / color / mirror / code / Atom feed
From: Peter Stephenson <pws@csr.com>
To: zsh-workers@sunsite.dk
Subject: Re: D07multibyte.ztst failure on HP-UX 11.11
Date: Fri, 8 May 2009 15:23:30 +0100	[thread overview]
Message-ID: <20090508152330.75dcbd11@news01> (raw)
In-Reply-To: <20090507220819.GA13920@svalbard>

On Thu, 7 May 2009 22:08:19 +0000
Paul Ackersviller <pda@sdf.lonestar.org> wrote:
> > This means your test still fails, however.  I'm not sure what to do about
> > this since clearly the test *does* fail; you don't get a valid character or
> > an error.  Since character set conversion isn't working on your system (as
> > evinced by the result of not setting the C locale), I suppose we could test
> > that and if so skip the test?
> 
> Sounds like about the best that can be done under the circumstances.

OK, let's do the following.  It prints a message but avoids looking like a
shell bug if character set conversion doesn't work in a simple case.

Index: Test/D07multibyte.ztst
===================================================================
RCS file: /cvsroot/zsh/zsh/Test/D07multibyte.ztst,v
retrieving revision 1.32
diff -u -r1.32 D07multibyte.ztst
--- Test/D07multibyte.ztst	7 May 2009 16:04:42 -0000	1.32
+++ Test/D07multibyte.ztst	8 May 2009 14:19:38 -0000
@@ -378,17 +378,25 @@
 >X$'\300'Y$'\a'Z$'\177'T
 
 # This also isn't strictly multibyte and is here to reduce the
-# likelihood of a "can't do character set conversion" error.
-  testfn() { (LC_ALL=C; print $'\u00e9') }
-  repeat 4 testfn 2>&1 | while read line; do
-    if [[ $line = *"character not in range"* ]]; then
-      print OK
-    elif [[ $line = "?" ]]; then
-      print OK
-    else
-      print Failed: no error message and no question mark
-    fi
-  done
+# likelihood of a "cannot do character set conversion" error.
+  (print $'\u00e9') 2>&1 | read
+  if [[ $REPLY != é ]]; then
+    print "warning: your system can't do simple Unicode conversion." >&$ZTST_fd
+    print "Check you have a correctly installed iconv library." >&$ZTST_fd
+    # cheat
+    repeat 4 print OK
+  else
+    testfn() { (LC_ALL=C; print $'\u00e9') }
+    repeat 4 testfn 2>&1 | while read line; do
+      if [[ $line = *"character not in range"* ]]; then
+        print OK
+      elif [[ $line = "?" ]]; then
+        print OK
+      else
+        print Failed: no error message and no question mark
+      fi
+    done
+  fi
   true
 0:error handling in Unicode quoting
 >OK



-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070


  parent reply	other threads:[~2009-05-08 14:28 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-27  3:17 Paul Ackersviller
2009-04-27  4:42 ` Andrey Borzenkov
2009-04-27 19:26   ` Paul Ackersviller
2009-04-28  8:48     ` Peter Stephenson
2009-04-28 19:19       ` Paul Ackersviller
2009-04-28 19:48         ` Peter Stephenson
2009-04-30  3:01           ` Paul Ackersviller
2009-04-30  8:41             ` Peter Stephenson
2009-04-30 15:58               ` Paul Ackersviller
2009-04-30 16:03                 ` Peter Stephenson
2009-05-01 14:52                   ` Paul Ackersviller
2009-05-01 15:18                     ` Peter Stephenson
2009-05-05 19:39                       ` Paul Ackersviller
2009-05-06 19:22                         ` Peter Stephenson
2009-05-06 21:50                           ` Paul Ackersviller
2009-05-07 15:38                             ` Peter Stephenson
2009-05-07 16:02                               ` Peter Stephenson
2009-05-07 22:08                                 ` Paul Ackersviller
2009-05-07 23:30                                   ` Modules on HP-UX (Re: D07multibyte.ztst failure on HP-UX 11.11) Bart Schaefer
2009-05-08  8:34                                     ` Peter Stephenson
2009-05-08 14:20                                       ` Bart Schaefer
2009-05-08 14:29                                         ` Peter Stephenson
     [not found]                                           ` <090508084125.ZM17697@torch.brasslantern.com>
2009-05-11  8:52                                             ` Peter Stephenson
2009-05-08 18:42                                     ` Modules on HP-UX Paul Ackersviller
2009-05-12 20:22                                     ` Modules on HP-UX, with small PATCH Paul Ackersviller
2009-05-08 14:23                                   ` Peter Stephenson [this message]
2009-05-02  1:00               ` D07multibyte.ztst failure on HP-UX 11.11 Phil Pennock

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=20090508152330.75dcbd11@news01 \
    --to=pws@csr.com \
    --cc=zsh-workers@sunsite.dk \
    /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).