Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

print name of uncaught signals #286

Open
wants to merge 1 commit into
base: bleeding
Choose a base branch
from
Open

Conversation

a1lu
Copy link
Contributor

@a1lu a1lu commented Feb 2, 2019

uses built-in kill command of sh to generate the list of signal names. If
sh is not available the signal number is printed

refs #128

@a1lu a1lu force-pushed the bleeding branch 8 times, most recently from a273d58 to 8ff064e Compare February 5, 2019 21:11
uses built-in kill command of sh to generate the list of signal names. If
sh is not available the signal number is printed
@@ -0,0 +1,30 @@
#!/bin/sh
Copy link
Contributor

@am11 am11 Oct 30, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can just implement it in C by simply using #include <signal.h>, without shelling out the introspection. e.g.:
https://github.com/karelzak/util-linux/blob/917f53cf13c36d32c175f80f2074576595830573/lib/signames.c#L55-L132

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could do that if we only support posix signals.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Snaipe Whats your opinion on this? Posix or the local signals?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants