[Pidgin] #2608: Including sys/sysctl.h requires sys/param.h on NetBSD.
Pidgin
trac at pidgin.im
Thu Jun 12 11:11:12 EDT 2008
#2608: Including sys/sysctl.h requires sys/param.h on NetBSD.
------------------------+---------------------------------------------------
Reporter: gdt | Owner: rlaager
Type: patch | Status: new
Priority: major | Milestone:
Component: libpurple | Version: 2.1.0
Resolution: | Keywords:
Pending: 0 |
------------------------+---------------------------------------------------
Comment (by gdt):
Sorry for the delay - I tried to reply by mail and realized that isn't
supported in trac.
The NetBSD pkgsrc entry has patch that includes sys/param.h:
http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/chat/libpurple/patches/patch-
ab?rev=HEAD&content-type=text/x-cvsweb-markup
I think the correct overall fix is:
autoconf check for sys/types.h sys/param.h as well as sys/sysctl.h
Before using sysctl.h, includes sys/types.h if present, and then
sys/param.h if present, in that order.
So similar to Marcus's patch, but conditionalize sys/types.h inclusion,
and also adjust where sys/sysctl.h is actually included.
--
Ticket URL: <http://developer.pidgin.im/ticket/2608#comment:10>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list