All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Add new package apr-util
@ 2012-04-24 10:11 Rico Bachmann
  2012-04-24 10:24 ` Thomas Petazzoni
  0 siblings, 1 reply; 8+ messages in thread
From: Rico Bachmann @ 2012-04-24 10:11 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
---
v1: Add package apr-util
    Add patch to remove check for apr
    because check for apr search for ./apr and not for defined folder
---
 package/Config.in                                  |    1 +
 package/apr-util/Config.in                         |    7 +++++++
 .../apr-util/apr-util-1.4.1-remove-checkapr.patch  |   12 ++++++++++++
 package/apr-util/apr-util.mk                       |   14 ++++++++++++++
 4 files changed, 34 insertions(+), 0 deletions(-)
 create mode 100644 package/apr-util/Config.in
 create mode 100644 package/apr-util/apr-util-1.4.1-remove-checkapr.patch
 create mode 100644 package/apr-util/apr-util.mk

diff --git a/package/Config.in b/package/Config.in
index 5ae1c81..805f56a 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -375,6 +375,7 @@ endmenu
 
 menu "Other"
 source "package/apr/Config.in"
+source "package/apr-util/Config.in"
 source "package/fftw/Config.in"
 source "package/libargtable2/Config.in"
 source "package/argp-standalone/Config.in"
diff --git a/package/apr-util/Config.in b/package/apr-util/Config.in
new file mode 100644
index 0000000..a4411ba
--- /dev/null
+++ b/package/apr-util/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_APR_UTIL
+	bool "apr-util"
+	select BR2_PACKAGE_APR
+	help
+	  The utility library for the apache runtime project
+
+	  http://apr.apache.org/
diff --git a/package/apr-util/apr-util-1.4.1-remove-checkapr.patch b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
new file mode 100644
index 0000000..46cc595
--- /dev/null
+++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
@@ -0,0 +1,12 @@
+diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
+--- apr-util-1.4.1/configure.in	2009-12-17 20:15:19.000000000 +0100
++++ apr-util-1.4.1-fix/configure.in	2012-04-24 10:10:32.000000000 +0200
+@@ -99,7 +99,7 @@ fi
+ dnl
+ dnl Find the APR includes directory and (possibly) the source (base) dir.
+ dnl
+-APU_FIND_APR
++dnl APU_FIND_APR
+ 
+ dnl
+ dnl even though we use apr_rules.mk for building apr-util, we need
diff --git a/package/apr-util/apr-util.mk b/package/apr-util/apr-util.mk
new file mode 100644
index 0000000..8c977fa
--- /dev/null
+++ b/package/apr-util/apr-util.mk
@@ -0,0 +1,14 @@
+#############################################################
+#
+# apr-util
+#
+#############################################################
+APR_UTIL_VERSION = 1.4.1
+APR_UTIL_SITE = http://archive.apache.org/dist/apr
+APR_UTIL_INSTALL_STAGING = YES
+APR_UTIL_DEPENDENCIES = apr
+APR_UTIL_CONF_OPT = \
+	--prefix=/usr \
+        --with-apr=$(BUILD_DIR)/apr-$(APR_VERSION)
+
+$(eval $(call AUTOTARGETS))
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
  2012-04-24 10:11 [Buildroot] [PATCH] Add new package apr-util Rico Bachmann
@ 2012-04-24 10:24 ` Thomas Petazzoni
  2012-04-24 10:51   ` Rico Bachmann
  0 siblings, 1 reply; 8+ messages in thread
From: Thomas Petazzoni @ 2012-04-24 10:24 UTC (permalink / raw)
  To: buildroot

Le Tue, 24 Apr 2012 12:11:42 +0200,
Rico Bachmann <bachmann@tofwerk.com> a ?crit :

> +diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
> +--- apr-util-1.4.1/configure.in	2009-12-17 20:15:19.000000000 +0100
> ++++ apr-util-1.4.1-fix/configure.in	2012-04-24 10:10:32.000000000 +0200

This patch lacks a header explaining what it is doing, why, and a
Signed-off-by line.

I'm also not convinced by this patch, can you detail why it is
necessary? I have the impression that by passing the right environment
variables and/or options to the configure script, you can convince it
to do the right thing, but I haven't looked into too much details about
how APU_FIND_APR is implemented.

Regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
  2012-04-24 10:24 ` Thomas Petazzoni
