Merge lp:~dobey/ubuntu/precise/avahi/bring-back-gir into lp:ubuntu/precise/avahi

Proposed by dobey
Status: Work in progress
Proposed branch: lp:~dobey/ubuntu/precise/avahi/bring-back-gir
Merge into: lp:ubuntu/precise/avahi
Diff against target: 85 lines (+39/-0)
4 files modified
debian/changelog (+8/-0)
debian/control (+27/-0)
debian/gir1.2-avahi-0.6.install (+2/-0)
debian/libavahi-gobject-dev.install (+2/-0)
To merge this branch: bzr merge lp:~dobey/ubuntu/precise/avahi/bring-back-gir
Reviewer Review Type Date Requested Status
Michael Terry Needs Fixing
Ubuntu branches Pending
Review via email: mp+80103@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

I tried building this in a pbuilder and got:

make[5]: *** No rule to make target `AvahiCore-0.6.typelib', needed by `all-am'. Stop.

review: Needs Fixing
Revision history for this message
dobey (dobey) wrote :

Apparently libgirepository-dev doesn't pull in gobject-introspection, so had to add that too.

Revision history for this message
Michael Terry (mterry) wrote :

Shouldn't AvahiCore.gir be in libavahi-core-dev instead of libavahi-gobject-dev? And a similar split for the gir1.2-* packages right?

Revision history for this message
dobey (dobey) wrote :

When they were packaged from the old gir-repository package before, they were both in gir1.0-avahi-0.6, with the .gir files in gir-repository-dev.

Also, in the source tree, they are both under the avahi-gobject/ directory, and gobject-introspection still requires libgobject to work, even though libavahi-core doesn't. So the gobject-dev package seems to be the most appropriate place in the avahi binary packaging.

Oh, I just realized though. I guess I probably need to Conflicts/Replaces gir-repository-dev from the libavahi-gobject-dev package as well, no?

Revision history for this message
Colin Watson (cjwatson) wrote :

If you're moving files from one package to another (even if there was a
period when the files were in neither package), please use
Breaks/Replaces rather than Conflicts/Replaces.

Revision history for this message
Michael Terry (mterry) wrote :

I still disagree that it makes sense to bundle the bindings. They have different namespaces and different associated libraries. True, you'll need gobject-introspection to understand the gir, but that is a different thing than needing libavahi-gobject. Imagine an app that only wants to use libavahi-core.

But maybe we need a third opinion.

110. By dobey

Use Breaks/Replaces for the old gir-repository packages

Revision history for this message
dobey (dobey) wrote :

Well, glib, GTK, GStreamer, and Pango aren't split up in that way for the various different libraries they install, and this preserves the way they used to be packaged in debian from gir-repository. It doesn't seem appropriate, and an app only using avahi-core I think is more likely to use the available static bindings, than use gobject-introspection without also using the GObject-based API.

It seems the manually created AvahiCore-0.6.gir only exists to satisfy the dependency needs of the auto-generated Avahi-0.6.gir. Also, it /is/ Avahi-0.6.gir and not AvahiGObject-0.6.gir, which suggests that for using avahi with gobject-introspection, one really should be using Avahi, and not AvahiCore.

Revision history for this message
dobey (dobey) wrote :

Also, AvahiCore-0.6.gir only includes a few type definitions, and one single function, so probably not useful to use by itself. :)

Revision history for this message
Michael Terry (mterry) wrote :

An update on this: since Debian hasn't included the girs yet, I agree that it doesn't make sense to innovate around the naming ourselves. So I'm OK with the naming for now. But I did notice that when built, the typelib files get put in a multiarch location, which Python does not pick up. That should be looked into.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Could you send your patch to Debian as well? It would be better if they reviewed and merged it there and spare us to have to decide on the binaries to use and the naming

Revision history for this message
Sebastien Bacher (seb128) wrote :

Rodney, could you send it to debian?

Revision history for this message
Stéphane Graber (stgraber) wrote :

No activity from author for a months. Please answer the questions above and mark the branch as needs merging again.

Thanks

Unmerged revisions

110. By dobey

Use Breaks/Replaces for the old gir-repository packages

109. By dobey

Also need gobject-introspection in build-depends

108. By dobey

