* [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++
@ 2018-12-29 22:03 james.hilliard1 at gmail.com
2018-12-29 22:14 ` Yann E. MORIN
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: james.hilliard1 at gmail.com @ 2018-12-29 22:03 UTC (permalink / raw)
To: buildroot
From: James Hilliard <james.hilliard1@gmail.com>
This is a backport from upstream.
Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
---
Changes v1 -> v2:
- Fixup patch and change commit message
---
...-use-cross-compilation-compatible-c-check.patch | 59 ++++++++++++++++++++++
1 file changed, 59 insertions(+)
create mode 100644 package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
diff --git a/package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch b/package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
new file mode 100644
index 0000000..b33e054
--- /dev/null
+++ b/package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
@@ -0,0 +1,59 @@
+From 180a940333c84495015bd5b24d41c1dbfff509f2 Mon Sep 17 00:00:00 2001
+From: James Hilliard <james.hilliard1@gmail.com>
+Date: Thu, 27 Dec 2018 09:16:20 +0800
+Subject: [PATCH] meson: use cross compilation compatible c++ check
+
+[james.hilliard1 at gmail.com: backport from upstream]
+Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
+---
+ meson.build | 4 +---
+ src/systemd/meson.build | 3 +--
+ src/test/meson.build | 2 +-
+ 3 files changed, 3 insertions(+), 6 deletions(-)
+
+diff --git a/meson.build b/meson.build
+index b338886..1964b1a 100644
+--- a/meson.build
++++ b/meson.build
+@@ -287,10 +287,8 @@ want_tests = get_option('tests')
+ slow_tests = want_tests != 'false' and get_option('slow-tests')
+ install_tests = get_option('install-tests')
+
+-cxx = find_program('c++', required : fuzzer_build)
+-if cxx.found()
++if add_languages('cpp', required : fuzzer_build)
+ # Used only for tests
+- add_languages('cpp')
+ cxx_cmd = ' '.join(meson.get_compiler('cpp').cmd_array())
+ else
+ cxx_cmd = ''
+diff --git a/src/systemd/meson.build b/src/systemd/meson.build
+index e0c967e..75c48b0 100644
+--- a/src/systemd/meson.build
++++ b/src/systemd/meson.build
+@@ -52,8 +52,7 @@ if cc.has_argument('-std=iso9899:2017')
+ opts += [['c', '-std=iso9899:2017']]
+ endif
+
+-cxx = find_program('c++', required : false)
+-if cxx.found()
++if add_languages('cpp', required : false)
+ opts += [['c++'],
+ ['c++', '-std=c++98'],
+ ['c++', '-std=c++11']]
+diff --git a/src/test/meson.build b/src/test/meson.build
+index ea049a6..d9d87e0 100644
+--- a/src/test/meson.build
++++ b/src/test/meson.build
+@@ -957,7 +957,7 @@ tests += [
+
+ ]
+
+-if cxx.found()
++if cxx_cmd != ''
+ tests += [
+ [['src/libsystemd/sd-bus/test-bus-vtable-cc.cc'],
+ [],
+--
+2.7.4
+
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++
2018-12-29 22:03 [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++ james.hilliard1 at gmail.com
@ 2018-12-29 22:14 ` Yann E. MORIN
2018-12-30 14:36 ` Thomas Petazzoni
2019-01-24 21:47 ` Peter Korsgaard
2 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2018-12-29 22:14 UTC (permalink / raw)
To: buildroot
James, All,
On 2018-12-30 06:03 +0800, james.hilliard1 at gmail.com spake thusly:
> From: James Hilliard <james.hilliard1@gmail.com>
>
> This is a backport from upstream.
>
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
Better, thanks! :-)
> ---
> Changes v1 -> v2:
> - Fixup patch and change commit message
> ---
> ...-use-cross-compilation-compatible-c-check.patch | 59 ++++++++++++++++++++++
> 1 file changed, 59 insertions(+)
> create mode 100644 package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
>
> diff --git a/package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch b/package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
> new file mode 100644
> index 0000000..b33e054
> --- /dev/null
> +++ b/package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
> @@ -0,0 +1,59 @@
> +From 180a940333c84495015bd5b24d41c1dbfff509f2 Mon Sep 17 00:00:00 2001
> +From: James Hilliard <james.hilliard1@gmail.com>
> +Date: Thu, 27 Dec 2018 09:16:20 +0800
> +Subject: [PATCH] meson: use cross compilation compatible c++ check
> +
> +[james.hilliard1 at gmail.com: backport from upstream]
Minor nit I forgot to mention: since you actually cgherry-picked it over
the v240 tag, then the sha1 above it no longer the same as the one that
was applied upstream, so it is customary to note the original upstream
sha1 in this case. Sorry for the omission. :-/
[james.hilliard1 at gmail.com:
- backport from upstream 46e63a2a3ed46ce97430ce38e4cc0798c57900eb
]
With this fixed (which can be done by a maintainer before applying, no
need to resend):
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> +Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
> +---
> + meson.build | 4 +---
> + src/systemd/meson.build | 3 +--
> + src/test/meson.build | 2 +-
> + 3 files changed, 3 insertions(+), 6 deletions(-)
> +
> +diff --git a/meson.build b/meson.build
> +index b338886..1964b1a 100644
> +--- a/meson.build
> ++++ b/meson.build
> +@@ -287,10 +287,8 @@ want_tests = get_option('tests')
> + slow_tests = want_tests != 'false' and get_option('slow-tests')
> + install_tests = get_option('install-tests')
> +
> +-cxx = find_program('c++', required : fuzzer_build)
> +-if cxx.found()
> ++if add_languages('cpp', required : fuzzer_build)
> + # Used only for tests
> +- add_languages('cpp')
> + cxx_cmd = ' '.join(meson.get_compiler('cpp').cmd_array())
> + else
> + cxx_cmd = ''
> +diff --git a/src/systemd/meson.build b/src/systemd/meson.build
> +index e0c967e..75c48b0 100644
> +--- a/src/systemd/meson.build
> ++++ b/src/systemd/meson.build
> +@@ -52,8 +52,7 @@ if cc.has_argument('-std=iso9899:2017')
> + opts += [['c', '-std=iso9899:2017']]
> + endif
> +
> +-cxx = find_program('c++', required : false)
> +-if cxx.found()
> ++if add_languages('cpp', required : false)
> + opts += [['c++'],
> + ['c++', '-std=c++98'],
> + ['c++', '-std=c++11']]
> +diff --git a/src/test/meson.build b/src/test/meson.build
> +index ea049a6..d9d87e0 100644
> +--- a/src/test/meson.build
> ++++ b/src/test/meson.build
> +@@ -957,7 +957,7 @@ tests += [
> +
> + ]
> +
> +-if cxx.found()
> ++if cxx_cmd != ''
> + tests += [
> + [['src/libsystemd/sd-bus/test-bus-vtable-cc.cc'],
> + [],
> +--
> +2.7.4
> +
> --
> 2.7.4
>
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread* [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++
2018-12-29 22:03 [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++ james.hilliard1 at gmail.com
2018-12-29 22:14 ` Yann E. MORIN
@ 2018-12-30 14:36 ` Thomas Petazzoni
2019-01-24 21:47 ` Peter Korsgaard
2 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2018-12-30 14:36 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 30 Dec 2018 06:03:14 +0800, james.hilliard1 at gmail.com wrote:
> From: James Hilliard <james.hilliard1@gmail.com>
>
> This is a backport from upstream.
>
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
>
> ---
> Changes v1 -> v2:
> - Fixup patch and change commit message
> ---
> ...-use-cross-compilation-compatible-c-check.patch | 59 ++++++++++++++++++++++
> 1 file changed, 59 insertions(+)
> create mode 100644 package/systemd/0003-meson-use-cross-compilation-compatible-c-check.patch
Applied to master after adjusting the patch description as suggested by
Yann. Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++
2018-12-29 22:03 [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++ james.hilliard1 at gmail.com
2018-12-29 22:14 ` Yann E. MORIN
2018-12-30 14:36 ` Thomas Petazzoni
@ 2019-01-24 21:47 ` Peter Korsgaard
2 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2019-01-24 21:47 UTC (permalink / raw)
To: buildroot
>>>>> "james" == james hilliard1 <james.hilliard1@gmail.com> writes:
> From: James Hilliard <james.hilliard1@gmail.com>
> This is a backport from upstream.
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
> ---
> Changes v1 -> v2:
> - Fixup patch and change commit message
Committed to 2018.11.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-01-24 21:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-29 22:03 [Buildroot] [PATCH v2 1/1] package/systemd: fix build on toolchain without C++ james.hilliard1 at gmail.com
2018-12-29 22:14 ` Yann E. MORIN
2018-12-30 14:36 ` Thomas Petazzoni
2019-01-24 21:47 ` Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox