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

Re: [libvirt] [PATCH 4/4] build: install libvirt sysctl file correctly



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

Attachment: signature.asc
Description: OpenPGP digital signature


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