* [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps.
@ 2014-06-11 23:08 Adam Buchbinder
2014-06-11 23:08 ` [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in man page Adam Buchbinder
2014-06-11 23:13 ` [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Jonathan Nieder
0 siblings, 2 replies; 7+ messages in thread
From: Adam Buchbinder @ 2014-06-11 23:08 UTC (permalink / raw)
To: dash; +Cc: jrnieder, Adam Buchbinder
I saw a discussion in which there was some confusion over whether or not
you can use a symbolic name, since the manpage doesn't specifically say so.
Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>
---
ChangeLog | 4 ++++
src/dash.1 | 4 +++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index 2a39e34..81df8d1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-11 Adam Buchbinder <adam.buchbinder@gmail.com>
+
+ * Clarify that 0 and EXIT are both acceptable for exit traps.
+
2013-08-23 Herbert Xu <herbert@gondor.apana.org.au>
* Propagate EXP_QPAT in subevalvar.
diff --git a/src/dash.1 b/src/dash.1
index 6241a61..a07a52d 100644
--- a/src/dash.1
+++ b/src/dash.1
@@ -2071,7 +2071,9 @@ The signals are specified by signal number or as the name of the signal.
If
.Ar signal
is
-.Li 0 ,
+.Li 0
+or
+.Li EXIT ,
the action is executed when the shell exits.
.Ar action
may be null, which cause the specified signals to be ignored.
--
2.0.0.526.g5318336
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in man page
2014-06-11 23:08 [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Adam Buchbinder
@ 2014-06-11 23:08 ` Adam Buchbinder
2014-06-11 23:13 ` [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Jonathan Nieder
1 sibling, 0 replies; 7+ messages in thread
From: Adam Buchbinder @ 2014-06-11 23:08 UTC (permalink / raw)
To: dash; +Cc: jrnieder
From: Jonathan Nieder <jrnieder@gmail.com>
When the manpage states
| <action> may be null, which cause the specified signals to be ignored.
it is not immediately obvious what it means for an action to be
null. Clarify by explicitly referring to an empty string, as
opposed to a NULL pointer or the string "null".
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
ChangeLog | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/ChangeLog b/ChangeLog
index 81df8d1..5559a24 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-11 Jonathan Nieder <jrnieder@gmail.com>
+
+ * Clarify "trap '' signals" syntax in manual page.
+
2014-06-11 Adam Buchbinder <adam.buchbinder@gmail.com>
* Clarify that 0 and EXIT are both acceptable for exit traps.
--
2.0.0.526.g5318336
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps.
2014-06-11 23:08 [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Adam Buchbinder
2014-06-11 23:08 ` [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in man page Adam Buchbinder
@ 2014-06-11 23:13 ` Jonathan Nieder
2014-06-11 23:17 ` Jonathan Nieder
1 sibling, 1 reply; 7+ messages in thread
From: Jonathan Nieder @ 2014-06-11 23:13 UTC (permalink / raw)
To: Adam Buchbinder; +Cc: dash, Adam Buchbinder
Adam Buchbinder wrote:
> Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>
> ---
> ChangeLog | 4 ++++
> src/dash.1 | 4 +++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps.
2014-06-11 23:13 ` [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Jonathan Nieder
@ 2014-06-11 23:17 ` Jonathan Nieder
2014-06-11 23:20 ` Jonathan Nieder
2014-06-11 23:21 ` [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in manual page Jonathan Nieder
0 siblings, 2 replies; 7+ messages in thread
From: Jonathan Nieder @ 2014-06-11 23:17 UTC (permalink / raw)
To: Adam Buchbinder; +Cc: dash, Adam Buchbinder
Jonathan Nieder wrote:
> Adam Buchbinder wrote:
>> Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>
>> ---
>> ChangeLog | 4 ++++
>> src/dash.1 | 4 +++-
>> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Mph, on second thought this still seems to have a trailing tab
for some reason. Patch 2 seems to be missing the change to
src/dash.1. Here are the same patches again with that fixed.
Thanks,
Adam Buchbinder (1):
[MAN] 0 and EXIT both work for exit traps.
Jonathan Nieder (1):
[MAN] Clarify "trap '' signals" syntax in manual page
ChangeLog | 4 ++++
src/dash.1 | 8 ++++++--
2 files changed, 10 insertions(+), 2 deletions(-)
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps.
2014-06-11 23:17 ` Jonathan Nieder
@ 2014-06-11 23:20 ` Jonathan Nieder
2014-09-26 14:21 ` Herbert Xu
2014-06-11 23:21 ` [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in manual page Jonathan Nieder
1 sibling, 1 reply; 7+ messages in thread
From: Jonathan Nieder @ 2014-06-11 23:20 UTC (permalink / raw)
To: Adam Buchbinder; +Cc: dash, Adam Buchbinder
From: Adam Buchbinder <adam.buchbinder@gmail.com>
I saw a discussion in which there was some confusion over whether or not
you can use a symbolic name, since the manpage doesn't specifically say so.
Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
ChangeLog | 4 ++++
src/dash.1 | 4 +++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index 2a39e34..3f59ac2 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-11 Adam Buchbinder <adam.buchbinder@gmail.com>
+
+ * Clarify that 0 and EXIT are both acceptable for exit traps.
+
2013-08-23 Herbert Xu <herbert@gondor.apana.org.au>
* Propagate EXP_QPAT in subevalvar.
diff --git a/src/dash.1 b/src/dash.1
index 6241a61..a07a52d 100644
--- a/src/dash.1
+++ b/src/dash.1
@@ -2071,7 +2071,9 @@ The signals are specified by signal number or as the name of the signal.
If
.Ar signal
is
-.Li 0 ,
+.Li 0
+or
+.Li EXIT ,
the action is executed when the shell exits.
.Ar action
may be null, which cause the specified signals to be ignored.
--
2.0.0.526.g5318336
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in manual page
2014-06-11 23:17 ` Jonathan Nieder
2014-06-11 23:20 ` Jonathan Nieder
@ 2014-06-11 23:21 ` Jonathan Nieder
1 sibling, 0 replies; 7+ messages in thread
From: Jonathan Nieder @ 2014-06-11 23:21 UTC (permalink / raw)
To: Adam Buchbinder; +Cc: dash, Adam Buchbinder
When the manpage states
| <action> may be null, which cause the specified signals to be ignored.
it is not immediately obvious what it means for an action to be
null. Clarify by explicitly referring to an empty string, as
opposed to a NULL pointer or the string "null".
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
ChangeLog | 4 ++++
src/dash.1 | 4 +++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index 3f59ac2..765b9b6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-11 Jonathan Nieder <jrnieder@gmail.com>
+
+ * Clarify "trap '' signals" syntax in manual page.
+
2014-06-11 Adam Buchbinder <adam.buchbinder@gmail.com>
* Clarify that 0 and EXIT are both acceptable for exit traps.
diff --git a/src/dash.1 b/src/dash.1
index a07a52d..68bd5fe 100644
--- a/src/dash.1
+++ b/src/dash.1
@@ -2076,7 +2076,9 @@ or
.Li EXIT ,
the action is executed when the shell exits.
.Ar action
-may be null, which cause the specified signals to be ignored.
+may be empty
+.Li ( "''" ) ,
+which causes the specified signals to be ignored.
With
.Ar action
omitted or set to `-' the specified signals are set to their default action.
--
2.0.0.526.g5318336
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-09-26 14:21 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-11 23:08 [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Adam Buchbinder
2014-06-11 23:08 ` [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in man page Adam Buchbinder
2014-06-11 23:13 ` [PATCH 1/2] [MAN] 0 and EXIT both work for exit traps Jonathan Nieder
2014-06-11 23:17 ` Jonathan Nieder
2014-06-11 23:20 ` Jonathan Nieder
2014-09-26 14:21 ` Herbert Xu
2014-06-11 23:21 ` [PATCH 2/2] [MAN] Clarify "trap '' signals" syntax in manual page Jonathan Nieder
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).