Opened 9 years ago

Closed 9 years ago

#1603 closed defect (fixed)

sugar should advertise it's file transfer capabilities

Reported by: tomeu Owned by: tomeu
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Sugar Version: Unspecified
Severity: Unspecified Keywords:
Cc: robot101, gdesmott Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

This will be required in platforms with telepathy-gabble 0.9.

(07:03:34 PM) Robot101: tomeu: we realised that we advertised the file transfer ability always, even when there was no client who said they could do it, which is a bug on eg N900 which has no file transfer UI - the files go there and time out...
(07:03:59 PM) Robot101: tomeu: so we need to just do a quick hack in p-s to set the capabilities on the new API if its there, to say we can do file transfer
(07:04:15 PM) Robot101: we're not recommending it atm but we should fix it up in sugar asap so we don't need a hack mode in 0.9
(07:04:37 PM) Robot101: in 0.8 we've got a hack for the N900 which turns off the internal assumed file transfer capability
(07:04:48 PM) Robot101: but that hack would break Sugar or Empathy 2.28.x

Attachments (1)

0001-Announce-our-file-transfer-capabilities-1603.patch (2.0 KB) - added by tomeu 9 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 9 years ago by tomeu

This is my understanding of what it's needed, but right now it fails for both connection managers with a:

KeyError: 'org.freedesktop.Telepathy.Connection.Interface.ContactCapabilities'

Can investigate further tomorrow, but would be nice if someone could give it a look and say if something obvious is wrong there.

comment:2 Changed 9 years ago by gdesmott

(CLIENT + ".FtExample", [ 

You should probably use something like .Sugar.Ft or something.

Which Gabble are you using? The final ContactCapabilities API is implemented only in Gabble 0.9.x. 0.8.x doesn't support it (it implements an old draft actually) but that should be fine as long it hasn't been build with --disable-assumed-ft-cap which would break Empathy 2.28.x so I guess Fedora won't do it for now.

comment:3 Changed 9 years ago by gdesmott

  • Cc gdesmott added

comment:4 Changed 9 years ago by sascha_silbe

Is this still relevant?

comment:5 Changed 9 years ago by tomeu

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.