[libvirt] RFC: spec file cleanup ideas

[Daniel P. Berrange]

On Fri, Apr 15, 2016 at 02:40:29PM +0300, Roman Bogorodskiy wrote:
>   Cole Robinson wrote:
> 
> > Thanks for the comments
> > 
> > On 04/15/2016 06:14 AM, Daniel P. Berrange wrote:
> > > 
> > >> * Drop with_* conditionals that are only used by old Fedora or RHEL5. I didn't
> > >> audit all of them but the obvious ones are straight forward like:
> > >> with_systemd_macros, with_polkit, with_capng, with_netcf, with_yajl,
> > >> with_capng, with_avahi, with_hal/all HAL support
> > > 
> > > For that matter I think we can drop HAL *code* entirely since it was
> > > EOL after RHEL5, replaced by the udev driver.
> > 
> > Maybe BSD still uses it? CCing Roman
> > 
> > Also Maxim fixed a WITH_HAL build issue in November, not sure if they are
> > using it. CCed
> 
> HAL nodedev driver builds fine on FreeBSD and basic stuff like
> nodedev-list or nodedev-dumpxml works, but that's the only things that I
> test for nodedev.
> 
> Also, I don't actually know what are the usage scenarios for the nodedev
> APIs, I assume the main function is to use it to manage devices
> passthrough to guests. For that case I doubt that important things like
> device detaching work on FreeBSD, but probably I'm wrong because I didn't
> look closer at this. Also, I didn't test PCI passthrough on FreeBSD at
> all. 
> 
> Anyway, once somebody gets to implement passthrough related things for
> libvirt on FreeBSD, probably it'd be good to base these things on the
> existing HAL driver.
> 
> On the other hand, I'm not aware if somebody's going to implement it
> soon and it's not on the top of my todo list either, so it would not be
> a huge lost if it gets removed (not hard to revive it when needed).

Ok, I didn't realize HAL had been ported to FreeBSD. I assume the udev
driver isn't going to be much use for FreeBSD, so we should certainly
*not* remove HAL for as long as its used on FreeBSD as that would be
a feature regression.

Regards,
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 :|