* [PATCH 0/1] [RFC] Fix hard path of opensp, Edwin, Sep9, 2011
@ 2011-09-09 2:53 edwin.zhai
2011-09-09 2:53 ` [PATCH 1/1] opensp: Fix hard path in native nsgmls edwin.zhai
0 siblings, 1 reply; 6+ messages in thread
From: edwin.zhai @ 2011-09-09 2:53 UTC (permalink / raw)
To: openembedded-core
From: Zhai Edwin <edwin.zhai@intel.com>
Saul,
This is one fix to use "-D" as wrapper rather than environment val according to
your suggestion. Could you pls. have a look?
Thanks,
Edwin
The following changes since commit 52835dd30110d51a839872f198eef3b12e369850:
avahi: only use gtk if DISTRO_FEATURES includes x11 (2011-09-07 23:44:29 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib gzhai/master2
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=gzhai/master2
Zhai Edwin (1):
opensp: Fix hard path in native nsgmls.
meta/recipes-devtools/opensp/opensp_1.5.bb | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/1] opensp: Fix hard path in native nsgmls.
2011-09-09 2:53 [PATCH 0/1] [RFC] Fix hard path of opensp, Edwin, Sep9, 2011 edwin.zhai
@ 2011-09-09 2:53 ` edwin.zhai
2011-09-09 4:49 ` Saul Wold
0 siblings, 1 reply; 6+ messages in thread
From: edwin.zhai @ 2011-09-09 2:53 UTC (permalink / raw)
To: openembedded-core
From: Zhai Edwin <edwin.zhai@intel.com>
This relocation issues caused following bug and introduce one partial fix bb024d:
[YOCTO #1440]
But the environment val hacking is not working all the time, as nsgmsl may
append wrong path in some case. This fix use search directory "-D" as solution.
It's better if got fixed from opensp side when next upgrade.
Signed-off-by: Zhai Edwin <edwin.zhai@intel.com>
CC: Saul Would <sgw@linux.intel.com>
---
meta/recipes-devtools/opensp/opensp_1.5.bb | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-devtools/opensp/opensp_1.5.bb b/meta/recipes-devtools/opensp/opensp_1.5.bb
index 0269c87..d692137 100644
--- a/meta/recipes-devtools/opensp/opensp_1.5.bb
+++ b/meta/recipes-devtools/opensp/opensp_1.5.bb
@@ -44,8 +44,8 @@ do_install_append() {
do_install_append_virtclass-native() {
for util in nsgmls sgmlnorm spam spcat spent sx; do
- create_wrapper ${D}/${bindir}/$util \
- SGML_CATALOG_FILES=${sysconfdir}/sgml/catalog
+ create_cmdline_wrapper ${D}/${bindir}/$util \
+ -D ${sysconfdir}/sgml
done
}
--
1.7.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] opensp: Fix hard path in native nsgmls.
2011-09-09 2:53 ` [PATCH 1/1] opensp: Fix hard path in native nsgmls edwin.zhai
@ 2011-09-09 4:49 ` Saul Wold
2011-09-09 14:16 ` Zhai, Edwin
0 siblings, 1 reply; 6+ messages in thread
From: Saul Wold @ 2011-09-09 4:49 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On 09/08/2011 07:53 PM, edwin.zhai@intel.com wrote:
> From: Zhai Edwin<edwin.zhai@intel.com>
>
> This relocation issues caused following bug and introduce one partial fix bb024d:
> [YOCTO #1440]
>
> But the environment val hacking is not working all the time, as nsgmsl may
> append wrong path in some case. This fix use search directory "-D" as solution.
> It's better if got fixed from opensp side when next upgrade.
>
> Signed-off-by: Zhai Edwin<edwin.zhai@intel.com>
> CC: Saul Would<sgw@linux.intel.com>
> ---
> meta/recipes-devtools/opensp/opensp_1.5.bb | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-devtools/opensp/opensp_1.5.bb b/meta/recipes-devtools/opensp/opensp_1.5.bb
> index 0269c87..d692137 100644
> --- a/meta/recipes-devtools/opensp/opensp_1.5.bb
> +++ b/meta/recipes-devtools/opensp/opensp_1.5.bb
> @@ -44,8 +44,8 @@ do_install_append() {
>
> do_install_append_virtclass-native() {
> for util in nsgmls sgmlnorm spam spcat spent sx; do
> - create_wrapper ${D}/${bindir}/$util \
> - SGML_CATALOG_FILES=${sysconfdir}/sgml/catalog
> + create_cmdline_wrapper ${D}/${bindir}/$util \
> + -D ${sysconfdir}/sgml
> done
> }
>
Have you tested this with iputils? When I tried this, it did not work
correctly for me, can you give me more of idea of how you tested this.
Sau!
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] opensp: Fix hard path in native nsgmls.
2011-09-09 4:49 ` Saul Wold
@ 2011-09-09 14:16 ` Zhai, Edwin
2011-09-09 15:11 ` Saul Wold
0 siblings, 1 reply; 6+ messages in thread
From: Zhai, Edwin @ 2011-09-09 14:16 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On 2011/9/9 12:49, Saul Wold wrote:
> On 09/08/2011 07:53 PM, edwin.zhai@intel.com wrote:
>> From: Zhai Edwin<edwin.zhai@intel.com>
>>
>> This relocation issues caused following bug and introduce one partial fix bb024d:
>> [YOCTO #1440]
>>
>> But the environment val hacking is not working all the time, as nsgmsl may
>> append wrong path in some case. This fix use search directory "-D" as solution.
>> It's better if got fixed from opensp side when next upgrade.
>>
>> Signed-off-by: Zhai Edwin<edwin.zhai@intel.com>
>> CC: Saul Would<sgw@linux.intel.com>
>> ---
>> meta/recipes-devtools/opensp/opensp_1.5.bb | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-devtools/opensp/opensp_1.5.bb b/meta/recipes-devtools/opensp/opensp_1.5.bb
>> index 0269c87..d692137 100644
>> --- a/meta/recipes-devtools/opensp/opensp_1.5.bb
>> +++ b/meta/recipes-devtools/opensp/opensp_1.5.bb
>> @@ -44,8 +44,8 @@ do_install_append() {
>>
>> do_install_append_virtclass-native() {
>> for util in nsgmls sgmlnorm spam spcat spent sx; do
>> - create_wrapper ${D}/${bindir}/$util \
>> - SGML_CATALOG_FILES=${sysconfdir}/sgml/catalog
>> + create_cmdline_wrapper ${D}/${bindir}/$util \
>> + -D ${sysconfdir}/sgml
>> done
>> }
>>
> Have you tested this with iputils? When I tried this, it did not work
> correctly for me, can you give me more of idea of how you tested this.
Sorry, I forgot to rename the tmp to make sysroot unavailable:( After
renaming, the bug still exist.
Upgrading to 1.5.2 doesn't help. After debug with strace, I found nsgmls
append ".." to other paths when inputting "../index.db" as you said.
It's reasonable for entity such as ping.sgml, as they usually sit in
same dir as index.db. But catalog should not do this.
I have tried "-c" to specify catalog path, and found it works. Could
you pls. have a try on the same contrib tree?
contrib/gzhai/master2
Thanks,
Edwin
>
> Sau!
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] opensp: Fix hard path in native nsgmls.
2011-09-09 14:16 ` Zhai, Edwin
@ 2011-09-09 15:11 ` Saul Wold
2011-09-10 1:00 ` Zhai, Edwin
0 siblings, 1 reply; 6+ messages in thread
From: Saul Wold @ 2011-09-09 15:11 UTC (permalink / raw)
To: Zhai, Edwin; +Cc: Patches and discussions about the oe-core layer
On 09/09/2011 07:16 AM, Zhai, Edwin wrote:
> On 2011/9/9 12:49, Saul Wold wrote:
>> On 09/08/2011 07:53 PM, edwin.zhai@intel.com wrote:
>>> From: Zhai Edwin<edwin.zhai@intel.com>
>>>
>>> This relocation issues caused following bug and introduce one partial
>>> fix bb024d:
>>> [YOCTO #1440]
>>>
>>> But the environment val hacking is not working all the time, as
>>> nsgmsl may
>>> append wrong path in some case. This fix use search directory "-D" as
>>> solution.
>>> It's better if got fixed from opensp side when next upgrade.
>>>
>>> Signed-off-by: Zhai Edwin<edwin.zhai@intel.com>
>>> CC: Saul Would<sgw@linux.intel.com>
>>> ---
>>> meta/recipes-devtools/opensp/opensp_1.5.bb | 4 ++--
>>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-devtools/opensp/opensp_1.5.bb
>>> b/meta/recipes-devtools/opensp/opensp_1.5.bb
>>> index 0269c87..d692137 100644
>>> --- a/meta/recipes-devtools/opensp/opensp_1.5.bb
>>> +++ b/meta/recipes-devtools/opensp/opensp_1.5.bb
>>> @@ -44,8 +44,8 @@ do_install_append() {
>>>
>>> do_install_append_virtclass-native() {
>>> for util in nsgmls sgmlnorm spam spcat spent sx; do
>>> - create_wrapper ${D}/${bindir}/$util \
>>> - SGML_CATALOG_FILES=${sysconfdir}/sgml/catalog
>>> + create_cmdline_wrapper ${D}/${bindir}/$util \
>>> + -D ${sysconfdir}/sgml
>>> done
>>> }
>>>
>> Have you tested this with iputils? When I tried this, it did not work
>> correctly for me, can you give me more of idea of how you tested this.
>
> Sorry, I forgot to rename the tmp to make sysroot unavailable:( After
> renaming, the bug still exist.
> Upgrading to 1.5.2 doesn't help. After debug with strace, I found nsgmls
> append ".." to other paths when inputting "../index.db" as you said.
> It's reasonable for entity such as ping.sgml, as they usually sit in
> same dir as index.db. But catalog should not do this.
>
> I have tried "-c" to specify catalog path, and found it works. Could you
> pls. have a try on the same contrib tree?
> contrib/gzhai/master2
>
Edwin,
Did you try the -c with the linuxdoc-tools-native recipe also? I found
that when I used -c option with that recipe I then get a failure, so as
I have mentioned before I think the problem maybe in the opensp code
itself and it should not append the ".." to the catalog search patch.
Sau!
> Thanks,
> Edwin
>
>>
>> Sau!
>>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] opensp: Fix hard path in native nsgmls.
2011-09-09 15:11 ` Saul Wold
@ 2011-09-10 1:00 ` Zhai, Edwin
0 siblings, 0 replies; 6+ messages in thread
From: Zhai, Edwin @ 2011-09-10 1:00 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On 2011/9/9 23:11, Saul Wold wrote:
> On 09/09/2011 07:16 AM, Zhai, Edwin wrote:
>> On 2011/9/9 12:49, Saul Wold wrote:
>>> On 09/08/2011 07:53 PM, edwin.zhai@intel.com wrote:
>>>> From: Zhai Edwin<edwin.zhai@intel.com>
>>>>
>>>> This relocation issues caused following bug and introduce one partial
>>>> fix bb024d:
>>>> [YOCTO #1440]
>>>>
>>>> But the environment val hacking is not working all the time, as
>>>> nsgmsl may
>>>> append wrong path in some case. This fix use search directory "-D" as
>>>> solution.
>>>> It's better if got fixed from opensp side when next upgrade.
>>>>
>>>> Signed-off-by: Zhai Edwin<edwin.zhai@intel.com>
>>>> CC: Saul Would<sgw@linux.intel.com>
>>>> ---
>>>> meta/recipes-devtools/opensp/opensp_1.5.bb | 4 ++--
>>>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/meta/recipes-devtools/opensp/opensp_1.5.bb
>>>> b/meta/recipes-devtools/opensp/opensp_1.5.bb
>>>> index 0269c87..d692137 100644
>>>> --- a/meta/recipes-devtools/opensp/opensp_1.5.bb
>>>> +++ b/meta/recipes-devtools/opensp/opensp_1.5.bb
>>>> @@ -44,8 +44,8 @@ do_install_append() {
>>>>
>>>> do_install_append_virtclass-native() {
>>>> for util in nsgmls sgmlnorm spam spcat spent sx; do
>>>> - create_wrapper ${D}/${bindir}/$util \
>>>> - SGML_CATALOG_FILES=${sysconfdir}/sgml/catalog
>>>> + create_cmdline_wrapper ${D}/${bindir}/$util \
>>>> + -D ${sysconfdir}/sgml
>>>> done
>>>> }
>>>>
>>> Have you tested this with iputils? When I tried this, it did not work
>>> correctly for me, can you give me more of idea of how you tested this.
>> Sorry, I forgot to rename the tmp to make sysroot unavailable:( After
>> renaming, the bug still exist.
>> Upgrading to 1.5.2 doesn't help. After debug with strace, I found nsgmls
>> append ".." to other paths when inputting "../index.db" as you said.
>> It's reasonable for entity such as ping.sgml, as they usually sit in
>> same dir as index.db. But catalog should not do this.
>>
>> I have tried "-c" to specify catalog path, and found it works. Could you
>> pls. have a try on the same contrib tree?
>> contrib/gzhai/master2
>>
> Edwin,
>
> Did you try the -c with the linuxdoc-tools-native recipe also? I found
> that when I used -c option with that recipe I then get a failure,
No. You have mentioned focus on iputils. Seems this fix is same as the
one in poky.
> so as
> I have mentioned before I think the problem maybe in the opensp code
> itself and it should not append the ".." to the catalog search patch.
Agree with you. but hack in the source code need some time.
>> Thanks,
>> Edwin
>>
>>> Sau!
>>>
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-09-10 1:05 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-09-09 2:53 [PATCH 0/1] [RFC] Fix hard path of opensp, Edwin, Sep9, 2011 edwin.zhai
2011-09-09 2:53 ` [PATCH 1/1] opensp: Fix hard path in native nsgmls edwin.zhai
2011-09-09 4:49 ` Saul Wold
2011-09-09 14:16 ` Zhai, Edwin
2011-09-09 15:11 ` Saul Wold
2011-09-10 1:00 ` Zhai, Edwin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox