[Home]

Summary:ASTERISK-10941: [patch] Small build issue on solaris
Reporter:snuffy (snuffy)Labels:
Date Opened:2007-11-30 16:50:10.000-0600Date Closed:2007-12-06 14:49:57.000-0600
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20071203__bug11430.diff.txt
( 1) smallfix.diff
Description:When configure checks for getloadavg its detected but solaris stores getloadavg inside the sys/loadavg.h header.
This is a small change to compat.h to reflect this.

Happens with both trunk and 1.4 svn
Comments:By: Tilghman Lesher (tilghman) 2007-12-03 15:30:17.000-0600

I think I'd prefer to change the Solaris compat file, as in this patch.  Could you please test this and ensure that it works?
By: snuffy (snuffy) 2007-12-03 16:53:37.000-0600

Corydon it appears that solaris-compat/compat.h is no longer used.
All defines etc that appear in compat.h under solaris dir are now included in the main compat.h (at least with trunk)


By: Digium Subversion (svnbot) 2007-12-03 17:48:12.000-0600

Repository: asterisk
Revision: 90753

U   branches/1.4/include/asterisk/compat.h

------------------------------------------------------------------------
r90753 | tilghman | 2007-12-03 17:48:12 -0600 (Mon, 03 Dec 2007) | 5 lines

Solaris requires the inclusion of sys/loadavg.h for getloadavg().
Reported by: snuffy
Patch by: snuffy,tilghman
(Closes issue ASTERISK-10941)

------------------------------------------------------------------------
By: Digium Subversion (svnbot) 2007-12-03 17:49:49.000-0600

Repository: asterisk
Revision: 90760

_U  trunk/
U   trunk/include/asterisk/compat.h

------------------------------------------------------------------------
r90760 | tilghman | 2007-12-03 17:49:48 -0600 (Mon, 03 Dec 2007) | 13 lines

Merged revisions 90753 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r90753 | tilghman | 2007-12-03 17:50:51 -0600 (Mon, 03 Dec 2007) | 5 lines

Solaris requires the inclusion of sys/loadavg.h for getloadavg().
Reported by: snuffy
Patch by: snuffy,tilghman
(Closes issue ASTERISK-10941)

........

------------------------------------------------------------------------
By: Digium Subversion (svnbot) 2007-12-06 14:49:57.000-0600

Repository: asterisk
Revision: 91561

_U  trunk/
U   trunk/cdr/cdr_pgsql.c

------------------------------------------------------------------------
r91561 | tilghman | 2007-12-06 14:49:57 -0600 (Thu, 06 Dec 2007) | 27 lines

Merged revisions 90166,90736,90753 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r90166 | tilghman | 2007-11-29 13:48:10 -0600 (Thu, 29 Nov 2007) | 3 lines

Properly escape cdr->src and cdr->dst and ensure we use thread-safe escaping
(Fixes AST-2007-026)

........
r90736 | tilghman | 2007-12-03 17:23:55 -0600 (Mon, 03 Dec 2007) | 5 lines

If both dbhost and dbsock were not set, a NULL deref could result
Reported by: xrg
Patch by: tilghman
(Closes issue ASTERISK-10898)

........
r90753 | tilghman | 2007-12-03 17:50:51 -0600 (Mon, 03 Dec 2007) | 5 lines

Solaris requires the inclusion of sys/loadavg.h for getloadavg().
Reported by: snuffy
Patch by: snuffy,tilghman
(Closes issue ASTERISK-10941)

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=91561