Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On 12.02.2014 21:15, Eric Blake wrote: On 02/05/2014 02:51 AM, Daniel P. Berrange wrote: On Tue, Feb 04, 2014 at 03:15:56PM -0700, Eric Blake wrote: On Fedora 20, with wireshark-devel installed, 'make rpm' failed due to installed but unpackaged files related to wireshark. As F20 is already released without wireshark, I chose to add a new sub-package that is enabled only for F21 and later. Furthermore, all existing wireshark plugins belong to the wireshark package, so I got to invent behavior of how the first third-part wireshark module will behave. * libvirt.spec.in (with_wireshark): Add new conditional. * configure.ac (ws-plugindir): Improve wording. Signed-off-by: Eric Blake ebl...@redhat.com --- I was tempted to push this as a build-breaker fix for 'make rpm', but rpms are close enough to black magic that I decided a review is safer, after all. Tested with both F20 (not built) and F21 (new subpackage built just fine), using normal build of all subpackages and also a build with '%client_only 1' in ~/.rpmmacros to ensure that it indeed works in a client-only setup. configure.ac| 4 ++-- libvirt.spec.in | 34 ++ 2 files changed, 36 insertions(+), 2 deletions(-) ACK Hmm, should the generic 'libvirt' metapackage also Require: the libvirt-wireshark subpackage, so that we keep the status quo that installing just libvirt pulls in all subpackages? Well, I think we should intentionally break that rule here. Otherwise: yum install libvirt drags in wireshark as a dependency. There wouldn't be anything bad about it, except the wireshark plugin is expected to be used by developers, not an ordinary users. If I were an ordinary user I'd be very curious why I need to install wireshark to run libvirt. Michal -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On Thu, Feb 13, 2014 at 09:38:53AM +0100, Michal Privoznik wrote: On 12.02.2014 21:15, Eric Blake wrote: On 02/05/2014 02:51 AM, Daniel P. Berrange wrote: On Tue, Feb 04, 2014 at 03:15:56PM -0700, Eric Blake wrote: On Fedora 20, with wireshark-devel installed, 'make rpm' failed due to installed but unpackaged files related to wireshark. As F20 is already released without wireshark, I chose to add a new sub-package that is enabled only for F21 and later. Furthermore, all existing wireshark plugins belong to the wireshark package, so I got to invent behavior of how the first third-part wireshark module will behave. * libvirt.spec.in (with_wireshark): Add new conditional. * configure.ac (ws-plugindir): Improve wording. Signed-off-by: Eric Blake ebl...@redhat.com --- I was tempted to push this as a build-breaker fix for 'make rpm', but rpms are close enough to black magic that I decided a review is safer, after all. Tested with both F20 (not built) and F21 (new subpackage built just fine), using normal build of all subpackages and also a build with '%client_only 1' in ~/.rpmmacros to ensure that it indeed works in a client-only setup. configure.ac| 4 ++-- libvirt.spec.in | 34 ++ 2 files changed, 36 insertions(+), 2 deletions(-) ACK Hmm, should the generic 'libvirt' metapackage also Require: the libvirt-wireshark subpackage, so that we keep the status quo that installing just libvirt pulls in all subpackages? Well, I think we should intentionally break that rule here. Otherwise: yum install libvirt drags in wireshark as a dependency. There wouldn't be anything bad about it, except the wireshark plugin is expected to be used by developers, not an ordinary users. If I were an ordinary user I'd be very curious why I need to install wireshark to run libvirt. I agree, we should *not* have 'libvirt' depend on the wireshark plugin. We should treat 'libvirt' as pulling in everything that it would have done in the past before we split the RPMs. So by that rationale new functionality is out of scope. Daniel -- |: http://berrange.com -o-http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On 02/13/2014 03:52 AM, Daniel P. Berrange wrote: yum install libvirt drags in wireshark as a dependency. There wouldn't be anything bad about it, except the wireshark plugin is expected to be used by developers, not an ordinary users. If I were an ordinary user I'd be very curious why I need to install wireshark to run libvirt. I agree, we should *not* have 'libvirt' depend on the wireshark plugin. We should treat 'libvirt' as pulling in everything that it would have done in the past before we split the RPMs. So by that rationale new functionality is out of scope. Then I'll revert the patch that I already pushed (sorry for not waiting for this conversation to come to a conclusion). -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On 02/13/2014 06:30 AM, Eric Blake wrote: On 02/13/2014 03:52 AM, Daniel P. Berrange wrote: yum install libvirt drags in wireshark as a dependency. There wouldn't be anything bad about it, except the wireshark plugin is expected to be used by developers, not an ordinary users. If I were an ordinary user I'd be very curious why I need to install wireshark to run libvirt. I agree, we should *not* have 'libvirt' depend on the wireshark plugin. We should treat 'libvirt' as pulling in everything that it would have done in the past before we split the RPMs. So by that rationale new functionality is out of scope. Then I'll revert the patch that I already pushed (sorry for not waiting for this conversation to come to a conclusion). A further argument in favor of reverting: 'libvirt' does not depend on 'libvirt-devel', so it already does not pull in EVERY subpackage, just subpackages related to FULL features of libvirtd. -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On 02/05/2014 02:51 AM, Daniel P. Berrange wrote: On Tue, Feb 04, 2014 at 03:15:56PM -0700, Eric Blake wrote: On Fedora 20, with wireshark-devel installed, 'make rpm' failed due to installed but unpackaged files related to wireshark. As F20 is already released without wireshark, I chose to add a new sub-package that is enabled only for F21 and later. Furthermore, all existing wireshark plugins belong to the wireshark package, so I got to invent behavior of how the first third-part wireshark module will behave. * libvirt.spec.in (with_wireshark): Add new conditional. * configure.ac (ws-plugindir): Improve wording. Signed-off-by: Eric Blake ebl...@redhat.com --- I was tempted to push this as a build-breaker fix for 'make rpm', but rpms are close enough to black magic that I decided a review is safer, after all. Tested with both F20 (not built) and F21 (new subpackage built just fine), using normal build of all subpackages and also a build with '%client_only 1' in ~/.rpmmacros to ensure that it indeed works in a client-only setup. configure.ac| 4 ++-- libvirt.spec.in | 34 ++ 2 files changed, 36 insertions(+), 2 deletions(-) ACK Hmm, should the generic 'libvirt' metapackage also Require: the libvirt-wireshark subpackage, so that we keep the status quo that installing just libvirt pulls in all subpackages? -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On Tue, Feb 04, 2014 at 03:15:56PM -0700, Eric Blake wrote: On Fedora 20, with wireshark-devel installed, 'make rpm' failed due to installed but unpackaged files related to wireshark. As F20 is already released without wireshark, I chose to add a new sub-package that is enabled only for F21 and later. Furthermore, all existing wireshark plugins belong to the wireshark package, so I got to invent behavior of how the first third-part wireshark module will behave. * libvirt.spec.in (with_wireshark): Add new conditional. * configure.ac (ws-plugindir): Improve wording. Signed-off-by: Eric Blake ebl...@redhat.com --- I was tempted to push this as a build-breaker fix for 'make rpm', but rpms are close enough to black magic that I decided a review is safer, after all. Tested with both F20 (not built) and F21 (new subpackage built just fine), using normal build of all subpackages and also a build with '%client_only 1' in ~/.rpmmacros to ensure that it indeed works in a client-only setup. configure.ac| 4 ++-- libvirt.spec.in | 34 ++ 2 files changed, 36 insertions(+), 2 deletions(-) ACK Daniel -- |: http://berrange.com -o-http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] rpm: create libvirt-wireshark sub-package
On 02/05/2014 02:51 AM, Daniel P. Berrange wrote: On Tue, Feb 04, 2014 at 03:15:56PM -0700, Eric Blake wrote: On Fedora 20, with wireshark-devel installed, 'make rpm' failed due to installed but unpackaged files related to wireshark. As F20 is already released without wireshark, I chose to add a new sub-package that is enabled only for F21 and later. Furthermore, all existing wireshark plugins belong to the wireshark package, so I got to invent behavior of how the first third-part wireshark module will behave. * libvirt.spec.in (with_wireshark): Add new conditional. * configure.ac (ws-plugindir): Improve wording. Signed-off-by: Eric Blake ebl...@redhat.com --- I was tempted to push this as a build-breaker fix for 'make rpm', but rpms are close enough to black magic that I decided a review is safer, after all. Tested with both F20 (not built) and F21 (new subpackage built just fine), using normal build of all subpackages and also a build with '%client_only 1' in ~/.rpmmacros to ensure that it indeed works in a client-only setup. configure.ac| 4 ++-- libvirt.spec.in | 34 ++ 2 files changed, 36 insertions(+), 2 deletions(-) ACK Thanks; pushed. -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list