Re: Patch replaces scriptlets with new systemd macros

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/25/2012 04:10 AM, Václav Pavlín wrote:
> Hi,

Hello,

[please don't top-post on technical lists]

> 
> sorry, I didn't realize I have to modify upstream version.

No problem; we can probably figure out how to modify the upstream
version based on your patch to downstream, if it comes to that, and if
we decide that the modifications even make sense.  Right now, our
biggest concern is that using the new scriptlets will break the use of
the same spec file when building for F17, so that concern needs to be
addressed before we can accept your patch.

> 
> I added Requires stanza to libvirtd service file for cgconfig.service,
> so you should not need to call enable in spec file for it.

That sounds independently useful, and probably worth applying even while
debating about the scriptlets.

> 
> I added %with_systemd_macros so it should now work in F17 with old
> scriptlets and in F18+/RHEL7+ with systemd macros

Ah, then maybe you did answer the big question.  Except that I don't see
the updated patch - did you forget to attach it?

> 
> I missed libvirt-guests.service because there is no systemctl call for
> it. So I only added systemd macros calls.
> 
> Release and Changelog are untouched so you can modify it as you need.
> 

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]