[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