[libvirt] [PATCH 4/4] build: install libvirt sysctl file correctly
Eric Blake
eblake at redhat.com
Mon Jan 7 16:11:10 UTC 2013
On 01/05/2013 11:27 AM, Jim Fehlig wrote:
> Eric Blake wrote:
>> https://bugzilla.redhat.com/show_bug.cgi?id=887017 reports that
>> even though libvirt attempts to set fs.aio-max-nr via sysctl,
>> the file was installed with the wrong name and gets ignored by
>> sysctl. Furthermore, 'man systcl.d' recommends that packages
>> install into hard-coded /usr/lib/sysctl.d (even when libdir is
>> /usr/lib64), so that sysadmins can use /etc/sysctl.d for overrides.
>>
>> * daemon/Makefile.am (install-sysctl, uninstall-sysctl): Use
>> correct location.
>> * libvirt.spec.in (network_files): Reflect this.
>> ---
>> daemon/Makefile.am | 10 ++++++----
>> libvirt.spec.in | 4 ++--
>> 2 files changed, 8 insertions(+), 6 deletions(-)
>>
>
> I didn't test this one in my rpm builds, but looks good to me.
>
> ACK.
Thanks for the reviews; I've gone ahead and pushed the series now
(trusting that my testing via autobuild.sh is sufficient for this one).
I am also in the process of backporting these to v0.10.2-maint, since
that is the version used by F18 where the bug report was first raised.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130107/ed9c4ef9/attachment-0001.sig>
More information about the libvir-list
mailing list