[Pidgin] #3798: portability patch

Pidgin trac at pidgin.im
Sun May 25 19:09:01 EDT 2008


#3798: portability patch
---------------------------+------------------------------------------------
  Reporter:  pogma         |       Owner:  rlaager
      Type:  patch         |      Status:  new    
  Priority:  minor         |   Milestone:         
 Component:  unclassified  |     Version:  2.2.2  
Resolution:                |    Keywords:         
   Pending:  1             |  
---------------------------+------------------------------------------------
Changes (by rlaager):

  * pending:  0 => 1

Comment:

 I think parts of this are ready to commit and other parts can be dropped
 entirely:

 The libgcrypt guys don't seem interested in changing this. I don't like
 that they're calling abort() (instead of exit() like they were at the time
 of previous comments), but that's the way it is. That said, why are you
 compiling in SSL support for machines with no way of using it? I think I'd
 rather leave those changes out.

 Not that it's likely to conflict with anything, but we should use
 something else instead of __use_fork__.

 HAVE_STRUCT_SOCKADDR_SA_LEN has an extra underscore after HAVE in the
 configure script.

 The changes to gtkmain.c should be obsoleted by newer GStreamer versions,
 as the comment in gtkmain.c says now.


 What real name would you like me to use for COPYRIGHT (if you're not
 already listed)? What e-mail address should I use for the commit?

 The config.h at the top of every file seems like something that needs more
 investigation. Do you know why that was required?

-- 
Ticket URL: <http://developer.pidgin.im/ticket/3798#comment:9>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list