@ 2012-04-24 10:51   ` Rico Bachmann
  0 siblings, 0 replies; 8+ messages in thread
From: Rico Bachmann @ 2012-04-24 10:51 UTC (permalink / raw)
  To: buildroot

Am 24.04.2012, 12:24 Uhr, schrieb Thomas Petazzoni  
<thomas.petazzoni@free-electrons.com>:

> Le Tue, 24 Apr 2012 12:11:42 +0200,
> Rico Bachmann <bachmann@tofwerk.com> a ?crit :
>
>> +diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
>> +--- apr-util-1.4.1/configure.in	2009-12-17 20:15:19.000000000 +0100
>> ++++ apr-util-1.4.1-fix/configure.in	2012-04-24 10:10:32.000000000 +0200
>
> This patch lacks a header explaining what it is doing, why, and a
> Signed-off-by line.

I added the Signed-off-by line and added some comments, I will send it  
soon.

>
> I'm also not convinced by this patch, can you detail why it is
> necessary? I have the impression that by passing the right environment
> variables and/or options to the configure script, you can convince it
> to do the right thing, but I haven't looked into too much details about
> how APU_FIND_APR is implemented.

I asked for some help in #apr, they told me that the check of the source  
folder
only works on folders called apr. Because I don't want to rename the source
folder of the apr package (from apr-1.4.6 to apr), I disabled the check  
and the
build works nice for me

I also haven't looked to much in the implementation of APU_FIND_APR, so I  
decided
to do the way of not checking...

>
> Regards,
>
> Thomas

-- 
Erstellt mit Operas revolution?rem E-Mail-Modul: http://www.opera.com/mail/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
  2012-04-24 10:46     ` Yegor Yefremov
