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
=== modified file 'debian/changelog'
--- debian/changelog 2011-10-18 14:46:25 +0000
+++ debian/changelog 2011-10-27 13:23:25 +0000
@@ -1,3 +1,11 @@
1avahi (0.6.30-5ubuntu2) precise; urgency=low
2
3 * Bring back the gir interface for Avahi (LP: #879647)
4 - gir1.2-avahi-0.6: package for Avahi gir typelibs
5 - libavahi-gobject-dev: include the .gir for Avahi
6
7 -- Rodney Dawes <rodney.dawes@ubuntu.com> Fri, 21 Oct 2011 15:52:17 -0400
8
1avahi (0.6.30-5ubuntu1) precise; urgency=low9avahi (0.6.30-5ubuntu1) precise; urgency=low
210
3 * Merge from Debian testing. Remaining changes:11 * Merge from Debian testing. Remaining changes:
412
=== modified file 'debian/control'
--- debian/control 2011-10-18 14:46:25 +0000
+++ debian/control 2011-10-27 13:23:25 +0000
@@ -18,6 +18,9 @@
18 libexpat-dev,18 libexpat-dev,
19 libdaemon-dev (>= 0.11),19 libdaemon-dev (>= 0.11),
20 libdbus-1-dev (>= 0.60),20 libdbus-1-dev (>= 0.60),
21 gobject-introspection (>= 0.9.5),
22 libgirepository1.0-dev (>= 0.9.5),
23 gir1.2-glib-2.0,
21 python-all-dev (>= 2.3.5-11),24 python-all-dev (>= 2.3.5-11),
22 python-gdbm (>= 2.4.3),25 python-gdbm (>= 2.4.3),
23 python-dbus,26 python-dbus,
@@ -339,14 +342,38 @@
339 .342 .
340 This library contains the GObject wrapper for Avahi.343 This library contains the GObject wrapper for Avahi.
341344
345Package: gir1.2-avahi-0.6
346Section: libs
347Architecture: any
348Depends: ${gir:Depends},
349 ${shlibs:Depends},
350 ${misc:Depends},
351 libavahi-gobject0 (= ${binary:Version})
352Breaks: gir1.0-avahi-0.6
353Replaces: gir1.0-avahi-0.6
354Description: GObject Introspection data for the Avahi library
355 This package contains introspection data for the Avahi library.
356 .
357 Avahi is a fully LGPL framework for Multicast DNS Service Discovery.
358 It allows programs to publish and discover services and hosts
359 running on a local network with no specific configuration. For
360 example you can plug into a network and instantly find printers to
361 print to, files to look at and people to talk to.
362 .
363 It can be used by packages using the GIRepository format to generate
364 dynamic bindings.
365
342Package: libavahi-gobject-dev366Package: libavahi-gobject-dev
343Section: libdevel367Section: libdevel
344Architecture: any368Architecture: any
345Depends: libavahi-gobject0 (= ${binary:Version}),369Depends: libavahi-gobject0 (= ${binary:Version}),
370 gir1.2-avahi-0.6 (= ${binary:Version}),
346 ${misc:Depends},371 ${misc:Depends},
347 libglib2.0-dev,372 libglib2.0-dev,
348 libavahi-client-dev,373 libavahi-client-dev,
349 libavahi-glib-dev374 libavahi-glib-dev
375Breaks: gir-repository-dev
376Replaces: gir-repository-dev
350Description: Development headers for the Avahi GObject library377Description: Development headers for the Avahi GObject library
351 Avahi is a fully LGPL framework for Multicast DNS Service Discovery.378 Avahi is a fully LGPL framework for Multicast DNS Service Discovery.
352 It allows programs to publish and discover services and hosts379 It allows programs to publish and discover services and hosts
353380
=== added file 'debian/gir1.2-avahi-0.6.install'
--- debian/gir1.2-avahi-0.6.install 1970-01-01 00:00:00 +0000
+++ debian/gir1.2-avahi-0.6.install 2011-10-27 13:23:25 +0000
@@ -0,0 +1,2 @@
1usr/lib/*/girepository-*/Avahi*.typelib
2
03
=== modified file 'debian/libavahi-gobject-dev.install'
--- debian/libavahi-gobject-dev.install 2011-07-01 13:29:42 +0000
+++ debian/libavahi-gobject-dev.install 2011-10-27 13:23:25 +0000
@@ -2,3 +2,5 @@
2usr/lib/*/libavahi-gobject.so2usr/lib/*/libavahi-gobject.so
3usr/lib/*/libavahi-gobject.a3usr/lib/*/libavahi-gobject.a
4usr/lib/*/pkgconfig/avahi-gobject.pc4usr/lib/*/pkgconfig/avahi-gobject.pc
5usr/share/gir-*/*.gir
6

Subscribers

People subscribed via source and target branches

to all changes: