Missing include <signal.h> in avmshell.cpp for solaris.

VERIFIED FIXED

Status

VERIFIED FIXED
11 years ago
9 years ago

People

(Reporter: leon.sha, Assigned: leon.sha)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
"/export/home/leon.sha/uild/tamarin-central/shell/avmshell.cpp", line 734: Error: The function "signal" must have a prototype.
1 Error(s) and 6 Warning(s) detected.

<signal.h> is required for function "signal". On linux signal.h is include by <pthread.h> while on solaris it is not include.

I am surprised that how can I do the build before.
(Assignee)

Comment 1

11 years ago
Created attachment 321552 [details] [diff] [review]
patch
Assignee: nobody → leon.sha
Status: NEW → ASSIGNED
Attachment #321552 - Flags: review?(stejohns)

Updated

11 years ago
Attachment #321552 - Flags: review?(stejohns) → review+
(Assignee)

Comment 2

11 years ago
Pushed.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED

Comment 3

9 years ago
Resolved fixed engineering / work item that has been pushed.  Setting status to verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.