@ 2012-04-24 10:56       ` Rico Bachmann
  0 siblings, 0 replies; 8+ messages in thread
From: Rico Bachmann @ 2012-04-24 10:56 UTC (permalink / raw)
  To: buildroot

Am 24.04.2012, 12:46 Uhr, schrieb Yegor Yefremov  
<yegorslists@googlemail.com>:

> On Tue, Apr 24, 2012 at 12:42 PM, Yegor Yefremov
> <yegor_sub1@visionsystems.de> wrote:
>> Am 24.04.2012 12:28, schrieb Rico Bachmann:
>>> Am 24.04.2012, 12:22 Uhr, schrieb Rico Bachmann <bachmann@tofwerk.com>:
>>>
>>>> Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
>>>> ---
>>>> v1: Add package apr-util
>>>>     Add patch to remove check for apr
>>>>     because check for apr search for ./apr and not for defined folder
>>>>
>>>> change spaces to tab
>>>>
>>>> Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
>>>> ---
>>>> v2: change spaces in *.mk to tab
>>>> ---
>>>>  package/Config.in                                  |    1 +
>>>>  package/apr-util/Config.in                         |    7 +++++++
>>>>  .../apr-util/apr-util-1.4.1-remove-checkapr.patch  |   12  
>>>> ++++++++++++
>>>>  package/apr-util/apr-util.mk                       |   14  
>>>> ++++++++++++++
>>>>  4 files changed, 34 insertions(+), 0 deletions(-)
>>>>  create mode 100644 package/apr-util/Config.in
>>>>  create mode 100644  
>>>> package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>>>  create mode 100644 package/apr-util/apr-util.mk
>>>>
>>>> diff --git a/package/Config.in b/package/Config.in
>>>> index 5ae1c81..805f56a 100644
>>>> --- a/package/Config.in
>>>> +++ b/package/Config.in
>>>> @@ -375,6 +375,7 @@ endmenu
>>>> menu "Other"
>>>>  source "package/apr/Config.in"
>>>> +source "package/apr-util/Config.in"
>>>>  source "package/fftw/Config.in"
>>>>  source "package/libargtable2/Config.in"
>>>>  source "package/argp-standalone/Config.in"
>>>> diff --git a/package/apr-util/Config.in b/package/apr-util/Config.in
>>>> new file mode 100644
>>>> index 0000000..a4411ba
>>>> --- /dev/null
>>>> +++ b/package/apr-util/Config.in
>>>> @@ -0,0 +1,7 @@
>>>> +config BR2_PACKAGE_APR_UTIL
>>>> +    bool "apr-util"
>>>> +    select BR2_PACKAGE_APR
>>>> +    help
>>>> +      The utility library for the apache runtime project
>>>> +
>>>> +      http://apr.apache.org/
>>>> diff --git a/package/apr-util/apr-util-1.4.1-remove-checkapr.patch  
>>>> b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>>> new file mode 100644
>>>> index 0000000..46cc595
>>>> --- /dev/null
>>>> +++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>>> @@ -0,0 +1,12 @@
>>>> +diff -rupN apr-util-1.4.1/configure.in  
>>>> apr-util-1.4.1-fix/configure.in
>>>> +--- apr-util-1.4.1/configure.in    2009-12-17 20:15:19.000000000  
>>>> +0100
>>>> ++++ apr-util-1.4.1-fix/configure.in    2012-04-24 10:10:32.000000000  
>>>> +0200
>>>> +@@ -99,7 +99,7 @@ fi
>>>> + dnl
>>>> + dnl Find the APR includes directory and (possibly) the source  
>>>> (base) dir.
>>>> + dnl
>>>> +-APU_FIND_APR
>>>> ++dnl APU_FIND_APR
>>>> +
>>>> + dnl
>>>> + dnl even though we use apr_rules.mk for building apr-util, we need
>>>> diff --git a/package/apr-util/apr-util.mk  
>>>> b/package/apr-util/apr-util.mk
>>>> new file mode 100644
>>>> index 0000000..f5f4d80
>>>> --- /dev/null
>>>> +++ b/package/apr-util/apr-util.mk
>>>> @@ -0,0 +1,14 @@
>>>> +#############################################################
>>>> +#
>>>> +# apr-util
>>>> +#
>>>> +#############################################################
>>>> +APR_UTIL_VERSION = 1.4.1
>>>> +APR_UTIL_SITE = http://archive.apache.org/dist/apr
>>>> +APR_UTIL_INSTALL_STAGING = YES
>>>> +APR_UTIL_DEPENDENCIES = apr
>>>> +APR_UTIL_CONF_OPT = \
>>>> +    --prefix=/usr \
>>>> +    --with-apr=$(BUILD_DIR)/apr-$(APR_VERSION)
>>>> +
>>>> +$(eval $(call AUTOTARGETS))
>>>
>>> hmmm... I sent this mail with the --subject-prefix='PATCH v2' option.
>>> Can anyone tell me why I can't see this prefix in the Subject?
>>
>> I do it this way:  
>> http://libbits.wordpress.com/2011/12/27/managing-patch-series-with-git/
>
> Have you removed the previous patch? Execute git format-patch and
> examine the new patch file to check if everything is correct.

Yeah I think the patch looks like the right one, all I have to change was
the spaces before --with-apr to a tab.

But in short time there will be a patch v3, with a Signed-off-by line in
the package-patch

>
> Yegor
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot


-- 
Erstellt mit Operas revolution?rem E-Mail-Modul: http://www.opera.com/mail/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
  2012-04-24 10:42   ` Yegor Yefremov
@ 2012-04-24 10:46     ` Yegor Yefremov
  2012-04-24 10:56       ` Rico Bachmann
  0 siblings, 1 reply; 8+ messages in thread
From: Yegor Yefremov @ 2012-04-24 10:46 UTC (permalink / raw)
  To: buildroot

On Tue, Apr 24, 2012 at 12:42 PM, Yegor Yefremov
<yegor_sub1@visionsystems.de> wrote:
> Am 24.04.2012 12:28, schrieb Rico Bachmann:
>> Am 24.04.2012, 12:22 Uhr, schrieb Rico Bachmann <bachmann@tofwerk.com>:
>>
>>> Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
>>> ---
>>> v1: Add package apr-util
>>> ? ? Add patch to remove check for apr
>>> ? ? because check for apr search for ./apr and not for defined folder
>>>
>>> change spaces to tab
>>>
>>> Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
>>> ---
>>> v2: change spaces in *.mk to tab
>>> ---
>>> ?package/Config.in ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?| ? ?1 +
>>> ?package/apr-util/Config.in ? ? ? ? ? ? ? ? ? ? ? ? | ? ?7 +++++++
>>> ?.../apr-util/apr-util-1.4.1-remove-checkapr.patch ?| ? 12 ++++++++++++
>>> ?package/apr-util/apr-util.mk ? ? ? ? ? ? ? ? ? ? ? | ? 14 ++++++++++++++
>>> ?4 files changed, 34 insertions(+), 0 deletions(-)
>>> ?create mode 100644 package/apr-util/Config.in
>>> ?create mode 100644 package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>> ?create mode 100644 package/apr-util/apr-util.mk
>>>
>>> diff --git a/package/Config.in b/package/Config.in
>>> index 5ae1c81..805f56a 100644
>>> --- a/package/Config.in
>>> +++ b/package/Config.in
>>> @@ -375,6 +375,7 @@ endmenu
>>> menu "Other"
>>> ?source "package/apr/Config.in"
>>> +source "package/apr-util/Config.in"
>>> ?source "package/fftw/Config.in"
>>> ?source "package/libargtable2/Config.in"
>>> ?source "package/argp-standalone/Config.in"
>>> diff --git a/package/apr-util/Config.in b/package/apr-util/Config.in
>>> new file mode 100644
>>> index 0000000..a4411ba
>>> --- /dev/null
>>> +++ b/package/apr-util/Config.in
>>> @@ -0,0 +1,7 @@
>>> +config BR2_PACKAGE_APR_UTIL
>>> + ? ?bool "apr-util"
>>> + ? ?select BR2_PACKAGE_APR
>>> + ? ?help
>>> + ? ? ?The utility library for the apache runtime project
>>> +
>>> + ? ? ?http://apr.apache.org/
>>> diff --git a/package/apr-util/apr-util-1.4.1-remove-checkapr.patch b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>> new file mode 100644
>>> index 0000000..46cc595
>>> --- /dev/null
>>> +++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>> @@ -0,0 +1,12 @@
>>> +diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
>>> +--- apr-util-1.4.1/configure.in ? ?2009-12-17 20:15:19.000000000 +0100
>>> ++++ apr-util-1.4.1-fix/configure.in ? ?2012-04-24 10:10:32.000000000 +0200
>>> +@@ -99,7 +99,7 @@ fi
>>> + dnl
>>> + dnl Find the APR includes directory and (possibly) the source (base) dir.
>>> + dnl
>>> +-APU_FIND_APR
>>> ++dnl APU_FIND_APR
>>> +
>>> + dnl
>>> + dnl even though we use apr_rules.mk for building apr-util, we need
>>> diff --git a/package/apr-util/apr-util.mk b/package/apr-util/apr-util.mk
>>> new file mode 100644
>>> index 0000000..f5f4d80
>>> --- /dev/null
>>> +++ b/package/apr-util/apr-util.mk
>>> @@ -0,0 +1,14 @@
>>> +#############################################################
>>> +#
>>> +# apr-util
>>> +#
>>> +#############################################################
>>> +APR_UTIL_VERSION = 1.4.1
>>> +APR_UTIL_SITE = http://archive.apache.org/dist/apr
>>> +APR_UTIL_INSTALL_STAGING = YES
>>> +APR_UTIL_DEPENDENCIES = apr
>>> +APR_UTIL_CONF_OPT = \
>>> + ? ?--prefix=/usr \
>>> + ? ?--with-apr=$(BUILD_DIR)/apr-$(APR_VERSION)
>>> +
>>> +$(eval $(call AUTOTARGETS))
>>
>> hmmm... I sent this mail with the --subject-prefix='PATCH v2' option.
>> Can anyone tell me why I can't see this prefix in the Subject?
>
> I do it this way: http://libbits.wordpress.com/2011/12/27/managing-patch-series-with-git/

