[Buildroot] [PATCH] Add libapr-package
Rico Bachmann
bachmann at tofwerk.com
Tue Apr 17 12:18:13 UTC 2012
Am 17.04.2012, 13:36 Uhr, schrieb Maxime Ripard
<maxime.ripard at free-electrons.com>:
> Hi,
>
> Thanks for your patch!
>
> Le 17/04/2012 12:59, Rico Bachmann a écrit :
>> Signed-off-by: Rico Bachmann
>> <bachmann at tofwerk.com>
>> ---
>> package/Config.in | 1 +
>> package/libapr/Config.in | 7 +++++++
>> package/libapr/libapr.mk | 12 ++++++++++++
>> 3 files changed, 20 insertions(+), 0 deletions(-)
>> create mode 100644 package/libapr/Config.in
>> create mode 100644 package/libapr/libapr.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"
>> source "package/libatomic_ops/Config.in"
>> source "package/libcap/Config.in"
>> source "package/libcap-ng/Config.in"
>> diff --git a/package/libapr/Config.in b/package/libapr/Config.in
>> new file mode 100644
>> index 0000000..51dea91
>> --- /dev/null
>> +++ b/package/libapr/Config.in
>> @@ -0,0 +1,7 @@
>> +config BR2_PACKAGE_LIBAPR
>> +
>> + bool "libapr"
>> + 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 wrap this line to 80 characters ?
I will wrap this line in my next patch
>
>> + http://apr.apache.org/
>> diff --git a/package/libapr/libapr.mk b/package/libapr/libapr.mk
>> new file mode 100644
>> index 0000000..57da8ef
>> --- /dev/null
>> +++ b/package/libapr/libapr.mk
>> @@ -0,0 +1,12 @@
>> +#############################################################
>> +#
>> +# libapr
>> +#
>> +#############################################################
>> +LIBAPR_VERSION = 1.4.6
>> +LIBAPR_SOURCE = apr-$(LIBAPR_VERSION).tar.gz
>> +LIBAPR_SITE = http://mirror.switch.ch/mirror/apache/dist/apr
>> +LIBAPR_INSTALL_STAGING = YES
>> +LIBAPR_CONF_OPT = 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 --enable-shared
>
> All the ac_cv_* stuff should go into LIBAPR_CONF_ENV variable.
> You should wrap the line to 80 characters also.
> --enabled-shared is automatically set by buildroot if it needs to when
> the package is built, so you don't need to set it here.
Ok i will add the apr_cv_* stuff to LIBAPR_CONF_ENV instead of
LIBAPR_CONF_OPT
and delete the --enable-shared argument
>
> Otherwise, it looks good to me.
>
> Thanks,
> Maxime
>
--
Erstellt mit Operas revolutionärem E-Mail-Modul: http://www.opera.com/mail/
More information about the buildroot
mailing list