<div dir="ltr">Can you rebase this on top of your upgrade patch? At the moment this and the upgrade conflict.<div><br></div><div>Ross</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 12 August 2016 at 03:08, Dengke Du <span dir="ltr"><<a href="mailto:dengke.du@windriver.com" target="_blank">dengke.du@windriver.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">1. Filter the extra white space in intl.right<br>
<br>
When the sub-test unicode2.sub of intl.tests executed, it produced<br>
compact results without extra white space, compared to intl.right,<br>
it failed.<br>
<br>
So we need to filter the extra white space in intl.right.<br>
<br>
Import this patch for intl.right from bash devel branch:<br>
<br>
<a href="http://git.savannah.gnu.org/cgit/bash.git/log/?h=devel" rel="noreferrer" target="_blank">http://git.savannah.gnu.org/<wbr>cgit/bash.git/log/?h=devel</a><br>
<br>
Commit is:<br>
<br>
85ec0778f9d778e1820fb8c0e3e996<wbr>f2d1103b45<br>
<br>
2. Change intl.right correspond to the unicode3.sub's output<br>
<br>
In sub-test unicode3.sub of intl.tests have this:<br>
<br>
printf %q "$payload"<br>
<br>
The payload variable was assigned by ASCII characters, when using<br>
'%q' format strings, it means print the associated argument shell-quoted.<br>
<br>
When the strings contain the non-alpha && non-digit && non-punctuation &&<br>
non-ISO 646 character(7-bit), it would output like this: " $'...', ANSI-C<br>
style quoted string. We can check the bash source code at:<br>
<br>
<a href="http://git.savannah.gnu.org/cgit/bash.git/tree/builtins/printf.def#n557" rel="noreferrer" target="_blank">http://git.savannah.gnu.org/<wbr>cgit/bash.git/tree/builtins/<wbr>printf.def#n557</a><br>
<a href="http://git.savannah.gnu.org/cgit/bash.git/tree/lib/sh/strtrans.c#n331" rel="noreferrer" target="_blank">http://git.savannah.gnu.org/<wbr>cgit/bash.git/tree/lib/sh/<wbr>strtrans.c#n331</a><br>
<br>
So we need to change the intl.right contain the correct output of unicode3.sub.<br>
<br>
Import parts of this patch for intl.right from bash devel branch:<br>
<br>
<a href="http://git.savannah.gnu.org/cgit/bash.git/log/?h=devel" rel="noreferrer" target="_blank">http://git.savannah.gnu.org/<wbr>cgit/bash.git/log/?h=devel</a><br>
<br>
Commit is:<br>
<br>
74b8cbb41398b4453d8ba04d0cdd1b<wbr>25f9dcb9e3<br>
<br>
NOTE:<br>
Ensure the system contain fr-fr & de-de locales<br>
<br>
Some tests of intl.tests need to change locales to accomplish. So you need add<br>
the following to a conf file:<br>
<br>
IMAGE_LINGUAS = "en-us fr-fr de-de"<br>
<br>
Signed-off-by: Dengke Du <<a href="mailto:dengke.du@windriver.com">dengke.du@windriver.com</a>><br>
---<br>
<span class=""> meta/recipes-extended/bash/<wbr>bash/fix-run-intl.patch | 110 +++++++++++++++++++++<br>
meta/recipes-extended/bash/<a href="http://bash_4.3.30.bb" rel="noreferrer" target="_blank">bas<wbr>h_4.3.30.bb</a> | 3 +-<br>
2 files changed, 112 insertions(+), 1 deletion(-)<br>
create mode 100644 meta/recipes-extended/bash/<wbr>bash/fix-run-intl.patch<br>
<br>
</span>diff --git a/meta/recipes-extended/bash/<wbr>bash/fix-run-intl.patch b/meta/recipes-extended/bash/<wbr>bash/fix-run-intl.patch<br>
new file mode 100644<br>
index 0000000..d4a3409<br>
--- /dev/null<br>
+++ b/meta/recipes-extended/bash/<wbr>bash/fix-run-intl.patch<br>
@@ -0,0 +1,110 @@<br>
+From a00d3161fd7b6a698bdd2ed5f0ac5f<wbr>aac580ee2a Mon Sep 17 00:00:00 2001<br>
+From: Dengke Du <<a href="mailto:dengke.du@windriver.com">dengke.du@windriver.com</a>><br>
+Date: Wed, 3 Aug 2016 23:13:00 -0400<br>
+Subject: [PATCH] fix run-intl failed<br>
+<br>
+1. Filter extra white space of intl.right<br>
+<br>
+ Due to the extra white space of intl.right, when the result of<br>
+ sub-test unicode2.sub of intl.tests compared to it, the test<br>
+ failed.<br>
+<br>
+ So we need to filter the extra white space of intl.right.<br>
+<br>
+ Import this patch for intl.right from bash devel branch:<br>
+<br>
+ <a href="http://git.savannah.gnu.org/cgit/bash.git/log/?h=devel" rel="noreferrer" target="_blank">http://git.savannah.gnu.org/<wbr>cgit/bash.git/log/?h=devel</a><br>
+<br>
+ commit is:<br>
+<br>
+ 85ec0778f9d778e1820fb8c0e3e996<wbr>f2d1103b45<br>
+<br>
+2. Change intl.right correspond to the unicode3.sub's output<br>
+<br>
+ In sub-test unicode3.sub of intl.tests, the payload value is:<br>
+<br>
+ payload=$'\065\247\100\063\<wbr>231\053\306\123\070\237\242\<wbr>352\263'<br>
+<br>
+ It used quoted string expansion(escaped octal) to assign ASCII<br>
+ characters to variables. So when the test run the following:<br>
+<br>
+ printf %q "$payload"<br>
+<br>
+ It produced:<br>
+<br>
+ $'5\247@3\231+\306S8\237\242\<wbr>352\263'<br>
+<br>
+ When compared to the intl.right(contain the converted character), it failed.<br>
+<br>
+ Import parts of patch for intl.right from bash devel branch:<br>
+<br>
+ <a href="http://git.savannah.gnu.org/cgit/bash.git/log/?h=devel" rel="noreferrer" target="_blank">http://git.savannah.gnu.org/<wbr>cgit/bash.git/log/?h=devel</a><br>
+<br>
+ commit is:<br>
+<br>
+ 74b8cbb41398b4453d8ba04d0cdd1b<wbr>25f9dcb9e3<br>
+<br>
+Upstream-Status: Backport<br>
+<br>
+Signed-off-by: Dengke Du <<a href="mailto:dengke.du@windriver.com">dengke.du@windriver.com</a>><br>
+---<br>
+ tests/intl.right | 30 +++++++++++++++---------------<br>
+ 1 file changed, 15 insertions(+), 15 deletions(-)<br>
+<br>
+diff --git a/tests/intl.right b/tests/intl.right<br>
+index acf108a..1efdfbe 100644<br>
+--- a/tests/intl.right<br>
++++ b/tests/intl.right<br>
+@@ -18,34 +18,34 @@ aéb<br>
+ 1.0000<br>
+ 1,0000<br>
+ Passed all 1378 Unicode tests<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 357 277 277 012<br>
++0000000 357 277 277 012<br>
+ 0000004<br>
+-0000000 357 277 277 012<br>
++0000000 357 277 277 012<br>
+ 0000004<br>
+-0000000 012<br>
++0000000 012<br>
+ 0000001<br>
+-0000000 012<br>
++0000000 012<br>
+ 0000001<br>
+-0000000 012<br>
++0000000 012<br>
+ 0000001<br>
+-0000000 012<br>
++0000000 012<br>
+ 0000001<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 303 277 012<br>
++0000000 303 277 012<br>
+ 0000003<br>
+-0000000 101 040 302 243 040 305 222 012<br>
++0000000 101 040 302 243 040 305 222 012<br>
+ 0000010<br>
+ ./unicode3.sub: line 2: 5§@3™+ÆS8Ÿ¢ê³: command not found<br>
+-5§@3™+ÆS8Ÿ¢ê³<br>
++$'5\247@3\231+\306S8\237\<wbr>242\352\263'<br>
+ + : $'5\247@3\231+\306S8\237\242\<wbr>352\263'<br>
+--<br>
+2.8.1<br>
+<br>
diff --git a/meta/recipes-extended/bash/<a href="http://bash_4.3.30.bb" rel="noreferrer" target="_blank">b<wbr>ash_4.3.30.bb</a> b/meta/recipes-extended/bash/<a href="http://bash_4.3.30.bb" rel="noreferrer" target="_blank">b<wbr>ash_4.3.30.bb</a><br>
index 95ed392..3f0dd8b 100644<br>
--- a/meta/recipes-extended/bash/<a href="http://bash_4.3.30.bb" rel="noreferrer" target="_blank">b<wbr>ash_4.3.30.bb</a><br>
+++ b/meta/recipes-extended/bash/<a href="http://bash_4.3.30.bb" rel="noreferrer" target="_blank">b<wbr>ash_4.3.30.bb</a><br>
@@ -20,7 +20,8 @@ SRC_URI = "${GNU_MIRROR}/bash/${BP}.tar.<wbr>gz;name=tarball \<br>
file://test-output.patch \<br>
file://fix-run-coproc-run-<wbr>heredoc-run-execscript-run-<wbr>test-f.patch \<br>
file://run-ptest \<br>
- file://fix-run-builtins.patch \<br>
+ file://fix-run-builtins.patch \<br>
+ file://fix-run-intl.patch \<br>
"<br>
<br>
SRC_URI[tarball.md5sum] = "<wbr>a27b3ee9be83bd3ba448c0ff52b284<wbr>47"<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.8.1<br>
<br>
</font></span><br>--<br>
______________________________<wbr>_________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.<wbr>openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/<wbr>mailman/listinfo/openembedded-<wbr>core</a><br>
<br></blockquote></div><br></div>