Have you removed the previous patch? Execute git format-patch and
examine the new patch file to check if everything is correct.

Yegor

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
  2012-04-24 10:28 ` Rico Bachmann
@ 2012-04-24 10:42   ` Yegor Yefremov
  2012-04-24 10:46     ` Yegor Yefremov
  0 siblings, 1 reply; 8+ messages in thread
From: Yegor Yefremov @ 2012-04-24 10:42 UTC (permalink / raw)
  To: buildroot

Am 24.04.2012 12:28, schrieb Rico Bachmann:
> Am 24.04.2012, 12:22 Uhr, schrieb Rico Bachmann <bachmann@tofwerk.com>:
> 
>> Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
>> ---
>> v1: Add package apr-util
>>     Add patch to remove check for apr
>>     because check for apr search for ./apr and not for defined folder
>>
>> change spaces to tab
>>
>> Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
>> ---
>> v2: change spaces in *.mk to tab
>> ---
>>  package/Config.in                                  |    1 +
>>  package/apr-util/Config.in                         |    7 +++++++
>>  .../apr-util/apr-util-1.4.1-remove-checkapr.patch  |   12 ++++++++++++
>>  package/apr-util/apr-util.mk                       |   14 ++++++++++++++
>>  4 files changed, 34 insertions(+), 0 deletions(-)
>>  create mode 100644 package/apr-util/Config.in
>>  create mode 100644 package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>>  create mode 100644 package/apr-util/apr-util.mk
>>
>> diff --git a/package/Config.in b/package/Config.in
>> index 5ae1c81..805f56a 100644
>> --- a/package/Config.in
>> +++ b/package/Config.in
>> @@ -375,6 +375,7 @@ endmenu
>> menu "Other"
>>  source "package/apr/Config.in"
>> +source "package/apr-util/Config.in"
>>  source "package/fftw/Config.in"
>>  source "package/libargtable2/Config.in"
>>  source "package/argp-standalone/Config.in"
>> diff --git a/package/apr-util/Config.in b/package/apr-util/Config.in
>> new file mode 100644
>> index 0000000..a4411ba
>> --- /dev/null
>> +++ b/package/apr-util/Config.in
>> @@ -0,0 +1,7 @@
>> +config BR2_PACKAGE_APR_UTIL
>> +    bool "apr-util"
>> +    select BR2_PACKAGE_APR
>> +    help
>> +      The utility library for the apache runtime project
>> +
>> +      http://apr.apache.org/
>> diff --git a/package/apr-util/apr-util-1.4.1-remove-checkapr.patch b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>> new file mode 100644
>> index 0000000..46cc595
>> --- /dev/null
>> +++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>> @@ -0,0 +1,12 @@
>> +diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
>> +--- apr-util-1.4.1/configure.in    2009-12-17 20:15:19.000000000 +0100
>> ++++ apr-util-1.4.1-fix/configure.in    2012-04-24 10:10:32.000000000 +0200
>> +@@ -99,7 +99,7 @@ fi
>> + dnl
>> + dnl Find the APR includes directory and (possibly) the source (base) dir.
>> + dnl
>> +-APU_FIND_APR
>> ++dnl APU_FIND_APR
>> +
>> + dnl
>> + dnl even though we use apr_rules.mk for building apr-util, we need
>> diff --git a/package/apr-util/apr-util.mk b/package/apr-util/apr-util.mk
>> new file mode 100644
>> index 0000000..f5f4d80
>> --- /dev/null
>> +++ b/package/apr-util/apr-util.mk
>> @@ -0,0 +1,14 @@
>> +#############################################################
>> +#
>> +# apr-util
>> +#
>> +#############################################################
>> +APR_UTIL_VERSION = 1.4.1
>> +APR_UTIL_SITE = http://archive.apache.org/dist/apr
>> +APR_UTIL_INSTALL_STAGING = YES
>> +APR_UTIL_DEPENDENCIES = apr
>> +APR_UTIL_CONF_OPT = \
>> +    --prefix=/usr \
>> +    --with-apr=$(BUILD_DIR)/apr-$(APR_VERSION)
>> +
>> +$(eval $(call AUTOTARGETS))
> 
> hmmm... I sent this mail with the --subject-prefix='PATCH v2' option.
> Can anyone tell me why I can't see this prefix in the Subject?

I do it this way: http://libbits.wordpress.com/2011/12/27/managing-patch-series-with-git/

Yegor

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
  2012-04-24 10:22 Rico Bachmann
@ 2012-04-24 10:28 ` Rico Bachmann
  2012-04-24 10:42   ` Yegor Yefremov
  0 siblings, 1 reply; 8+ messages in thread
From: Rico Bachmann @ 2012-04-24 10:28 UTC (permalink / raw)
  To: buildroot

Am 24.04.2012, 12:22 Uhr, schrieb Rico Bachmann  
<bachmann@tofwerk.com>:

> Signed-off-by: Rico Bachmann  
> <bachmann@tofwerk.com>
> ---
> v1: Add package apr-util
>     Add patch to remove check for apr
>     because check for apr search for ./apr and not for defined folder
>
> change spaces to tab
>
> Signed-off-by: Rico Bachmann  
> <bachmann@tofwerk.com>
> ---
> v2: change spaces in *.mk to tab
> ---
>  package/Config.in                                  |    1 +
>  package/apr-util/Config.in                         |    7 +++++++
>  .../apr-util/apr-util-1.4.1-remove-checkapr.patch  |   12 ++++++++++++
>  package/apr-util/apr-util.mk                       |   14 ++++++++++++++
>  4 files changed, 34 insertions(+), 0 deletions(-)
>  create mode 100644 package/apr-util/Config.in
>  create mode 100644 package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>  create mode 100644 package/apr-util/apr-util.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 5ae1c81..805f56a 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -375,6 +375,7 @@ endmenu
> menu "Other"
>  source "package/apr/Config.in"
> +source "package/apr-util/Config.in"
>  source "package/fftw/Config.in"
>  source "package/libargtable2/Config.in"
>  source "package/argp-standalone/Config.in"
> diff --git a/package/apr-util/Config.in b/package/apr-util/Config.in
> new file mode 100644
> index 0000000..a4411ba
> --- /dev/null
> +++ b/package/apr-util/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_APR_UTIL
> +	bool "apr-util"
> +	select BR2_PACKAGE_APR
> +	help
> +	  The utility library for the apache runtime project
> +
> +	  http://apr.apache.org/
> diff --git a/package/apr-util/apr-util-1.4.1-remove-checkapr.patch  
> b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
> new file mode 100644
> index 0000000..46cc595
> --- /dev/null
> +++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
> @@ -0,0 +1,12 @@
> +diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
> +--- apr-util-1.4.1/configure.in	2009-12-17 20:15:19.000000000 +0100
> ++++ apr-util-1.4.1-fix/configure.in	2012-04-24 10:10:32.000000000 +0200
> +@@ -99,7 +99,7 @@ fi
> + dnl
> + dnl Find the APR includes directory and (possibly) the source (base)  
> dir.
> + dnl
> +-APU_FIND_APR
> ++dnl APU_FIND_APR
> +
> + dnl
> + dnl even though we use apr_rules.mk for building apr-util, we need
> diff --git a/package/apr-util/apr-util.mk b/package/apr-util/apr-util.mk
> new file mode 100644
> index 0000000..f5f4d80
> --- /dev/null
> +++ b/package/apr-util/apr-util.mk
> @@ -0,0 +1,14 @@
> +#############################################################
> +#
> +# apr-util
> +#
> +#############################################################
> +APR_UTIL_VERSION = 1.4.1
> +APR_UTIL_SITE = http://archive.apache.org/dist/apr
> +APR_UTIL_INSTALL_STAGING = YES
> +APR_UTIL_DEPENDENCIES = apr
> +APR_UTIL_CONF_OPT = \
> +	--prefix=/usr \
> +	--with-apr=$(BUILD_DIR)/apr-$(APR_VERSION)
> +
> +$(eval $(call AUTOTARGETS))

hmmm... I sent this mail with the --subject-prefix='PATCH v2' option.
Can anyone tell me why I can't see this prefix in the Subject?

And with my commit messages I also messed up something...

I hope the patch is still usable!

regards
Rico

-- 
Erstellt mit Operas revolution?rem E-Mail-Modul: http://www.opera.com/mail/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH] Add new package apr-util
@ 2012-04-24 10:22 Rico Bachmann
  2012-04-24 10:28 ` Rico Bachmann
  0 siblings, 1 reply; 8+ messages in thread
