[PATCH pkg-libvirt/libguestfs] Split off ZFS support from libguestfs0.
Hilko, This is an experimental patch showing how to split libguestfs into subpackages, so the main libguestfs0 doesn't depend on everything. Splitting packages only works for packages which are in the appliance (ie. in appliance/packagelist or supermin.d/packages). In this patch, only zfs-fuse [which previously caused trouble for users: https://bugs.launchpad.net/bugs/1053911] has been split off. However the same principle can be used for other large dependencies. The links below are how we split the package in Fedora. This may not be a suitable split for Debian, and it's probably not even the best split for Fedora, but it's here as a reference. [It'd be good to do some analysis on the largest dependent packages and a knapsack-type best division analysis.] http://pkgs.fedoraproject.org/cgit/libguestfs.git/tree/libguestfs.spec?id=c2... http://pkgs.fedoraproject.org/cgit/libguestfs.git/tree/libguestfs.spec?id=c2... This patch probably won't apply cleanly to your latest experimental branch: in particular you moved /usr/lib/guestfs to a multiarch subdirectory, but this patch still uses /usr/lib/guestfs. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into KVM guests. http://libguestfs.org/virt-v2v
On Sat, Mar 15, 2014 at 10:29:08PM +0000, Richard W.M. Jones wrote:
Hilko,
This is an experimental patch showing how to split libguestfs into subpackages, so the main libguestfs0 doesn't depend on everything.
Splitting packages only works for packages which are in the appliance (ie. in appliance/packagelist or supermin.d/packages).
In this patch, only zfs-fuse [which previously caused trouble for users: https://bugs.launchpad.net/bugs/1053911] has been split off.
However the same principle can be used for other large dependencies. The links below are how we split the package in Fedora. This may not be a suitable split for Debian, and it's probably not even the best split for Fedora, but it's here as a reference. [It'd be good to do some analysis on the largest dependent packages and a knapsack-type best division analysis.]
http://pkgs.fedoraproject.org/cgit/libguestfs.git/tree/libguestfs.spec?id=c2... http://pkgs.fedoraproject.org/cgit/libguestfs.git/tree/libguestfs.spec?id=c2...
This patch probably won't apply cleanly to your latest experimental branch: in particular you moved /usr/lib/guestfs to a multiarch subdirectory, but this patch still uses /usr/lib/guestfs.
Hi Hilko, There's a thread on openstack-operators: http://www.gossamer-threads.com/lists/openstack/operators/37556 about the zfs-fuse dependency being a problem. Is it possible you can include a patch to split off libguestfs-zfs as a subpackage? Since I wrote the above email, there are two further developments. Firstly I examined the size of dependencies in Fedora to see which ones were worth splitting off from the core library (although in the ZFS case it's not really the size of the dependency, but the fact that there are two conflicting implementations in Debian): http://rwmj.wordpress.com/2014/03/20/analysis-of-the-size-of-libguestfs-depe... Secondly I changed the subpackages in Fedora slightly. See: http://pkgs.fedoraproject.org/cgit/libguestfs.git/tree/libguestfs.spec?id=73... http://pkgs.fedoraproject.org/cgit/libguestfs.git/tree/libguestfs.spec?id=73... Rich.
From f0bc374caed67294b885869c1bc1ea22189fac39 Mon Sep 17 00:00:00 2001 From: "Richard W.M. Jones" <rjones@redhat.com> Date: Sat, 15 Mar 2014 16:39:18 -0400 Subject: [PATCH] Split off ZFS support from libguestfs0.
--- debian/changelog | 6 ++++++ debian/control | 14 +++++++++++++- debian/libguestfs-zfs-fuse.install | 1 + debian/libguestfs0.dirs | 2 ++ debian/libguestfs0.install | 8 +++++++- debian/rules | 12 ++++++++++++ 6 files changed, 41 insertions(+), 2 deletions(-) create mode 100644 debian/libguestfs-zfs-fuse.install
diff --git a/debian/changelog b/debian/changelog index c0b8769..2cddcf2 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +libguestfs (1:1.25.43-3) experimental; urgency=medium + + * Split off ZFS support from libguestfs0. + + -- Richard W.M. Jones <rjones@redhat.com> Sat, 15 Mar 2014 16:38:45 -0400 + libguestfs (1:1.25.43-2) experimental; urgency=medium
* Remove update-guestfs-appliance. diff --git a/debian/control b/debian/control index cfc8eec..e6579e2 100644 --- a/debian/control +++ b/debian/control @@ -114,7 +114,6 @@ Depends: ${shlibs:Depends}, ${misc:Depends}, udev, vim-tiny, xz-utils, - zfs-fuse, binutils, cpio, dosfstools, @@ -152,6 +151,19 @@ Description: guest disk image management system - shared library what file system is in each LV, etc.); to run commands in the context of the guest; and to upload and download files and directories.
+Package: libguestfs-zfs-fuse +Section: libs +Architecture: linux-any +Multi-Arch: same +Pre-Depends: ${misc:Pre-Depends} +Depends: libguestfs0 (= ${binary:Version}), ${shlibs:Depends}, ${misc:Depends}, + zfs-fuse +Description: guest disk image management system - shared library + The libguestfs library allows accessing and modifying guest disk + images. + . + Install this package to get ZFS support. + Package: libguestfs-tools Section: admin Architecture: linux-any diff --git a/debian/libguestfs-zfs-fuse.install b/debian/libguestfs-zfs-fuse.install new file mode 100644 index 0000000..11a6249 --- /dev/null +++ b/debian/libguestfs-zfs-fuse.install @@ -0,0 +1 @@ +usr/lib/guestfs/supermin.d/zz-packages-zfs-fuse diff --git a/debian/libguestfs0.dirs b/debian/libguestfs0.dirs index fcf16f2..334dbd2 100644 --- a/debian/libguestfs0.dirs +++ b/debian/libguestfs0.dirs @@ -1 +1,3 @@ usr/share/man/man1 +usr/lib/guestfs +usr/lib/guestfs/supermin.d diff --git a/debian/libguestfs0.install b/debian/libguestfs0.install index 0045dd8..9d8de70 100644 --- a/debian/libguestfs0.install +++ b/debian/libguestfs0.install @@ -1,5 +1,11 @@ usr/lib/*-*/libguestfs.so.* -usr/lib/guestfs/* +usr/lib/guestfs/supermin.d/base.tar.gz +usr/lib/guestfs/supermin.d/daemon.tar.gz +usr/lib/guestfs/supermin.d/excludefiles +usr/lib/guestfs/supermin.d/hostfiles +usr/lib/guestfs/supermin.d/init.tar.gz +usr/lib/guestfs/supermin.d/packages +usr/lib/guestfs/supermin.d/udev-rules.tar.gz usr/share/man/man1/guestfs-* usr/share/man/*/man1/guestfs-* usr/share/locale/*/LC_MESSAGES/libguestfs.mo diff --git a/debian/rules b/debian/rules index 211c26e..9c6f232 100755 --- a/debian/rules +++ b/debian/rules @@ -108,6 +108,18 @@ override_dh_auto_install: done
override_dh_install: +# Split the supermin appliance packages list into subpackages, so the +# main libguestfs0 can avoid depending on everything. + cd $(CURDIR)/debian/tmp/usr/lib/guestfs/supermin.d && \ + for f in zfs-fuse; do \ + if grep -Esq "^$$f\$$" packages; then \ + mv packages packages~; \ + grep -Ev "^$$f\$$" < packages~ > packages; \ + rm packages~; \ + echo $$f > zz-packages-$$f; \ + fi; \ + done + erlang-depends -perlang-guestfs
dh_install -X.la -X.so.owner -Xbindtests -X/usr/lib/go/ \ -- 1.9.0
_______________________________________________ Libguestfs mailing list Libguestfs@redhat.com https://www.redhat.com/mailman/listinfo/libguestfs
-- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-df lists disk usage of guests without needing to install any software inside the virtual machine. Supports Linux and Windows. http://people.redhat.com/~rjones/virt-df/
participants (2)
-
Hilko Bengen -
Richard W.M. Jones