[Home]

Summary:ASTERISK-03427: latest cvs head compile fails for openbsd
Reporter:mxhess (mxhess)Labels:
Date Opened:2005-02-03 08:13:54.000-0600Date Closed:2008-01-15 15:24:15.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) openbsd.patch
( 1) patch1.txt
( 2) patch2.txt
Description:few minor errors with pbx_dundi.c and app_curl.c.. solved with attached patch
Comments:By: twisted (twisted) 2005-02-03 08:44:32.000-0600

Please provide a diff -u, and if possible, post with a .txt file extension.  Also, please confirm your disclaimer status.

By: mxhess (mxhess) 2005-02-03 09:21:14.000-0600

individual diff -u files now attached with .txt extensions.
Disclaimer status? Hmm, I don't do anything less than BSD license.. I'll fax in the 'official' disclaimer later today so everyone has a warm fuzzy..

By: twisted (twisted) 2005-02-03 12:34:54.000-0600

is the default path for OpenBSD in the makefile always /usr/local/include?  Also, the second part of the pbx_dundi.c patch,

-#if defined(__NetBSD__) && !defined(SOLARIS)
+#if defined(__NetBSD__) && !defined(SOLARIS) && !defined(__OpenBSD__)

OpenBSD shouldn't work the same as NetBSD in this situation?

By: mxhess (mxhess) 2005-02-03 13:23:23.000-0600

well.. most *BSD ports put their files in happy/friendly places like /usr/local/<whatever> and the curl application was being detected but the port include file is at /usr/local/include/curl/curl.h which was not being picked up by the compile. My thought process was that if we have curl it is highly likely to be installed from the port.. and if the OS is openbsd to go ahead and include that /usr/local/include location.. I'm open to cleaner suggestions.

you are correct.. I don't know how that little guy snuck in there but the behavior for that line you are referencing should indeed be like netbsd.

this is what I get for being bored, sick, and awake at 3 a.m.
:)

By: twisted (twisted) 2005-02-03 18:13:14.000-0600

Fixed in CVS HEAD with minor adjustments.

By: Digium Subversion (svnbot) 2008-01-15 15:24:15.000-0600

Repository: asterisk
Revision: 4959

U   trunk/apps/Makefile
U   trunk/pbx/pbx_dundi.c

------------------------------------------------------------------------
r4959 | twisted | 2008-01-15 15:24:15 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix build on OpenBSD and fix small typo.  (Bug ASTERISK-3427)

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

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