[Buildroot] [PATCH] rsync: use system zlib and popt, also do not lose debug options
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Oct 16 12:26:06 UTC 2013
Dear Denis Mingulov,
On Wed, 16 Oct 2013 14:40:31 +0300, Denis Mingulov wrote:
> rsync by default compiles statically its own zlib and popt
> libraries, it is better to use system wide libraries instead -
> both for disk and memory consumption. Change rsync package
> accordingly.
>
> Also previously a debug configure option has been lost, as
> overwritten by '--with-included-popt' option - fixed.
>
> For example, on ARMv5 rsync binary size is reduced by about 25%.
>
> Signed-off-by: Denis Mingulov <denis at mingulov.com>
Thanks, looks good, but I have one suggestion.
> RSYNC_CONF_OPT = $(if $(BR2_ENABLE_DEBUG),--enable-debug,--disable-debug)
> -RSYNC_CONF_OPT = --with-included-popt
> +RSYNC_CONF_OPT += --with-included-zlib=no
> +RSYNC_CONF_OPT += --with-included-popt=no
Make this:
RSYNC_CONF_OPT = \
$(if $(BR2_ENABLE_DEBUG),--enable-debug,--disable-debug) \
--with-included-zlib=no \
--with-included-popt=no
while it makes sense to append to <pkg>_CONF_OPT using += when such
appends are done under conditions, for the general configuration
options, just one assignment is clearer and that's what we use more or
less everywhere in Buildroot.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list