[Pidgin] #393: Add receive Handwritten messages, Winks and Voice Clips to MSN

Pidgin trac at pidgin.im
Wed May 2 07:06:08 EDT 2007


#393: Add receive Handwritten messages, Winks and Voice Clips to MSN
------------------------+---------------------------------------------------
  Reporter:  notak      |       Owner:  khc                 
      Type:  patch      |      Status:  assigned            
  Priority:  minor      |   Milestone:                      
 Component:  libpurple  |     Version:  2.0                 
Resolution:             |    Keywords:  msn handwritten wink
   Pending:  0          |  
------------------------+---------------------------------------------------
Comment (by notak):

 > Yes, you're doing: g_free(f); where f is a FILE *.

 Excellent, I'll fix that then.

 > What is libmspack? apt-cache search mspack is not returning any result
 on ubuntu.

 http://www.kyz.uklinux.net/libmspack/
 It's statically linked within cabextract and wine so it doesn't get
 deployed in most dists as a separate target. The only way I can figure to
 do this without the awkward dep is:
  * include the library in our tree (waste of space - it's bigger than it
 needs to be)
  * hack the required code straight in (It's only ~200 lines, but I think
 that would contravene their license)
  * exec cabextract if that's there, which is probably uglier than the dep.

 My feeling is that it's easy to package for windows, reasonably easy for
 people who are building themselves to satisfy the dep if they want it, and
 the #ifdefs mean that binary builds will just not contain the code at all
 (unless any of the distros get bored and decide to package mspack I
 guess). Don't know how that plays with anybody else

 On the subject of voiceclip: uris. I guess it could be done in half a
 dozen lines (check for URI starting voiceclip:, grab following filename,
 purple_sound_play_file it). Is it likely to be acceptable to just put this
 directly into purple_handle_uri before the call to ui_ops?

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


More information about the Tracker mailing list