[Bug 26866] Conn.I.Addressing etc. — requesting handles for a vCard field or URI

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=26866

--- Comment #79 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2012-02-15 16:58:06 PST ---
(In reply to comment #74)
> Versioned interfaces were mostly wjt's idea, I think. Will, if I'm not understanding this correctly, please correct me :-)

I'm not Will, but I can pretend to be him.

I've seen you've pushed this change to master. I personally don't really care what you do with versioning in master, but I do not agree with this way for next.
It's way way way way too complicated. I find it extremely difficult to read the four paragraphs you wrote in that README without losing interest and going to
drink some free coke.

I think tp:causes-havoc should just be noticed by the HTML generation code and display a pretty warning at the top of the generated HTML and that's it -- don't
change any codegen behaviour on it.

In next, *every* non-core interface is versioned. Did you see that?

I think in next, tp-glib should generate code for everything, depending on the interface name, not the node name. So, generate
tp_svc_connection_interface_vengaboys1_call_go_to_ibiza for im.telepathy1.Connection.Interface.Vengaboys1.GoToIbiza in node Connection_Interface_Vengaboys. I
guess we'll need to version node names so we can generate code for both versions of Conn.Iface.Vengaboys{1,2} but that's an implementation detail.

So just generate code for everything. If something's not ready at all for any code, then don't put it in the spec at all... If we come out with
Channel.Iface.RoomConfig19 then who cares? tp_svc_channel_interface_room_config19_… looks ugly but no-one cares. This will be in the lowlevel library which
only tp-glib highlevel should be using anyway. It'll look a bit ugly in the dbus-monitor logs too, but I don't care about that either.

So, yeah, make things reaaaaaaally simple. What's wrong with this?

I still haven't managed to get through those instructions in the README. Feel free to keep it for master though.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.