[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 06/14] Rename SLIRP_OBJS to slirp-obj-y
From: |
Mark McLoughlin |
Subject: |
Re: [Qemu-devel] [PATCH 06/14] Rename SLIRP_OBJS to slirp-obj-y |
Date: |
Thu, 25 Jun 2009 07:59:25 +0100 |
On Thu, 2009-06-25 at 00:08 +0200, address@hidden wrote:
> diff --git a/Makefile b/Makefile
> index 802d276..d316388 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -188,10 +188,10 @@ endif
>
> ifdef CONFIG_SLIRP
> CPPFLAGS+=-I$(SRC_PATH)/slirp
> -SLIRP_OBJS=cksum.o if.o ip_icmp.o ip_input.o ip_output.o \
> -slirp.o mbuf.o misc.o sbuf.o socket.o tcp_input.o tcp_output.o \
> -tcp_subr.o tcp_timer.o udp.o bootp.o debug.o tftp.o
> -OBJS+=$(addprefix slirp/, $(SLIRP_OBJS))
> +slirp-obj-y = cksum.o if.o ip_icmp.o ip_input.o ip_output.o
> +slirp-obj-y += slirp.o mbuf.o misc.o sbuf.o socket.o tcp_input.o tcp_output.o
> +slirp-obj-y += tcp_subr.o tcp_timer.o udp.o bootp.o debug.o tftp.o
> +OBJS+=$(addprefix slirp/, $(slirp-obj-y))
I like the whole series, but I think I'd drop this change and the
XEN_OBJS change.
We won't be individually configuring out any of SLIRP_OBJS, so an end
result like:
obj-$(CONFIG_SLIRP) += $(addprefix slirp/, $(SLIRP_OBJS))
seems nicer to me than:
obj-$(CONFIG_SLIRP) += $(addprefix slirp/, $(slirp-obj-y))
Cheers,
Mark.
- [Qemu-devel] [PATCH 00/14] Refactor Makefile object selection, quintela, 2009/06/24
- [Qemu-devel] [PATCH 01/14] Rename OBJS to obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 02/14] Rename LIBOBJS to libobj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 03/14] Rename OBJS to obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 04/14] Rename BLOCK_OBJS to block-obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 05/14] Rename AUDIO_OBJS to audio-obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 06/14] Rename SLIRP_OBJS to slirp-obj-y, quintela, 2009/06/24
- Re: [Qemu-devel] [PATCH 06/14] Rename SLIRP_OBJS to slirp-obj-y,
Mark McLoughlin <=
- [Qemu-devel] [PATCH 07/14] Rename XEN_OBJS to xen-obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 08/14] Rename USER_OBJS to user-obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 09/14] Rename OBJS to obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 11/14] Substitute ifdef CONFIG_FOO by obj-, quintela, 2009/06/24
- [Qemu-devel] [PATCH 10/14] move CONFIG_* values from yes to y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 12/14] Substitute ifdef CONFIG_FOO by obj-, quintela, 2009/06/24
- [Qemu-devel] [PATCH 13/14] Rename SOUND_HW to sound-obj-y, quintela, 2009/06/24
- [Qemu-devel] [PATCH 14/14] Substitute ifdef CONFIG_FOO by sound-obj-, quintela, 2009/06/24