[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] add --confdir option to configure
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] add --confdir option to configure |
Date: |
Mon, 19 Mar 2012 12:31:40 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Mar 19, 2012 at 10:14:54AM -0500, Anthony Liguori wrote:
> On 03/19/2012 09:47 AM, Eduardo Habkost wrote:
> >This series change the Makefile to use $(confdir) instead of
> >$(sysconfdir)/qemu, and allows the full config path configurable instead of
> >forcing the use of $(sysconfdir)/qemu.
>
> What's the use case here? Is this to allow $(sysconfdir)/qemu-kvm?
Yes. On RHEL we usually package qemu-kvm only, but we try to avoid
conflicts in case other flavors of qemu be provided by third-parties
(read: EPEL).
>
> I'd rather we use a PACKAGE_NAME define to do that and have qemu-kvm
> change PACKAGE_NAME.
I tried to mimic --mandir, --datadir, --docdir, and all other options
that expect full paths instead of trying to building one itself.
A --package-name option could be provided to make it easier to override
all the defaults at the same time, but I don't see why not include an
option to define the full path for confdir, just like we allow for
datadir, docdir, and mandir.
--
Eduardo
- [Qemu-devel] [PATCH 0/3] add --confdir option to configure, Eduardo Habkost, 2012/03/19
- [Qemu-devel] [PATCH 2/3] qemu-options.hx: refer to confdir instead of sysconfdir on docs, Eduardo Habkost, 2012/03/19
- [Qemu-devel] [PATCH 3/3] configure: make full config dir path configurable, Eduardo Habkost, 2012/03/19
- [Qemu-devel] [PATCH 1/3] Makefile: use $(confdir) instead of hardcoding $(sysconfdir)/qemu, Eduardo Habkost, 2012/03/19
- Re: [Qemu-devel] [PATCH 0/3] add --confdir option to configure, Anthony Liguori, 2012/03/19
- Re: [Qemu-devel] [PATCH 0/3] add --confdir option to configure, Anthony Liguori, 2012/03/19
- Re: [Qemu-devel] [PATCH 0/3] add --confdir option to configure, Eduardo Habkost, 2012/03/19