All of 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 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.