Add gir1.2-glib-2.0 to build-depends also

107. By dobey

Add bug to changelog

106. By dobey

* Bring back the gir interface for Avahi
  - gir1.2-avahi-0.6: package for Avahi gir typelibs
  - libavahi-gobject-dev: include the .gir for Avahi

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-10-18 14:46:25 +0000
3+++ debian/changelog 2011-10-27 13:23:25 +0000
4@@ -1,3 +1,11 @@
5+avahi (0.6.30-5ubuntu2) precise; urgency=low
6+
7+ * Bring back the gir interface for Avahi (LP: #879647)
8+ - gir1.2-avahi-0.6: package for Avahi gir typelibs
9+ - libavahi-gobject-dev: include the .gir for Avahi
10+
11+ -- Rodney Dawes <rodney.dawes@ubuntu.com> Fri, 21 Oct 2011 15:52:17 -0400
12+
13 avahi (0.6.30-5ubuntu1) precise; urgency=low
14
15 * Merge from Debian testing. Remaining changes:
16
17=== modified file 'debian/control'
18--- debian/control 2011-10-18 14:46:25 +0000
19+++ debian/control 2011-10-27 13:23:25 +0000
20@@ -18,6 +18,9 @@
21 libexpat-dev,
22 libdaemon-dev (>= 0.11),
23 libdbus-1-dev (>= 0.60),
24+ gobject-introspection (>= 0.9.5),
25+ libgirepository1.0-dev (>= 0.9.5),
26+ gir1.2-glib-2.0,
27 python-all-dev (>= 2.3.5-11),
28 python-gdbm (>= 2.4.3),
29 python-dbus,
30@@ -339,14 +342,38 @@
31 .
32 This library contains the GObject wrapper for Avahi.
33
34+Package: gir1.2-avahi-0.6
35+Section: libs
36+Architecture: any
37+Depends: ${gir:Depends},
38+ ${shlibs:Depends},
39+ ${misc:Depends},
40+ libavahi-gobject0 (= ${binary:Version})
41+Breaks: gir1.0-avahi-0.6
42+Replaces: gir1.0-avahi-0.6
43+Description: GObject Introspection data for the Avahi library
44+ This package contains introspection data for the Avahi library.
45+ .
46+ Avahi is a fully LGPL framework for Multicast DNS Service Discovery.
47+ It allows programs to publish and discover services and hosts
48+ running on a local network with no specific configuration. For
49+ example you can plug into a network and instantly find printers to
50+ print to, files to look at and people to talk to.
51+ .
52+ It can be used by packages using the GIRepository format to generate
53+ dynamic bindings.
54+
55 Package: libavahi-gobject-dev
56 Section: libdevel
57 Architecture: any
58 Depends: libavahi-gobject0 (= ${binary:Version}),
59+ gir1.2-avahi-0.6 (= ${binary:Version}),
60 ${misc:Depends},
61 libglib2.0-dev,
62 libavahi-client-dev,
63 libavahi-glib-dev
64+Breaks: gir-repository-dev
65+Replaces: gir-repository-dev
66 Description: Development headers for the Avahi GObject library
67 Avahi is a fully LGPL framework for Multicast DNS Service Discovery.
68 It allows programs to publish and discover services and hosts
69
70=== added file 'debian/gir1.2-avahi-0.6.install'
71--- debian/gir1.2-avahi-0.6.install 1970-01-01 00:00:00 +0000
72+++ debian/gir1.2-avahi-0.6.install 2011-10-27 13:23:25 +0000
73@@ -0,0 +1,2 @@
74+usr/lib/*/girepository-*/Avahi*.typelib
75+
76
77=== modified file 'debian/libavahi-gobject-dev.install'
78--- debian/libavahi-gobject-dev.install 2011-07-01 13:29:42 +0000
79+++ debian/libavahi-gobject-dev.install 2011-10-27 13:23:25 +0000
80@@ -2,3 +2,5 @@
81 usr/lib/*/libavahi-gobject.so
82 usr/lib/*/libavahi-gobject.a
83 usr/lib/*/pkgconfig/avahi-gobject.pc
84+usr/share/gir-*/*.gir
85+

Subscribers

People subscribed via source and target branches

to all changes: