[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