Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
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/

  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