[Pidgin] #14936: Port to Farstream

Pidgin trac at pidgin.im
Tue Apr 3 20:49:35 EDT 2012


#14936: Port to Farstream
-----------------------------------------+----------------------------------
 Reporter:  ocrete                       |        Owner:  Maiku          
     Type:  patch                        |       Status:  new            
Milestone:  Patches Needing Improvement  |    Component:  Voice and Video
  Version:  2.10.1                       |   Resolution:                 
 Keywords:                               |  
-----------------------------------------+----------------------------------

Comment(by rlaager):

 I don't know about that. The change to libpurple/media.c seems like it
 could be accepted immediately, as it seems like a trivial cleanup
 unrelated to farsight->farstream.

 The configure test could support both with minor tweaks. (The farsight
 check could set FARSTREAM_CFLAGS, to avoid extra changes to the
 Makefile.am.)

 The pile of #ifdefs is probably still less code than duplicating
 libpurple/media/backend-fs2.c.

 I'm not sure why we have a dependency on farsight in
 purple_media_manager_get_pipeline(), but I don't really understand the vv
 code because I haven't looked at it much. I agree it'd be nice to get rid
 of that dependency, if possible.

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


More information about the Tracker mailing list