From: Martijn Dekker <martijn@inlv.org>
To: zsh-workers@zsh.org
Cc: Peter Stephenson <p.stephenson@samsung.com>
Subject: Re: [BUG] Sticky-sh POSIX_TRAPS are function-local
Date: Wed, 17 Feb 2016 00:38:15 +0100 [thread overview]
Message-ID: <56C3B2E7.1090806@inlv.org> (raw)
In-Reply-To: <20160216095744.52cb8389@pwslap01u.europe.root.pri>
Peter Stephenson schreef op 16-02-16 om 10:57:
> On Mon, 15 Feb 2016 06:11:13 +0100
> Martijn Dekker <martijn@inlv.org> wrote:
>> The POSIX_TRAPS option for the EXIT trap does not work if both of the
>> following conditions apply:
>
> Oh, you mean the POSIXness of the trap is not sticky. It should be
> marked as to be run at the end of the programme based on the emulation
> when it was started. I don't think we've ever claimed it would be,
> but it would be clearly be useful to change as the intention in
> POSIX emulation is unambiguous.
That's a much better way of putting it.
> I think that's probably fixable with a bit of flaggery, but we'll
> probably need to compromise and agree that if someone sets an EXIT trap
> once the emulation is left, which therefore picks up non-POSIX behaviour
> (if that's how the shell was started), it will wipe out the trap
> completely. Otherwise we're in a weird and wonderful world of multiple
> parallel EXIT traps. Presumably that's no big issue, since even in
> POSIX mode a new EXIT trap wipes out the one you've just set.
However, since POSIX traps are inherently global, the expectation that
POSIX traps wipe out previous traps only applies to global traps. Could
you make an exception for function-local traps?
IMO, no one would expect a function-local trap to wipe out a global
trap, whether or not the global trap was set in POSIX mode. For
sticky-POSIX library functions to play nice with a native zsh script, it
would be necessary for native function-local traps to still work as
expected.
In other words, in my view, the following ought to work:
#! Src/zsh -f
emulate sh -c 'trap "echo POSIX exit trap triggered" EXIT'
fn() {
trap "echo native zsh function-local exit trap triggered" EXIT
echo entering native zsh function
}
fn
Expected output:
entering program
entering native zsh function
native zsh function-local exit trap triggered
exiting program
POSIX exit trap triggered
With your current patch, the function-local trap wipes out the global
POSIX exit trap, so the last line of expected output doesn't appear.
Thanks,
- M.
next prev parent reply other threads:[~2016-02-16 23:38 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-15 5:11 Martijn Dekker
2016-02-16 9:57 ` Peter Stephenson
2016-02-16 12:46 ` Peter Stephenson
2016-02-16 23:38 ` Martijn Dekker [this message]
2016-02-17 4:25 ` Bart Schaefer
2016-02-17 5:24 ` Martijn Dekker
2016-02-19 18:59 ` Bart Schaefer
2016-02-17 10:36 ` Peter Stephenson
2016-02-25 11:53 ` Peter Stephenson
2016-02-25 13:52 ` Martijn Dekker
2016-02-25 14:17 ` Peter Stephenson
2016-03-04 18:02 ` Peter Stephenson
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=56C3B2E7.1090806@inlv.org \
--to=martijn@inlv.org \
--cc=p.stephenson@samsung.com \
--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).