[haiku-bugs] Re: [Haiku] #9046: kill -l command shows incorrect list of signals
- From: "Prasad" <trac@xxxxxxxxxxxx>
- Date: Sat, 06 Oct 2012 19:14:31 -0000
#9046: kill -l command shows incorrect list of signals
-----------------------------------------------+---------------------------
Reporter: Prasad | Owner: nobody
Type: bug | Status: new
Priority: normal | Milestone: R1
Component: Applications/Command Line Tools | Version:
Resolution: | R1/Development
Blocked By: | Keywords: kill, signal
Has a Patch: 0 | Blocking:
| Platform: All
-----------------------------------------------+---------------------------
Comment (by Prasad):
Replying to [comment:2 bonefish]:
> Replying to [comment:1 Prasad]:
> > This seems to be exact list reported on a Linux system, I mean, the
signal name and position matches with Linux machine.
>
> This is rather unsurprising since the tool was built and ran on Linux. I
suspect on Haiku it would print better results.
>
> > AFAIK, this auto-generated list cannot be used, since Haiku uses few
propitiatory signals like 'SIGKILLTHR' for example.
>
> That and not all of the standard signals have the same value.
>
> > So the question I have is, How was the signames_haiku.h generated
initially? was it auto-generated?
>
> I don't recall. The respective [http://cgit.haiku-
os.org/haiku/commit/?id=d348ab947fdc4a0adb384457184d6c4655a8a847 commit]
says hand-edited, but it may have been generated on Haiku.
>
> > If it was not auto-generated, would it be okay if I manually edit the
file and add SIGBUS at 30'th location?
>
> The array needs quite a bit more editing, since the signal number has
changed as well and real-time signals have been added. I guess the easiest
solution is to generate the header on Haiku and verify that it looks OK.
Here is the diff of the header file generated on Haiku machine. This diff
seems to be correct
{{{
prasad@pjoshi:~/sources/Haiku/haiku$ git diff
diff --git a/src/bin/bash/signames_haiku.h b/src/bin/bash/signames_haiku.h
index 8125c7a..784b80a 100644
--- a/src/bin/bash/signames_haiku.h
+++ b/src/bin/bash/signames_haiku.h
@@ -33,6 +33,41 @@ char *signal_names[NSIG + 4] = {
"SIGVTALRM",
"SIGXCPU",
"SIGXFSZ",
+ "SIGBUS",
+ "SIGJUNK(31)",
+ "SIGJUNK(32)",
+ "SIGRTMIN",
+ "SIGRTMIN+1",
+ "SIGRTMIN+2",
+ "SIGRTMIN+3",
+ "SIGRTMAX-3",
+ "SIGRTMAX-2",
+ "SIGRTMAX-1",
+ "SIGRTMAX",
+ "SIGJUNK(41)",
+ "SIGJUNK(42)",
+ "SIGJUNK(43)",
+ "SIGJUNK(44)",
+ "SIGJUNK(45)",
+ "SIGJUNK(46)",
+ "SIGJUNK(47)",
+ "SIGJUNK(48)",
+ "SIGJUNK(49)",
+ "SIGJUNK(50)",
+ "SIGJUNK(51)",
+ "SIGJUNK(52)",
+ "SIGJUNK(53)",
+ "SIGJUNK(54)",
+ "SIGJUNK(55)",
+ "SIGJUNK(56)",
+ "SIGJUNK(57)",
+ "SIGJUNK(58)",
+ "SIGJUNK(59)",
+ "SIGJUNK(60)",
+ "SIGJUNK(61)",
+ "SIGJUNK(62)",
+ "SIGJUNK(63)",
+ "SIGJUNK(64)",
"DEBUG",
"ERR",
"RETURN",
}}}
I will verify once and will attach a patch.
--
Ticket URL: <http://dev.haiku-os.org/ticket/9046#comment:3>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.
Other related posts:
