From: Robert Yang <liezhi.yang@windriver.com>
To: Koen Kooi <koen@dominion.thruhere.net>,
Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: Koen Kooi <koen.kooi@linaro.org>,
openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] strace: fix build for aarch64
Date: Wed, 15 Apr 2015 21:53:10 +0800 [thread overview]
Message-ID: <552E6D46.1010701@windriver.com> (raw)
In-Reply-To: <6A30645B-9ABE-4498-AF6C-4E432984ED95@dominion.thruhere.net>
On 04/15/2015 09:32 PM, Koen Kooi wrote:
>
>> Op 15 apr. 2015, om 15:29 heeft Richard Purdie <richard.purdie@linuxfoundation.org> het volgende geschreven:
>>
>> On Wed, 2015-04-15 at 14:32 +0200, Koen Kooi wrote:
>>> Upstream added aarch64 support but forgot to update 'make dist' leading to missing files in the tarball.
>>>
>>> Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
>>> ---
>>> .../0001-Add-linux-aarch64-arch_regs.h.patch | 25 ++++++++++++++++++++++
>>> meta/recipes-devtools/strace/strace_4.10.bb | 1 +
>>> 2 files changed, 26 insertions(+)
>>> create mode 100644 meta/recipes-devtools/strace/strace/0001-Add-linux-aarch64-arch_regs.h.patch
>>
>> I'm guessing this is a better fix than Robert's fix for strace?
>
> This shrinks the diff between strace.git and ${S}, Roberts version increases it. It’s the same one-liner type fix, though.
Thanks, please drop my patch.
// Robert
>
>> Let me know if they're different issues…
>
> It’s the same issue.
>
> regards,
>
> Koen.
>
>>
>> Cheers,
>>
>> Richard
>>
>>> diff --git a/meta/recipes-devtools/strace/strace/0001-Add-linux-aarch64-arch_regs.h.patch b/meta/recipes-devtools/strace/strace/0001-Add-linux-aarch64-arch_regs.h.patch
>>> new file mode 100644
>>> index 0000000..f70766b
>>> --- /dev/null
>>> +++ b/meta/recipes-devtools/strace/strace/0001-Add-linux-aarch64-arch_regs.h.patch
>>> @@ -0,0 +1,25 @@
>>> +From f85854131c8265f2eb59c714dcea5c4b3dc09bed Mon Sep 17 00:00:00 2001
>>> +From: Koen Kooi <koen.kooi@linaro.org>
>>> +Date: Wed, 15 Apr 2015 14:29:37 +0200
>>> +Subject: [PATCH] Add linux/aarch64/arch_regs.h
>>> +
>>> +It is missing from the tarball, but it is present in git for v4.10
>>> +
>>> +Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
>>> +Upstream-status: Pending
>>> +---
>>> + linux/aarch64/arch_regs.h | 2 ++
>>> + 1 file changed, 2 insertions(+)
>>> + create mode 100644 linux/aarch64/arch_regs.h
>>> +
>>> +diff --git a/linux/aarch64/arch_regs.h b/linux/aarch64/arch_regs.h
>>> +new file mode 100644
>>> +index 0000000..9a5e33e
>>> +--- /dev/null
>>> ++++ b/linux/aarch64/arch_regs.h
>>> +@@ -0,0 +1,2 @@
>>> ++extern uint64_t *const aarch64_sp_ptr;
>>> ++extern uint32_t *const arm_sp_ptr;
>>> +--
>>> +1.9.3
>>> +
>>> diff --git a/meta/recipes-devtools/strace/strace_4.10.bb b/meta/recipes-devtools/strace/strace_4.10.bb
>>> index 416c387..707cb76 100644
>>> --- a/meta/recipes-devtools/strace/strace_4.10.bb
>>> +++ b/meta/recipes-devtools/strace/strace_4.10.bb
>>> @@ -5,6 +5,7 @@ LICENSE = "BSD"
>>> LIC_FILES_CHKSUM = "file://COPYING;md5=124500c21e856f0912df29295ba104c7"
>>>
>>> SRC_URI = "${SOURCEFORGE_MIRROR}/strace/strace-${PV}.tar.xz \
>>> + file://0001-Add-linux-aarch64-arch_regs.h.patch \
>>> file://git-version-gen \
>>> file://strace-add-configure-options.patch \
>>> file://Makefile-ptest.patch \
>>> --
>>> 1.9.3
>>>
>>
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
>
>
prev parent reply other threads:[~2015-04-15 13:53 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-15 12:32 [PATCH] strace: fix build for aarch64 Koen Kooi
2015-04-15 13:29 ` Richard Purdie
2015-04-15 13:32 ` Koen Kooi
2015-04-15 13:53 ` Robert Yang [this message]
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=552E6D46.1010701@windriver.com \
--to=liezhi.yang@windriver.com \
--cc=koen.kooi@linaro.org \
--cc=koen@dominion.thruhere.net \
--cc=openembedded-core@lists.openembedded.org \
--cc=richard.purdie@linuxfoundation.org \
/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.