From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 2/7] package/pmake: add host pmake
Date: Wed, 30 Jan 2013 23:47:18 +0100 [thread overview]
Message-ID: <5109A2F6.5090106@mind.be> (raw)
In-Reply-To: <201301302343.22202.yann.morin.1998@free.fr>
On 30/01/13 23:43, Yann E. MORIN wrote:
> Thomas, Arnout, All,
>
> On Wednesday 30 January 2013 Thomas Petazzoni wrote:
>> Dear Arnout Vandecappelle,
>>
>> On Wed, 30 Jan 2013 17:55:59 +0100, Arnout Vandecappelle wrote:
>>
>>> Doesn't it? It seems to work for liblockfile...
>>
>> Does it really work properly?
>>
>> In the past, a tarball was directly fed into patch, which was finding
>> some patch contents and that's it. Since then, the apply-patches.sh
>> script has been reworked to properly apply the patches within a
>> tarball, but I don't think it applies patches that are contained in
>> a debian/patches/ directory.
>
> Ah, I missed that. Indeed it works.
>
> What happens (eek! apply-patches.sh is hard to read) is that, if a patch
> is neamd like a tarball, it is extracted in a temp dir, and that temp dir
> is scanned as if it was passed as arg to apply-patches.sh:
It's scanned recursively, even.
And it uses the series file, if present.
>
> ---8<---
> elif echo "$i" | grep -q -E "\.tar(\..*)?$|\.tbz2?$|\.tgz$" ; then
> unpackedarchivedir="$builddir/.patches-$(basename $i)-unpacked"
> rm -rf "$unpackedarchivedir" 2> /dev/null
> mkdir "$unpackedarchivedir"
> tar -C "$unpackedarchivedir" -xaf "${path}/$i"
> scan_patchdir "$unpackedarchivedir"
> ---8<---
>
> So, yes, it works. Yet, I still find it a bit ugly (but less so than I
> initially suspected). And it is not documented.
>
> Should this be considered a feature? Can we rely on it?
>
> If so, then I'll update the doc accordingly.
I seem to remember that support for tar and a series file was added for
exactly this purpose.
In terms of ugliness: is there an alternative? Except not relying on
Debian patches, of course... I was going to say: you could try the Gentoo
version, but that one is actually using the Debian patches too :-)
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
next prev parent reply other threads:[~2013-01-30 22:47 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-29 22:33 [Buildroot] [pull request v3] Pull request for branch yem-new-packages Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 1/7] package/usbredir: update to version 0.6 Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 2/7] package/pmake: add host pmake Yann E. MORIN
2013-01-30 7:52 ` Arnout Vandecappelle
2013-01-30 13:13 ` Yann E. MORIN
2013-01-30 13:24 ` Thomas Petazzoni
2013-01-30 14:34 ` Yann E. MORIN
2013-01-30 14:49 ` Thomas Petazzoni
2013-01-30 18:50 ` Yann E. MORIN
2013-01-30 16:55 ` Arnout Vandecappelle
2013-01-30 22:32 ` Thomas Petazzoni
2013-01-30 22:43 ` Yann E. MORIN
2013-01-30 22:47 ` Arnout Vandecappelle [this message]
2013-01-30 23:20 ` Yann E. MORIN
2013-01-30 22:36 ` Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 3/7] package/libbsd: new package Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 4/7] package/libedit2: " Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 5/7] package/ceph: " Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 6/7] package/dtc: " Yann E. MORIN
2013-01-29 22:33 ` [Buildroot] [PATCH 7/7] package/dtc: add option to install programs Yann E. MORIN
-- strict thread matches above, loose matches on Subject: below --
2013-02-03 13:40 [Buildroot] [pull request v4] Pull request for branch yem-new-packages Yann E. MORIN
2013-02-03 13:40 ` [Buildroot] [PATCH 2/7] package/pmake: add host pmake Yann E. MORIN
2013-02-04 16:01 [Buildroot] [pull request v5] Pull request for branch yem-new-packages Yann E. MORIN
2013-02-04 16:01 ` [Buildroot] [PATCH 2/7] package/pmake: add host pmake Yann E. MORIN
2013-02-04 18:54 ` Arnout Vandecappelle
2013-02-06 18:51 ` Yann E. MORIN
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=5109A2F6.5090106@mind.be \
--to=arnout@mind.be \
--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