zsh-workers
 help / color / mirror / code / Atom feed
From: Peter Stephenson <pws@csr.com>
To: zsh-workers@sunsite.dk (Zsh hackers list)
Subject: Re: PATCH: test output to terminal dehackery
Date: Wed, 30 May 2007 12:17:08 +0100	[thread overview]
Message-ID: <200705301117.l4UBH88X010726@news01.csr.com> (raw)
In-Reply-To: <070530040340.ZM30303@torch.brasslantern.com>

Bart Schaefer wrote:
> The test suite should restrict itself only to the constructs that are
> least likely to be affected by operating system and compiler portability
> issues, so that it isn't necessary for a person building and testing on
> a brand-new or otherwise unusual platform to fix an obscure or difficult
> problem based on an unknown failure in a very complex piece of shell code
> when there's a minimal example buried somewhere in the suite that might
> immediately reveal the actual problem.
> 
> The test suite isn't just about you or me finding something we broke in
> development.

That's fair enough, but I don't think a change in the redirection
syntax qualifies as something likely to produce a portability problem.
The underlying features used are the same; the addition is an internal
shell parameter assignment (which had also better work).  Unless you
think using file descriptors over 10 is likely to be problematic?  But
the shell has used those internally for a long time.

-- 
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


To access the latest news from CSR copy this link into a web browser:  http://www.csr.com/email_sig.php

To get further information regarding CSR, please visit our Investor Relations page at http://ir.csr.com/csr/about/overview


      reply	other threads:[~2007-05-30 11:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-29 14:43 Peter Stephenson
2007-05-29 15:01 ` Stephane Chazelas
2007-05-30  3:29 ` Bart Schaefer
2007-05-30  9:49   ` Peter Stephenson
2007-05-30 11:03     ` Bart Schaefer
2007-05-30 11:17       ` Peter Stephenson [this message]

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=200705301117.l4UBH88X010726@news01.csr.com \
    --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).