From: Rico Bachmann @ 2012-04-24 10:22 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
---
v1: Add package apr-util
    Add patch to remove check for apr
    because check for apr search for ./apr and not for defined folder

change spaces to tab

Signed-off-by: Rico Bachmann <bachmann@tofwerk.com>
---
v2: change spaces in *.mk to tab
---
 package/Config.in                                  |    1 +
 package/apr-util/Config.in                         |    7 +++++++
 .../apr-util/apr-util-1.4.1-remove-checkapr.patch  |   12 ++++++++++++
 package/apr-util/apr-util.mk                       |   14 ++++++++++++++
 4 files changed, 34 insertions(+), 0 deletions(-)
 create mode 100644 package/apr-util/Config.in
 create mode 100644 package/apr-util/apr-util-1.4.1-remove-checkapr.patch
 create mode 100644 package/apr-util/apr-util.mk

diff --git a/package/Config.in b/package/Config.in
index 5ae1c81..805f56a 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -375,6 +375,7 @@ endmenu
 
 menu "Other"
 source "package/apr/Config.in"
+source "package/apr-util/Config.in"
 source "package/fftw/Config.in"
 source "package/libargtable2/Config.in"
 source "package/argp-standalone/Config.in"
diff --git a/package/apr-util/Config.in b/package/apr-util/Config.in
new file mode 100644
index 0000000..a4411ba
--- /dev/null
+++ b/package/apr-util/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_APR_UTIL
+	bool "apr-util"
+	select BR2_PACKAGE_APR
+	help
+	  The utility library for the apache runtime project
+
+	  http://apr.apache.org/
diff --git a/package/apr-util/apr-util-1.4.1-remove-checkapr.patch b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
new file mode 100644
index 0000000..46cc595
--- /dev/null
+++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
@@ -0,0 +1,12 @@
+diff -rupN apr-util-1.4.1/configure.in apr-util-1.4.1-fix/configure.in
+--- apr-util-1.4.1/configure.in	2009-12-17 20:15:19.000000000 +0100
++++ apr-util-1.4.1-fix/configure.in	2012-04-24 10:10:32.000000000 +0200
+@@ -99,7 +99,7 @@ fi
+ dnl
+ dnl Find the APR includes directory and (possibly) the source (base) dir.
+ dnl
+-APU_FIND_APR
++dnl APU_FIND_APR
+ 
+ dnl
+ dnl even though we use apr_rules.mk for building apr-util, we need
diff --git a/package/apr-util/apr-util.mk b/package/apr-util/apr-util.mk
new file mode 100644
index 0000000..f5f4d80
--- /dev/null
+++ b/package/apr-util/apr-util.mk
@@ -0,0 +1,14 @@
+#############################################################
+#
+# apr-util
+#
+#############################################################
+APR_UTIL_VERSION = 1.4.1
+APR_UTIL_SITE = http://archive.apache.org/dist/apr
+APR_UTIL_INSTALL_STAGING = YES
+APR_UTIL_DEPENDENCIES = apr
+APR_UTIL_CONF_OPT = \
+	--prefix=/usr \
+	--with-apr=$(BUILD_DIR)/apr-$(APR_VERSION)
+
+$(eval $(call AUTOTARGETS))
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2012-04-24 10:56 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-24 10:11 [Buildroot] [PATCH] Add new package apr-util Rico Bachmann
2012-04-24 10:24 ` Thomas Petazzoni
2012-04-24 10:51   ` Rico Bachmann
2012-04-24 10:22 Rico Bachmann
2012-04-24 10:28 ` Rico Bachmann
2012-04-24 10:42   ` Yegor Yefremov
2012-04-24 10:46     ` Yegor Yefremov
2012-04-24 10:56       ` Rico Bachmann

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.