* [scarthgap][PATCH] openvswitch: fix ptest contains reference to TMPDIR [buildpaths]
@ 2025-03-31 20:36 martin.jansa
2025-04-02 2:23 ` Bruce Ashfield
0 siblings, 1 reply; 2+ messages in thread
From: martin.jansa @ 2025-03-31 20:36 UTC (permalink / raw)
To: meta-virtualization; +Cc: Bin Lan, Bruce Ashfield
From: Bin Lan <bin.lan.cn@windriver.com>
The EGREP in ptest/tests/atlocal contains the build paths.
The CFLAGS in ptest/tests/atlocal contains the build paths.
This change set fixs:
- set EGREP to "grep -E" in ptest/tests/atlocal
- set CFLAGS to " " in ptest/tests/atlocal
by updating the patch
openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch.
Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com>
---
...est-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch b/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch
index 571753a5..abad1458 100644
--- a/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch
+++ b/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch
@@ -16,6 +16,13 @@ Signed-off-by: He Zhe <zhe.he@windriver.com>
refresh patch to fix patch-fuzz warning
Signed-off-by: Changqing Li <changqing.li@windriver.com>
+Refresh patch to fix file ptest/tests/atlocal
+contains reference to TMPDIR [buildpaths]. The fix is:
+ - set EGREP to "grep -E" in ptest/tests/atlocal
+ - set CFLAGS to " " in ptest/tests/atlocal
+
+Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
+
Upstream-Status: Inappropriate [embedded specific]
---
Makefile.am | 1 +
@@ -37,7 +44,7 @@ new file mode 100644
index 0000000..0b4587c
--- /dev/null
+++ b/test.mk
-@@ -0,0 +1,74 @@
+@@ -0,0 +1,76 @@
+TEST_DEST ?= ${prefix}/lib/openvswitch
+TEST_ROOT ?= ${prefix}/lib/openvswitch
+TEST_DEPEND =
@@ -112,3 +119,5 @@ index 0000000..0b4587c
+ sed -i 's|$$srcdir|$$abs_srcdir|g' $(TEST_DEST)/tests/testsuite
+ sed -i 's|ovs-appctl-bashcomp\.bash|/etc/bash_completion.d/ovs-appctl-bashcomp\.bash|g' $(TEST_DEST)/tests/testsuite
+ sed -i 's|ovs-vsctl-bashcomp\.bash|/etc/bash_completion.d/ovs-vsctl-bashcomp\.bash|g' $(TEST_DEST)/tests/testsuite
++ sed -i 's|EGREP=.*|EGREP='"'"'grep -E'"'"'|g' $(TEST_DEST)/tests/atlocal
++ sed -i 's|CFLAGS=.*|CFLAGS='"'"' '"'"'|g' $(TEST_DEST)/tests/atlocal
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [scarthgap][PATCH] openvswitch: fix ptest contains reference to TMPDIR [buildpaths]
2025-03-31 20:36 [scarthgap][PATCH] openvswitch: fix ptest contains reference to TMPDIR [buildpaths] martin.jansa
@ 2025-04-02 2:23 ` Bruce Ashfield
0 siblings, 0 replies; 2+ messages in thread
From: Bruce Ashfield @ 2025-04-02 2:23 UTC (permalink / raw)
To: martin.jansa; +Cc: meta-virtualization, Bin Lan
merged.
Bruce
In message: [scarthgap][PATCH] openvswitch: fix ptest contains reference to TMPDIR [buildpaths]
on 31/03/2025 martin.jansa@gmail.com wrote:
> From: Bin Lan <bin.lan.cn@windriver.com>
>
> The EGREP in ptest/tests/atlocal contains the build paths.
> The CFLAGS in ptest/tests/atlocal contains the build paths.
> This change set fixs:
> - set EGREP to "grep -E" in ptest/tests/atlocal
> - set CFLAGS to " " in ptest/tests/atlocal
> by updating the patch
> openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch.
>
> Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
> Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com>
> ---
> ...est-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch b/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch
> index 571753a5..abad1458 100644
> --- a/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch
> +++ b/recipes-networking/openvswitch/openvswitch-git/openvswitch-add-ptest-71d553b995d0bd527d3ab1e9fbaf5a2ae34de2f3.patch
> @@ -16,6 +16,13 @@ Signed-off-by: He Zhe <zhe.he@windriver.com>
> refresh patch to fix patch-fuzz warning
> Signed-off-by: Changqing Li <changqing.li@windriver.com>
>
> +Refresh patch to fix file ptest/tests/atlocal
> +contains reference to TMPDIR [buildpaths]. The fix is:
> + - set EGREP to "grep -E" in ptest/tests/atlocal
> + - set CFLAGS to " " in ptest/tests/atlocal
> +
> +Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
> +
> Upstream-Status: Inappropriate [embedded specific]
> ---
> Makefile.am | 1 +
> @@ -37,7 +44,7 @@ new file mode 100644
> index 0000000..0b4587c
> --- /dev/null
> +++ b/test.mk
> -@@ -0,0 +1,74 @@
> +@@ -0,0 +1,76 @@
> +TEST_DEST ?= ${prefix}/lib/openvswitch
> +TEST_ROOT ?= ${prefix}/lib/openvswitch
> +TEST_DEPEND =
> @@ -112,3 +119,5 @@ index 0000000..0b4587c
> + sed -i 's|$$srcdir|$$abs_srcdir|g' $(TEST_DEST)/tests/testsuite
> + sed -i 's|ovs-appctl-bashcomp\.bash|/etc/bash_completion.d/ovs-appctl-bashcomp\.bash|g' $(TEST_DEST)/tests/testsuite
> + sed -i 's|ovs-vsctl-bashcomp\.bash|/etc/bash_completion.d/ovs-vsctl-bashcomp\.bash|g' $(TEST_DEST)/tests/testsuite
> ++ sed -i 's|EGREP=.*|EGREP='"'"'grep -E'"'"'|g' $(TEST_DEST)/tests/atlocal
> ++ sed -i 's|CFLAGS=.*|CFLAGS='"'"' '"'"'|g' $(TEST_DEST)/tests/atlocal
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-04-02 2:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-31 20:36 [scarthgap][PATCH] openvswitch: fix ptest contains reference to TMPDIR [buildpaths] martin.jansa
2025-04-02 2:23 ` Bruce Ashfield
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).