From: Rico Bachmann <bachmann@tofwerk.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] Add new package apr-util
Date: Tue, 24 Apr 2012 12:56:05 +0200 [thread overview]
Message-ID: <op.wc9nbrwt705kjs@rico> (raw)
In-Reply-To: CAGm1_ksW2Rj=yQViE118DYWJd3+NUm3Uo0uvP9amk421N6QxXw@mail.gmail.com
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/
next prev parent reply other threads:[~2012-04-24 10:56 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-24 10:22 [Buildroot] [PATCH] Add new package apr-util 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 message]
-- strict thread matches above, loose matches on Subject: below --
2012-04-24 10:11 Rico Bachmann
2012-04-24 10:24 ` Thomas Petazzoni
2012-04-24 10:51 ` Rico Bachmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=op.wc9nbrwt705kjs@rico \
--to=bachmann@tofwerk.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox