* [Buildroot] [PATCH v2,1/1] package/domoticz: fix build with RELRO
@ 2019-11-08 17:04 Fabrice Fontaine
2019-11-08 20:56 ` [Buildroot] [PATCH v2, 1/1] " Thomas Petazzoni
2019-11-09 17:46 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2019-11-08 17:04 UTC (permalink / raw)
To: buildroot
Fixes:
- http://autobuild.buildroot.org/results/5c1ca3083ad672401d1e050c6c3a07b8c33b851d
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
Changes v1 -> v2:
- Retrieve upstream patch that adds -c as suggested by Arnout during
review of v1
...dd-c-argument-to-build-precompiled-h.patch | 34 +++++++++++++++++++
1 file changed, 34 insertions(+)
create mode 100644 package/domoticz/0003-CMakeLists.txt-add-c-argument-to-build-precompiled-h.patch
diff --git a/package/domoticz/0003-CMakeLists.txt-add-c-argument-to-build-precompiled-h.patch b/package/domoticz/0003-CMakeLists.txt-add-c-argument-to-build-precompiled-h.patch
new file mode 100644
index 0000000000..b14f4c56c3
--- /dev/null
+++ b/package/domoticz/0003-CMakeLists.txt-add-c-argument-to-build-precompiled-h.patch
@@ -0,0 +1,34 @@
+From 17107b876fb308e3ef3e759ab90f3d8e4755cdc4 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Fri, 8 Nov 2019 14:17:58 +0100
+Subject: [PATCH] CMakeLists.txt: add -c argument to build precompiled headers
+
+Add "-c" argument when building precompiler headers as suggested
+by Arnout in https://patchwork.ozlabs.org/patch/1187328.
+
+This will fix the build with RELRO
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Retrieved from:
+https://github.com/domoticz/domoticz/commit/17107b876fb308e3ef3e759ab90f3d8e4755cdc4]
+---
+ CMakeLists.txt | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 32a0dd2a8c..010cdf2db8 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -277,10 +277,10 @@ MACRO(ADD_PRECOMPILED_HEADER _targetName _input)
+ ENDFOREACH(item)
+
+ SEPARATE_ARGUMENTS(_compiler_FLAGS)
+- MESSAGE("${CMAKE_CXX_COMPILER} ${CMAKE_CXX_COMPILER_ARG1} ${_compiler_FLAGS} -x c++-header -o ${_output} ${_source}")
++ MESSAGE("${CMAKE_CXX_COMPILER} ${CMAKE_CXX_COMPILER_ARG1} ${_compiler_FLAGS} -x c++-header -c -o ${_output} ${_source}")
+ ADD_CUSTOM_COMMAND(
+ OUTPUT ${_output}
+- COMMAND ${CMAKE_CXX_COMPILER} ${CMAKE_CXX_COMPILER_ARG1} ${_compiler_FLAGS} -x c++-header -o ${_output} ${_source}
++ COMMAND ${CMAKE_CXX_COMPILER} ${CMAKE_CXX_COMPILER_ARG1} ${_compiler_FLAGS} -x c++-header -c -o ${_output} ${_source}
+ DEPENDS ${_source} )
+ ADD_CUSTOM_TARGET(${_targetName}_gch DEPENDS ${_output})
+ ADD_DEPENDENCIES(${_targetName} ${_targetName}_gch)
--
2.23.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH v2, 1/1] package/domoticz: fix build with RELRO
2019-11-08 17:04 [Buildroot] [PATCH v2,1/1] package/domoticz: fix build with RELRO Fabrice Fontaine
@ 2019-11-08 20:56 ` Thomas Petazzoni
2019-11-09 17:46 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2019-11-08 20:56 UTC (permalink / raw)
To: buildroot
On Fri, 8 Nov 2019 18:04:45 +0100
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
> Fixes:
> - http://autobuild.buildroot.org/results/5c1ca3083ad672401d1e050c6c3a07b8c33b851d
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> Changes v1 -> v2:
> - Retrieve upstream patch that adds -c as suggested by Arnout during
> review of v1
Thanks for the additional work, this seems like a much better solution!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH v2, 1/1] package/domoticz: fix build with RELRO
2019-11-08 17:04 [Buildroot] [PATCH v2,1/1] package/domoticz: fix build with RELRO Fabrice Fontaine
2019-11-08 20:56 ` [Buildroot] [PATCH v2, 1/1] " Thomas Petazzoni
@ 2019-11-09 17:46 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2019-11-09 17:46 UTC (permalink / raw)
To: buildroot
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
> Fixes:
> - http://autobuild.buildroot.org/results/5c1ca3083ad672401d1e050c6c3a07b8c33b851d
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> Changes v1 -> v2:
> - Retrieve upstream patch that adds -c as suggested by Arnout during
> review of v1
Committed to 2019.02.x and 2019.08.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-11-09 17:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-08 17:04 [Buildroot] [PATCH v2,1/1] package/domoticz: fix build with RELRO Fabrice Fontaine
2019-11-08 20:56 ` [Buildroot] [PATCH v2, 1/1] " Thomas Petazzoni
2019-11-09 17:46 ` 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.