[Buildroot] [PATCH] Add libapr-package
Maxime Ripard
maxime.ripard at free-electrons.com
Tue Apr 17 13:25:13 UTC 2012
Hi,
Le 17/04/2012 15:16, Rico Bachmann a écrit :
> Signed-off-by: Rico Bachmann <bachmann at tofwerk.com>
> ---
> package/Config.in | 1 +
> package/apr/Config.in | 7 +++++++
> package/apr/apr.mk | 18 ++++++++++++++++++
> 3 files changed, 26 insertions(+), 0 deletions(-)
> create mode 100644 package/apr/Config.in
> create mode 100644 package/apr/apr.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 4c6d4d8..ceb5f6d 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -376,6 +376,7 @@ source "package/fftw/Config.in"
> source "package/libargtable2/Config.in"
> source "package/argp-standalone/Config.in"
> source "package/boost/Config.in"
> +source "package/libapr/Config.in"
You should change this to "package/apr/Config.in" now that you have
changed the location of this file. This won't even work.
> source "package/libatomic_ops/Config.in"
> source "package/libcap/Config.in"
> source "package/libcap-ng/Config.in"
> diff --git a/package/apr/Config.in b/package/apr/Config.in
> new file mode 100644
> index 0000000..d156909
> --- /dev/null
> +++ b/package/apr/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_LIBAPR
Same thing here, it should be called BR2_PACKAGE_APR.
> + bool "apr"
> + help
> + The mission of the Apache Portable Runtime (APR) project is to create
> + and maintain software libraries that provide a predictable and
> + consistent interface to underlying platform-specific implementations
Could you add an extra line here ?
> + http://apr.apache.org/
> diff --git a/package/apr/apr.mk b/package/apr/apr.mk
> new file mode 100644
> index 0000000..ae91eb1
> --- /dev/null
> +++ b/package/apr/apr.mk
> @@ -0,0 +1,18 @@
> +#############################################################
> +#
> +# apr
> +#
> +#############################################################
> +APR_VERSION = 1.4.6
> +APR_SITE = archive.apache.org/dist/apr
You should put http:// here. We use the URL's scheme to select the
appropriate downloading method. I guess it defaults to http if no scheme
is set, but hey, explicit is always better than implicit :)
> +APR_INSTALL_STAGING = YES
> +APR_CONF_ENV = \
> + ac_cv_file__dev_zero=yes \
> + ac_cv_func_setpgrp_void=yes \
> + apr_cv_process_shared_works=yes \
> + apr_cv_mutex_robust_shared=no \
> + apr_cv_tcp_nodelay_with_cork=yes \
> + ac_cv_sizeof_struct_iovec=8 \
> + apr_cv_mutex_recursive=yes
> +
> +$(eval $(call AUTOTARGETS))
Thanks,
Maxime
--
Maxime Ripard, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
More information about the buildroot
mailing list