[Buildroot] [PATCH v4] bandwidthd: Version bump to fix autobuilder errors
Baruch Siach
baruch at tkos.co.il
Thu Aug 21 10:34:21 UTC 2014
Hi Nathaniel,
On Thu, Aug 21, 2014 at 06:24:55PM +0800, Nathaniel Roach wrote:
> Thanks to Romain Naour, we've found that certain tests
> in configure fail if their dependencies aren't tested for
> beforehand.
>
> zlib has been added as a dependency just to make sure it
> gets built.
>
> In upstream the changes to fix this issue are:
> - Test for libusb-1.0 purely to ensure that variables are set
> (it's only needed if libpcap was built with it).
> - Test for libz and continue/fail because it's actually needed by libpng
> and it's probably more meaningful to fail on libz rather than
> libpng if libz is the issue.
>
> Also, the X.Org tests have been removed from the configure script
> and therefore we don't need --without-x to be defined anymore.
>
> Additional upstream changes:
> - Handle the postgres/sqlite flags better (should not effect BR).
> - Other general configure.ac cleanups.
>
> Fixes:
> autobuild.buildroot.net/results/37a/37a94e0f2d3faa0aec9c488e987cfff706fc9f3b/build-end.log
>
> Signed-off-by: Nathaniel Roach <nroach44 at gmail.com>
> Acked-by: Romain Naour <romain.naour at openwide.fr>
> ---
> Changes v1 to v2
> - Fixed spelling of Romain's name
> - Added link to the build failure
>
> Changes v2 to v3
> - Added back the SOff and Ack lines
>
> Changes v3 to v4
> - Removed the commend on Config.in regarding zlib
> - Corrected a libpng instance in commit message to zlib
> ---
> package/bandwidthd/Config.in | 1 +
> package/bandwidthd/bandwidthd.mk | 6 ++----
> 2 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/package/bandwidthd/Config.in b/package/bandwidthd/Config.in
> index c1f0ee1..4da685c 100644
> --- a/package/bandwidthd/Config.in
> +++ b/package/bandwidthd/Config.in
> @@ -1,6 +1,7 @@
> config BR2_PACKAGE_BANDWIDTHD
> bool "bandwidthd"
> depends on BR2_USE_MMU # fork()
> + select BR2_PACKAGE_ZLIB
Is bandwidthd using libz directly? If not, libpng already selects
BR2_PACKAGE_ZLIB so this may not actually be needed?
> select BR2_PACKAGE_LIBPNG
> select BR2_PACKAGE_LIBPCAP
> select BR2_PACKAGE_GD
> diff --git a/package/bandwidthd/bandwidthd.mk b/package/bandwidthd/bandwidthd.mk
> index 3e11d6a..59b6143 100644
> --- a/package/bandwidthd/bandwidthd.mk
> +++ b/package/bandwidthd/bandwidthd.mk
> @@ -4,19 +4,17 @@
> #
> ################################################################################
>
> -BANDWIDTHD_VERSION = v2.0.1-auto-r08
> +BANDWIDTHD_VERSION = v2.0.1-auto-r10
> BANDWIDTHD_SITE = $(call github,nroach44,bandwidthd,$(BANDWIDTHD_VERSION))
>
> # Specified as "any version of the GPL that is current as of your
> # download" by upstream.
> BANDWIDTHD_LICENSE = GPL
>
> -BANDWIDTHD_DEPENDENCIES = gd libpng libpcap
> +BANDWIDTHD_DEPENDENCIES = gd libpng libpcap zlib
Same here.
baruch
>
> BANDWIDTHD_AUTORECONF = YES
>
> -BANDWIDTHD_CONF_OPT += --without-x
> -
> ifeq ($(BR2_PACKAGE_BANDWIDTHD_POSTGRESQL),y)
> BANDWIDTHD_DEPENDENCIES += postgresql
> BANDWIDTHD_CONF_OPT += --with-postgresql-logging=true
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
More information about the buildroot
mailing list