Bug 69540

Summary: Some misc 'next' tweaks
Product: Telepathy Reporter: Xavier Claessens <xclaesse>
Component: tp-glibAssignee: Telepathy bugs list <telepathy-bugs>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: normal    
Priority: medium Keywords: patch
Version: git master   
Hardware: Other   
OS: All   
URL: http://cgit.collabora.com/git/user/xclaesse/telepathy-glib.git/log/?h=next-misc
Whiteboard:
i915 platform: i915 features:

Description Xavier Claessens 2013-09-18 18:24:44 UTC
Opening a bug to get review comments...
Comment 1 Simon McVittie 2013-09-18 18:29:01 UTC
19:20 < smcv> 10:18 < smcv> + libtelepathy-glib-1-core.la \
19:20 < smcv> 10:18 < smcv> xclaesse: ^ does that result in the "core" symbols 
              being  duplicated in TelepathyGLibDBus-1.gir?
19:20 < smcv> 10:18 < smcv> xclaesse: (look at the .gir)
Comment 2 Simon McVittie 2013-09-18 18:30:32 UTC
> No need to pretend we are in telepathy-glib.h when we are in telepathy-glib-d...

OK

> Update UPGRADING file

OK

> protocol name can contain '_' but not '-'

OK when tests pass :-)
Comment 3 Xavier Claessens 2013-09-18 19:05:11 UTC
(In reply to comment #2)
> > No need to pretend we are in telepathy-glib.h when we are in telepathy-glib-d...
> 
> OK

pushed.

> > Update UPGRADING file
> 
> OK

pushed

> > protocol name can contain '_' but not '-'
> 
> OK when tests pass :-)

Unit tests pass now, branch updated. OK to push ?
Comment 4 Simon McVittie 2013-09-19 15:10:11 UTC
(In reply to comment #3)
> Unit tests pass now, branch updated. OK to push ?

Yes please!
Comment 5 Xavier Claessens 2013-09-19 15:13:13 UTC
Thanks.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.