[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/14] configure: rename $datadir to $qemu_datad
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 01/14] configure: rename $datadir to $qemu_datadir |
Date: |
Wed, 18 Apr 2012 17:09:26 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Apr 18, 2012 at 01:57:51PM -0600, Eric Blake wrote:
> On 04/18/2012 01:55 PM, Eduardo Habkost wrote:
> > The variable is used for the qemu-specific directory and has a different
> > meaning of the autoconf "datadir" variable (that's used for the
> > $prefix/share directory, not for $prefix/share/PACKAGE).
>
> Automake uses the name $pkgdatadir for $prefix/share/PACKAGE; is it
> worth respinning this patch to s/qemu_datadir/pkgdatadir/ so that our
> files have at least a bit more similarity to autotool'd packages?
Personally, I think $qemu_datadir more clear than $pkgdatadir, as the
name makes it clear that it's a qemu-specific variable. (But maybe
people with more experience with Autotools would say it's the oppposite.
I can respin the series if others think it's worth it.)
>
> > -datadir="\${prefix}/share/qemu"
> > +qemu_datadir="\${prefix}/share/qemu"
>
--
Eduardo
- [Qemu-devel] [PATCH v5 00/14] configure: --with-confsuffix option, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 01/14] configure: rename $datadir to $qemu_datadir, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 08/14] config-host.mak: rename docdir to qemu_docdir, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 06/14] config-host.mak: rename confdir to qemu_confdir, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 03/14] configure: rename $confdir to $qemu_confdir, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 12/14] create_config: simplify prefix=* block, remove CONFIG_QEMU_PREFIX, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 11/14] create_config: remove *dir block, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 04/14] create_config: separate section for qemu_*dir variables (v2), Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 13/14] configure: change meaning of --datadir to Autoconf convention, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 10/14] config-host.mak: reorder variables a bit, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 07/14] Makefile: use $(qemu_confdir) instead of $(sysconfdir)/qemu, Eduardo Habkost, 2012/04/18
- [Qemu-devel] [PATCH 05/14] config-host.mak: rename datadir to qemu_datadir, Eduardo Habkost, 2012/04/18