All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/libcgi: disable tests
@ 2022-05-09 20:46 Fabrice Fontaine
  2022-05-13 21:56 ` Arnout Vandecappelle
  2022-05-28 19:23 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2022-05-09 20:46 UTC (permalink / raw)
  To: buildroot; +Cc: Fabrice Fontaine

Disable tests to avoid the following static build failure on sh4aeb:

In file included from /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:20:
/home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/include/libcgi/cgi.h:81:31: note: expected 'char *' but argument is of type 'const char *'
   81 | extern int slist_delete(char *name, formvars **start, formvars **last);
      |                         ~~~~~~^~~~
/home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:352:1: error: unable to find a register to spill in class 'FPUL_REGS'
  352 | }
      | ^
/home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:352:1: error: this is the insn:
(insn 1084 1081 1085 105 (set (reg:SI 4 r4)
        (mem/f:SI (post_inc:SI (reg:SI 76 fr12 [orig:343 ivtmp.286 ] [343])) [0 MEM[base: _414, offset: 0B]+0 S4 A32])) "/home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c":323:3 189 {movsi_ie}
     (expr_list:REG_INC (reg:SI 76 fr12 [orig:343 ivtmp.286 ] [343])
        (nil)))

Fixes:
 - http://autobuild.buildroot.org/results/84a2339568d23b328af2416bfcec1ef41eccdce5

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 ...-CMakeLists.txt-honour-BUILD_TESTING.patch | 44 +++++++++++++++++++
 1 file changed, 44 insertions(+)
 create mode 100644 package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch

diff --git a/package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch b/package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch
new file mode 100644
index 0000000000..3ec674ae70
--- /dev/null
+++ b/package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch
@@ -0,0 +1,44 @@
+From 15ec267520efbe45193eb1df5361a4ab56164294 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Sun, 8 May 2022 17:54:42 +0200
+Subject: [PATCH] CMakeLists.txt: honour BUILD_TESTING
+
+Allow the user to disable tests through the standard BUILD_TESTING
+option: https://cmake.org/cmake/help/latest/module/CTest.html
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Retrieved from:
+https://github.com/rafaelsteil/libcgi/commit/15ec267520efbe45193eb1df5361a4ab56164294]
+---
+ CMakeLists.txt | 12 +++++++++---
+ 1 file changed, 9 insertions(+), 3 deletions(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index cbf0d97..e3329a1 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -29,15 +29,21 @@ option(BUILD_SHARED_LIBS
+ 	"Global flag to cause add_library to create shared libraries if on."
+ 	ON
+ )
++option(BUILD_TESTING
++	"Build tests."
++	ON
++)
+ 
+ # subdirectories
+ add_subdirectory("include/libcgi")
+ add_subdirectory("src")
+ 
+ # test
+-enable_testing()
+-include(CTest)
+-add_subdirectory("test")
++if(BUILD_TESTING)
++	enable_testing()
++	include(CTest)
++	add_subdirectory("test")
++endif(BUILD_TESTING)
+ 
+ # cmake package stuff
+ configure_package_config_file(${PROJECT_NAME_LC}-config.cmake.in
-- 
2.35.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/libcgi: disable tests
  2022-05-09 20:46 [Buildroot] [PATCH 1/1] package/libcgi: disable tests Fabrice Fontaine
@ 2022-05-13 21:56 ` Arnout Vandecappelle
  2022-05-28 19:23 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2022-05-13 21:56 UTC (permalink / raw)
  To: Fabrice Fontaine, buildroot



On 09/05/2022 22:46, Fabrice Fontaine wrote:
> Disable tests to avoid the following static build failure on sh4aeb:
> 
> In file included from /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:20:
> /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/include/libcgi/cgi.h:81:31: note: expected 'char *' but argument is of type 'const char *'
>     81 | extern int slist_delete(char *name, formvars **start, formvars **last);
>        |                         ~~~~~~^~~~
> /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:352:1: error: unable to find a register to spill in class 'FPUL_REGS'
>    352 | }
>        | ^
> /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:352:1: error: this is the insn:
> (insn 1084 1081 1085 105 (set (reg:SI 4 r4)
>          (mem/f:SI (post_inc:SI (reg:SI 76 fr12 [orig:343 ivtmp.286 ] [343])) [0 MEM[base: _414, offset: 0B]+0 S4 A32])) "/home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c":323:3 189 {movsi_ie}
>       (expr_list:REG_INC (reg:SI 76 fr12 [orig:343 ivtmp.286 ] [343])
>          (nil)))
> 
> Fixes:
>   - http://autobuild.buildroot.org/results/84a2339568d23b328af2416bfcec1ef41eccdce5
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

  Applied to master, thanks.

  Regards,
  Arnout

