On Fri, Nov 11, 2011 at 05:19:33PM +0100, Paolo Bonzini wrote: > On 11/11/2011 03:32 PM, Daniel P. Berrange wrote: > >+%makeinstall SYSTEMD_UNIT_DIR=%{?buildroot:%{buildroot}}%{_unitdir} > > Is this correct since DESTDIR is prepended to it? Shouldn't it just > be just %{_unitdir}? Otherwise looks good; it would be nicer to > make it a configure option, but that's not too important. Yes,the %buildroot is not required 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@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list