diff mbox

Add new package apr-util

Message ID 1335262938-31434-1-git-send-email-bachmann@tofwerk.com
State Superseded, archived
Headers show

Commit Message

Rico Bachmann April 24, 2012, 10:22 a.m. UTC
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

Comments

Rico Bachmann April 24, 2012, 10:28 a.m. UTC | #1
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
Yegor Yefremov April 24, 2012, 10:42 a.m. UTC | #2
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
Yegor Yefremov April 24, 2012, 10:46 a.m. UTC | #3
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
Rico Bachmann April 24, 2012, 10:56 a.m. UTC | #4
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@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox

Patch

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))