From: Ian Campbell <ian.campbell@citrix.com>
To: Ian Jackson <ian.jackson@eu.citrix.com>, xen-devel@lists.xenproject.org
Subject: Re: [OSSTEST PATCH 3/3] Shell fixup: Use bash in posix mode
Date: Fri, 25 Sep 2015 09:47:54 +0100 [thread overview]
Message-ID: <1443170874.25250.54.camel@citrix.com> (raw)
In-Reply-To: <1443115762-7046-3-git-send-email-ian.jackson@eu.citrix.com>
On Thu, 2015-09-24 at 18:29 +0100, Ian Jackson wrote:
> I have read through the wheezy
> bash(1) manpage and searched for posix, and the following behavioural
> differences are described:
>
> * Differences in interative startup (not relevant to us).
> * Minor (irrelevant) differences in which startup files are read
> during noninteractive startup. (Eg, BASH_ENV not honoured.)
> * Differences to the parsing of invocations of `time'
> * `test a == b' may be unsupported (but it's wrong and we say `=')
> * -e not inherited by some subshells (this is what I am trying to fix)
> * `.' and `source' do not search the cwd.
> * `set' with no arguments does not print shell functions etc.
I expected this to also disable "[[ expression ]]", "$((maths))", array
variables and in particular $PIPESTATUS too, but it looks like I was wrong
and it doesn't disable bash _extensions_, just areas where bash differs
from what posix specifies.
Of those we seem to use $((maths)) and $PIPESTATUS (once in the standalone
wrapper), although array variables are tricky to grep for.
I also found http://tiswww.case.edu/php/chet/bash/POSIX (referenced from
the manpage) handy.
> I have done some ad-hoc testing but it seems like much of this is
> difficult to test. I suggest we push it at a time when we can keep a
> close eye on the behaviour.
Agreed.
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
next prev parent reply other threads:[~2015-09-25 8:47 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-24 17:29 [OSSTEST PATCH 1/3] Remove obsolete file "test.sched" Ian Jackson
2015-09-24 17:29 ` [OSSTEST PATCH 2/3] Shell fixup: Make all invocations of `.' (`source') use ./ Ian Jackson
2015-09-25 8:40 ` Ian Campbell
2015-09-24 17:29 ` [OSSTEST PATCH 3/3] Shell fixup: Use bash in posix mode Ian Jackson
2015-09-25 8:47 ` Ian Campbell [this message]
2015-09-25 8:37 ` [OSSTEST PATCH 1/3] Remove obsolete file "test.sched" Ian Campbell
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=1443170874.25250.54.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=xen-devel@lists.xenproject.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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.