> ---
>   ...-CMakeLists.txt-honour-BUILD_TESTING.patch | 44 +++++++++++++++++++
>   1 file changed, 44 insertions(+)
>   create mode 100644 package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch
> 
> diff --git a/package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch b/package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch
> new file mode 100644
> index 0000000000..3ec674ae70
> --- /dev/null
> +++ b/package/libcgi/0001-CMakeLists.txt-honour-BUILD_TESTING.patch
> @@ -0,0 +1,44 @@
> +From 15ec267520efbe45193eb1df5361a4ab56164294 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Date: Sun, 8 May 2022 17:54:42 +0200
> +Subject: [PATCH] CMakeLists.txt: honour BUILD_TESTING
> +
> +Allow the user to disable tests through the standard BUILD_TESTING
> +option: https://cmake.org/cmake/help/latest/module/CTest.html
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +[Retrieved from:
> +https://github.com/rafaelsteil/libcgi/commit/15ec267520efbe45193eb1df5361a4ab56164294]
> +---
> + CMakeLists.txt | 12 +++++++++---
> + 1 file changed, 9 insertions(+), 3 deletions(-)
> +
> +diff --git a/CMakeLists.txt b/CMakeLists.txt
> +index cbf0d97..e3329a1 100644
> +--- a/CMakeLists.txt
> ++++ b/CMakeLists.txt
> +@@ -29,15 +29,21 @@ option(BUILD_SHARED_LIBS
> + 	"Global flag to cause add_library to create shared libraries if on."
> + 	ON
> + )
> ++option(BUILD_TESTING
> ++	"Build tests."
> ++	ON
> ++)
> +
> + # subdirectories
> + add_subdirectory("include/libcgi")
> + add_subdirectory("src")
> +
> + # test
> +-enable_testing()
> +-include(CTest)
> +-add_subdirectory("test")
> ++if(BUILD_TESTING)
> ++	enable_testing()
> ++	include(CTest)
> ++	add_subdirectory("test")
> ++endif(BUILD_TESTING)
> +
> + # cmake package stuff
> + configure_package_config_file(${PROJECT_NAME_LC}-config.cmake.in
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/libcgi: disable tests
  2022-05-09 20:46 [Buildroot] [PATCH 1/1] package/libcgi: disable tests Fabrice Fontaine
  2022-05-13 21:56 ` Arnout Vandecappelle
@ 2022-05-28 19:23 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2022-05-28 19:23 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: buildroot

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Disable tests to avoid the following static build failure on sh4aeb:
 > In file included from
 > /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:20:
 > /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/include/libcgi/cgi.h:81:31:
 > note: expected 'char *' but argument is of type 'const char *'
 >    81 | extern int slist_delete(char *name, formvars **start, formvars **last);
 >       |                         ~~~~~~^~~~
 > /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:352:1:
 > error: unable to find a register to spill in class 'FPUL_REGS'
 >   352 | }
 >       | ^
 > /home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c:352:1: error: this is the insn:
 > (insn 1084 1081 1085 105 (set (reg:SI 4 r4)
 >         (mem/f:SI (post_inc:SI (reg:SI 76 fr12 [orig:343 ivtmp.286 ] [343])) [0 MEM[base: _414, offset: 0B]+0 S4 A32])) "/home/autobuild/autobuild/instance-5/output-1/build/libcgi-1.3.0/test/test_slist.c":323:3 189 {movsi_ie}
 >      (expr_list:REG_INC (reg:SI 76 fr12 [orig:343 ivtmp.286 ] [343])
 >         (nil)))

 > Fixes:
 >  - http://autobuild.buildroot.org/results/84a2339568d23b328af2416bfcec1ef41eccdce5

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2022.02.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-05-28 19:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-09 20:46 [Buildroot] [PATCH 1/1] package/libcgi: disable tests Fabrice Fontaine
2022-05-13 21:56 ` Arnout Vandecappelle
2022-05-28 19:23 ` Peter Korsgaard

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.