[Buildroot] [PATCH v3] Add new package apr-util
Rico Bachmann
bachmann at tofwerk.com
Wed Apr 25 07:40:09 UTC 2012
Hi Thomas
Am 24.04.2012, 14:05 Uhr, schrieb Thomas Petazzoni
<thomas.petazzoni at free-electrons.com>:
> Hello Rico,
>
> Le Tue, 24 Apr 2012 13:55:24 +0200,
> Rico Bachmann <bachmann at tofwerk.com> a écrit :
>
>> Signed-off-by: Rico Bachmann <bachmann at 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
>> v2: Change spaces to tab in *.mk
>> v3: Add Signed-off-by and comments to package-patch
>> ---
>> ---
>
> Those two lines with '---' are useless.
>
>> 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..128a1b0
>> --- /dev/null
>> +++ b/package/apr-util/apr-util-1.4.1-remove-checkapr.patch
>> @@ -0,0 +1,17 @@
>> +Signed-off-by: Rico Bachmann <bachmann at tofwerk.com>
>> +---
>> +disable the check for APU_FIND_APR because the check only looks into
>> folders
>> +named apr, and our source-tree is called apr-versionnumber (e.g.
>> apr-1.4.6)
>> +---
>
> The description should be *before* the Signed-off-by, and not separated
> by ---.
>
> It seems like you misunderstand what the format of a patch should be.
> Here is an explanation that I hope will clarify things:
I'm really not that into patches... no diffrence if it is the patch
format or patches at all... At the moment, I'm learning by doing...
Thanks for your explanation.
>
> ------------------------8<-------------------
> This is the title, should be less than ~80 chars
>
> This is a long description that explains what the patch is doing, why,
> how, and all other details that are useful. This description (and the
> title above, and the above Signed-off-by line) will be part of the
> commit log and remain in the Git history. Therefore, it should *not*
> contain any changelog information about the patch, or other "personal"
> messages. The title and the description must be separated by one empty
> line.
>
> Signed-off-by: Foo Bar <foo at bar.com>
> ---
> This is some text that will *not* be included in the Git history, so it
> is typically used for the changelog.
>
> package/Config.in | 1 +
> package/apr-util/Config.in | 7 +++++++
> .../apr-util/apr-util-1.4.1-remove-checkapr.patch | 17
> +++++++++++++++++
> package/apr-util/apr-util.mk | 14 ++++++++++++++
> 4 files changed, 39 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"
> ------------------------8<-------------------
>
> Regards,
>
> Thomas
I'll try to send a proper formated patch today. As soon as I find time.
And I do my best to get my next patch in the right format from beginning!
so long
Rico
--
Erstellt mit Operas revolutionärem E-Mail-Modul: http://www.opera.com/mail/
More information about the buildroot
mailing list