* [PATCH] gmp: remove referencing paths from the build host
@ 2015-04-17 8:50 Junling Zheng
2015-04-17 18:24 ` Andre McCurdy
0 siblings, 1 reply; 5+ messages in thread
From: Junling Zheng @ 2015-04-17 8:50 UTC (permalink / raw)
To: openembedded-core
Currently ${D}${includedir}/gmp.h references the path of build host
incorrectly, remove it.
Signed-off-by: Junling Zheng <zhengjunling@huawei.com>
---
meta/recipes-support/gmp/gmp.inc | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/meta/recipes-support/gmp/gmp.inc b/meta/recipes-support/gmp/gmp.inc
index 558ee06..a80b356 100644
--- a/meta/recipes-support/gmp/gmp.inc
+++ b/meta/recipes-support/gmp/gmp.inc
@@ -26,4 +26,8 @@ EXTRA_OECONF += " --enable-cxx=detect"
PACKAGES =+ "libgmpxx"
FILES_libgmpxx = "${libdir}/libgmpxx${SOLIBS}"
+do_install_append_class-target() {
+ sed -i "s+--sysroot=${STAGING_DIR_HOST}++g" ${D}${includedir}/gmp.h
+}
+
SSTATE_SCAN_FILES += "gmp.h"
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] gmp: remove referencing paths from the build host
2015-04-17 8:50 [PATCH] gmp: remove referencing paths from the build host Junling Zheng
@ 2015-04-17 18:24 ` Andre McCurdy
2015-04-20 1:30 ` Junling Zheng
0 siblings, 1 reply; 5+ messages in thread
From: Andre McCurdy @ 2015-04-17 18:24 UTC (permalink / raw)
To: Junling Zheng; +Cc: OE Core mailing list
On Fri, Apr 17, 2015 at 1:50 AM, Junling Zheng <zhengjunling@huawei.com> wrote:
> Currently ${D}${includedir}/gmp.h references the path of build host
> incorrectly, remove it.
>
> Signed-off-by: Junling Zheng <zhengjunling@huawei.com>
> ---
> meta/recipes-support/gmp/gmp.inc | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/meta/recipes-support/gmp/gmp.inc b/meta/recipes-support/gmp/gmp.inc
> index 558ee06..a80b356 100644
> --- a/meta/recipes-support/gmp/gmp.inc
> +++ b/meta/recipes-support/gmp/gmp.inc
> @@ -26,4 +26,8 @@ EXTRA_OECONF += " --enable-cxx=detect"
> PACKAGES =+ "libgmpxx"
> FILES_libgmpxx = "${libdir}/libgmpxx${SOLIBS}"
>
> +do_install_append_class-target() {
> + sed -i "s+--sysroot=${STAGING_DIR_HOST}++g" ${D}${includedir}/gmp.h
> +}
It's possible that STAGING_DIR_HOST could contain '+' characters, so
'|' would be a safer sed separator to use here.
> +
> SSTATE_SCAN_FILES += "gmp.h"
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] gmp: remove referencing paths from the build host
2015-04-17 18:24 ` Andre McCurdy
@ 2015-04-20 1:30 ` Junling Zheng
2015-04-20 15:15 ` Richard Purdie
0 siblings, 1 reply; 5+ messages in thread
From: Junling Zheng @ 2015-04-20 1:30 UTC (permalink / raw)
To: Andre McCurdy; +Cc: OE Core mailing list
On 2015/4/18 2:24, Andre McCurdy wrote:
> On Fri, Apr 17, 2015 at 1:50 AM, Junling Zheng <zhengjunling@huawei.com> wrote:
>> Currently ${D}${includedir}/gmp.h references the path of build host
>> incorrectly, remove it.
>>
>> Signed-off-by: Junling Zheng <zhengjunling@huawei.com>
>> ---
>> meta/recipes-support/gmp/gmp.inc | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/meta/recipes-support/gmp/gmp.inc b/meta/recipes-support/gmp/gmp.inc
>> index 558ee06..a80b356 100644
>> --- a/meta/recipes-support/gmp/gmp.inc
>> +++ b/meta/recipes-support/gmp/gmp.inc
>> @@ -26,4 +26,8 @@ EXTRA_OECONF += " --enable-cxx=detect"
>> PACKAGES =+ "libgmpxx"
>> FILES_libgmpxx = "${libdir}/libgmpxx${SOLIBS}"
>>
>> +do_install_append_class-target() {
>> + sed -i "s+--sysroot=${STAGING_DIR_HOST}++g" ${D}${includedir}/gmp.h
>> +}
>
> It's possible that STAGING_DIR_HOST could contain '+' characters, so
> '|' would be a safer sed separator to use here.
>
Ok, thanks for your suggestion:)
>> +
>> SSTATE_SCAN_FILES += "gmp.h"
>> --
>> 1.9.1
>>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] gmp: remove referencing paths from the build host
2015-04-20 1:30 ` Junling Zheng
@ 2015-04-20 15:15 ` Richard Purdie
2015-04-21 1:06 ` Junling Zheng
0 siblings, 1 reply; 5+ messages in thread
From: Richard Purdie @ 2015-04-20 15:15 UTC (permalink / raw)
To: Junling Zheng; +Cc: OE Core mailing list
On Mon, 2015-04-20 at 09:30 +0800, Junling Zheng wrote:
> On 2015/4/18 2:24, Andre McCurdy wrote:
> > On Fri, Apr 17, 2015 at 1:50 AM, Junling Zheng <zhengjunling@huawei.com> wrote:
> >> Currently ${D}${includedir}/gmp.h references the path of build host
> >> incorrectly, remove it.
> >>
> >> Signed-off-by: Junling Zheng <zhengjunling@huawei.com>
> >> ---
> >> meta/recipes-support/gmp/gmp.inc | 4 ++++
> >> 1 file changed, 4 insertions(+)
> >>
> >> diff --git a/meta/recipes-support/gmp/gmp.inc b/meta/recipes-support/gmp/gmp.inc
> >> index 558ee06..a80b356 100644
> >> --- a/meta/recipes-support/gmp/gmp.inc
> >> +++ b/meta/recipes-support/gmp/gmp.inc
> >> @@ -26,4 +26,8 @@ EXTRA_OECONF += " --enable-cxx=detect"
> >> PACKAGES =+ "libgmpxx"
> >> FILES_libgmpxx = "${libdir}/libgmpxx${SOLIBS}"
> >>
> >> +do_install_append_class-target() {
> >> + sed -i "s+--sysroot=${STAGING_DIR_HOST}++g" ${D}${includedir}/gmp.h
> >> +}
> >
> > It's possible that STAGING_DIR_HOST could contain '+' characters, so
> > '|' would be a safer sed separator to use here.
> >
>
> Ok, thanks for your suggestion:)
I fixed this one whilst merging it (faster than dropping and re-adding).
Cheers,
Richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] gmp: remove referencing paths from the build host
2015-04-20 15:15 ` Richard Purdie
@ 2015-04-21 1:06 ` Junling Zheng
0 siblings, 0 replies; 5+ messages in thread
From: Junling Zheng @ 2015-04-21 1:06 UTC (permalink / raw)
To: Richard Purdie; +Cc: OE Core mailing list
On 2015/4/20 23:15, Richard Purdie wrote:
> On Mon, 2015-04-20 at 09:30 +0800, Junling Zheng wrote:
>> On 2015/4/18 2:24, Andre McCurdy wrote:
>>> On Fri, Apr 17, 2015 at 1:50 AM, Junling Zheng <zhengjunling@huawei.com> wrote:
>>>> Currently ${D}${includedir}/gmp.h references the path of build host
>>>> incorrectly, remove it.
>>>>
>>>> Signed-off-by: Junling Zheng <zhengjunling@huawei.com>
>>>> ---
>>>> meta/recipes-support/gmp/gmp.inc | 4 ++++
>>>> 1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/meta/recipes-support/gmp/gmp.inc b/meta/recipes-support/gmp/gmp.inc
>>>> index 558ee06..a80b356 100644
>>>> --- a/meta/recipes-support/gmp/gmp.inc
>>>> +++ b/meta/recipes-support/gmp/gmp.inc
>>>> @@ -26,4 +26,8 @@ EXTRA_OECONF += " --enable-cxx=detect"
>>>> PACKAGES =+ "libgmpxx"
>>>> FILES_libgmpxx = "${libdir}/libgmpxx${SOLIBS}"
>>>>
>>>> +do_install_append_class-target() {
>>>> + sed -i "s+--sysroot=${STAGING_DIR_HOST}++g" ${D}${includedir}/gmp.h
>>>> +}
>>>
>>> It's possible that STAGING_DIR_HOST could contain '+' characters, so
>>> '|' would be a safer sed separator to use here.
>>>
>>
>> Ok, thanks for your suggestion:)
>
> I fixed this one whilst merging it (faster than dropping and re-adding).
>
> Cheers,
>
> Richard
>
>
>
Thanks:)
Cheers,
Junling
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-04-21 1:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-17 8:50 [PATCH] gmp: remove referencing paths from the build host Junling Zheng
2015-04-17 18:24 ` Andre McCurdy
2015-04-20 1:30 ` Junling Zheng
2015-04-20 15:15 ` Richard Purdie
2015-04-21 1:06 ` Junling Zheng
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.