All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] package/domoticz: drop first patch
@ 2021-04-18 20:07 Fabrice Fontaine
  2021-04-18 20:07 ` [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency Fabrice Fontaine
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Fabrice Fontaine @ 2021-04-18 20:07 UTC (permalink / raw)
  To: buildroot

Patch not needed since commit 37f197f8634352750f169b6a287588a09b82e00e
which bumped host-cmake dependency from 3.10 to 3.15

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 ...nd-placeholders-in-global-namespace.patch} |  0
 ...CMake-configuration-file-to-use-vers.patch | 55 -------------------
 ... 0002-Build-failure-with-python-3-9.patch} |  0
 ...=> 0003-Load-python-3-9-shared-libs.patch} |  0
 4 files changed, 55 deletions(-)
 rename package/domoticz/{0002-Avoid-deprecated-bind-placeholders-in-global-namespace.patch => 0001-Avoid-deprecated-bind-placeholders-in-global-namespace.patch} (100%)
 delete mode 100644 package/domoticz/0001-Revert-Updating-CMake-configuration-file-to-use-vers.patch
 rename package/domoticz/{0003-Build-failure-with-python-3-9.patch => 0002-Build-failure-with-python-3-9.patch} (100%)
 rename package/domoticz/{0004-Load-python-3-9-shared-libs.patch => 0003-Load-python-3-9-shared-libs.patch} (100%)

diff --git a/package/domoticz/0002-Avoid-deprecated-bind-placeholders-in-global-namespace.patch b/package/domoticz/0001-Avoid-deprecated-bind-placeholders-in-global-namespace.patch
similarity index 100%
rename from package/domoticz/0002-Avoid-deprecated-bind-placeholders-in-global-namespace.patch
rename to package/domoticz/0001-Avoid-deprecated-bind-placeholders-in-global-namespace.patch
diff --git a/package/domoticz/0001-Revert-Updating-CMake-configuration-file-to-use-vers.patch b/package/domoticz/0001-Revert-Updating-CMake-configuration-file-to-use-vers.patch
deleted file mode 100644
index 5393b0caf0..0000000000
--- a/package/domoticz/0001-Revert-Updating-CMake-configuration-file-to-use-vers.patch
+++ /dev/null
@@ -1,55 +0,0 @@
-From 2cd95289939b4a7c2576176c7f6268028aa093b2 Mon Sep 17 00:00:00 2001
-From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-Date: Fri, 24 May 2019 21:50:16 +0200
-Subject: [PATCH] Revert "Updating CMake configuration file to use
- version 3.14 as minimum and added boost dependency targets"
-
-This reverts commit 4e4ee999c4b323514c0ba81ab1ff3afb10d2f1cd to avoid
-bumping host-cmake requirement version from 3.8 to 3.14 in buildroot.
-
-[Fabrice: Updated for 2020.1]
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
----
- CMakeLists.txt | 8 +++-----
- 1 file changed, 3 insertions(+), 5 deletions(-)
-
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index c5b7832ff..a21f5b87b 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -1,5 +1,5 @@
--#set to minimum version that supports clean build
--cmake_minimum_required(VERSION 3.14.0)
-+#set to minimum version that supports clean build on cygwin
-+cmake_minimum_required(VERSION 2.8.4)
- 
- project(domoticz)
- 
-@@ -583,6 +583,7 @@ set(Boost_USE_STATIC_LIBS ${USE_STATIC_BOOST})
- set(Boost_USE_MULTITHREADED ON)
- unset(Boost_INCLUDE_DIR CACHE)
- unset(Boost_LIBRARY_DIRS CACHE)
-+find_package(Boost REQUIRED COMPONENTS thread system)
- 
- if(USE_STATIC_BOOST)
-    message(STATUS "Linking against boost static libraries")
-@@ -590,16 +591,6 @@ else(USE_STATIC_BOOST)
-    message(STATUS "Linking against boost dynamic libraries")
- endif(USE_STATIC_BOOST)
- 
--find_package(Boost REQUIRED COMPONENTS thread system)
--if(Boost_FOUND)
--    MESSAGE(STATUS "BOOST libraries found at: ${Boost_LIBRARY_DIRS}")
--    MESSAGE(STATUS "Boost includes found at: ${Boost_INCLUDE_DIR}")
--else(Boost_FOUND)
--    MESSAGE(FATAL_ERROR "Boost thread/system library not found on your system, try to get this installed.")
--endif(Boost_FOUND)
--
--target_link_libraries(domoticz Boost::thread Boost::system)
--
- # compare found vs required libBoost version
- if(Boost_VERSION VERSION_LESS DOMO_MIN_LIBBOOST_VERSION)
-    message(FATAL_ERROR "Found libBoost version ${Boost_VERSION}, ${DOMO_MIN_LIBBOOST_VERSION} or newer required")
--- 
-2.20.1
-
diff --git a/package/domoticz/0003-Build-failure-with-python-3-9.patch b/package/domoticz/0002-Build-failure-with-python-3-9.patch
similarity index 100%
rename from package/domoticz/0003-Build-failure-with-python-3-9.patch
rename to package/domoticz/0002-Build-failure-with-python-3-9.patch
diff --git a/package/domoticz/0004-Load-python-3-9-shared-libs.patch b/package/domoticz/0003-Load-python-3-9-shared-libs.patch
similarity index 100%
rename from package/domoticz/0004-Load-python-3-9-shared-libs.patch
rename to package/domoticz/0003-Load-python-3-9-shared-libs.patch
-- 
2.30.2

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

* [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency
  2021-04-18 20:07 [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Fabrice Fontaine
@ 2021-04-18 20:07 ` Fabrice Fontaine
  2021-04-19 20:34   ` Thomas Petazzoni
  2021-04-26  9:51   ` Peter Korsgaard
  2021-04-19 20:34 ` [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Thomas Petazzoni
  2021-04-26  9:51 ` Peter Korsgaard
  2 siblings, 2 replies; 8+ messages in thread
From: Fabrice Fontaine @ 2021-04-18 20:07 UTC (permalink / raw)
  To: buildroot

boost date-time is not a dependency since version 4.9700 and
https://github.com/domoticz/domoticz/commit/a3eacbc987b59dd4aa73ada24a0a6e9a0d27b740

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/domoticz/Config.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/package/domoticz/Config.in b/package/domoticz/Config.in
index 88de232642..4d1a7f9289 100644
--- a/package/domoticz/Config.in
+++ b/package/domoticz/Config.in
@@ -11,7 +11,6 @@ config BR2_PACKAGE_DOMOTICZ
 	depends on BR2_TOOLCHAIN_SUPPORTS_ALWAYS_LOCKFREE_ATOMIC_INTS # boost-thread
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # exception_ptr
 	select BR2_PACKAGE_BOOST
-	select BR2_PACKAGE_BOOST_DATE_TIME
 	select BR2_PACKAGE_BOOST_SYSTEM
 	select BR2_PACKAGE_BOOST_THREAD
 	select BR2_PACKAGE_JSONCPP
-- 
2.30.2

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

* [Buildroot] [PATCH 1/2] package/domoticz: drop first patch
  2021-04-18 20:07 [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Fabrice Fontaine
  2021-04-18 20:07 ` [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency Fabrice Fontaine
@ 2021-04-19 20:34 ` Thomas Petazzoni
  2021-04-26  9:51 ` Peter Korsgaard
  2 siblings, 0 replies; 8+ messages in thread
From: Thomas Petazzoni @ 2021-04-19 20:34 UTC (permalink / raw)
  To: buildroot

On Sun, 18 Apr 2021 22:07:29 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Patch not needed since commit 37f197f8634352750f169b6a287588a09b82e00e
> which bumped host-cmake dependency from 3.10 to 3.15
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  ...nd-placeholders-in-global-namespace.patch} |  0
>  ...CMake-configuration-file-to-use-vers.patch | 55 -------------------
>  ... 0002-Build-failure-with-python-3-9.patch} |  0
>  ...=> 0003-Load-python-3-9-shared-libs.patch} |  0  
>  4 files changed, 55 deletions(-)
>  rename package/domoticz/{0002-Avoid-deprecated-bind-placeholders-in-global-namespace.patch => 0001-Avoid-deprecated-bind-placeholders-in-global-namespace.patch} (100%)
>  delete mode 100644 package/domoticz/0001-Revert-Updating-CMake-configuration-file-to-use-vers.patch
>  rename package/domoticz/{0003-Build-failure-with-python-3-9.patch => 0002-Build-failure-with-python-3-9.patch} (100%)
>  rename package/domoticz/{0004-Load-python-3-9-shared-libs.patch => 0003-Load-python-3-9-shared-libs.patch} (100%)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency
  2021-04-18 20:07 ` [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency Fabrice Fontaine
@ 2021-04-19 20:34   ` Thomas Petazzoni
  2021-04-19 20:44     ` Fabrice Fontaine
  2021-04-26  9:51   ` Peter Korsgaard
  1 sibling, 1 reply; 8+ messages in thread
From: Thomas Petazzoni @ 2021-04-19 20:34 UTC (permalink / raw)
  To: buildroot

On Sun, 18 Apr 2021 22:07:30 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> boost date-time is not a dependency since version 4.9700 and
> https://github.com/domoticz/domoticz/commit/a3eacbc987b59dd4aa73ada24a0a6e9a0d27b740
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/domoticz/Config.in | 1 -
>  1 file changed, 1 deletion(-)

Applied to master, thanks. I'm curious: how do you notice this kind of
change?

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency
  2021-04-19 20:34   ` Thomas Petazzoni
@ 2021-04-19 20:44     ` Fabrice Fontaine
  2021-04-19 21:42       ` Thomas Petazzoni
  0 siblings, 1 reply; 8+ messages in thread
From: Fabrice Fontaine @ 2021-04-19 20:44 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

Le lun. 19 avr. 2021 ? 22:34, Thomas Petazzoni
<thomas.petazzoni@bootlin.com> a ?crit :
>
> On Sun, 18 Apr 2021 22:07:30 +0200
> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
>
> > boost date-time is not a dependency since version 4.9700 and
> > https://github.com/domoticz/domoticz/commit/a3eacbc987b59dd4aa73ada24a0a6e9a0d27b740
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > ---
> >  package/domoticz/Config.in | 1 -
> >  1 file changed, 1 deletion(-)
>
> Applied to master, thanks. I'm curious: how do you notice this kind of
> change?
I was working on bumping domoticz to latest version and I noticed this
change when going through CMakeLists.txt.
Unfortunately, I should have noticed it sooner (back in 2018) but this
kind of changes can easily be missed.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
Best Regards,

Fabrice

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

* [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency
  2021-04-19 20:44     ` Fabrice Fontaine
@ 2021-04-19 21:42       ` Thomas Petazzoni
  0 siblings, 0 replies; 8+ messages in thread
From: Thomas Petazzoni @ 2021-04-19 21:42 UTC (permalink / raw)
  To: buildroot

On Mon, 19 Apr 2021 22:44:39 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> I was working on bumping domoticz to latest version and I noticed this
> change when going through CMakeLists.txt.
> Unfortunately, I should have noticed it sooner (back in 2018) but this
> kind of changes can easily be missed.

Absolutely: it's very easy to spot a missing dependency, as it
typically causes a build failure. But spotting dependencies that are no
longer necessary is much, much trickier, which is why I asked how you
found out this one.

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH 1/2] package/domoticz: drop first patch
  2021-04-18 20:07 [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Fabrice Fontaine
  2021-04-18 20:07 ` [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency Fabrice Fontaine
  2021-04-19 20:34 ` [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Thomas Petazzoni
@ 2021-04-26  9:51 ` Peter Korsgaard
  2 siblings, 0 replies; 8+ messages in thread
From: Peter Korsgaard @ 2021-04-26  9:51 UTC (permalink / raw)
  To: buildroot

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

 > Patch not needed since commit 37f197f8634352750f169b6a287588a09b82e00e
 > which bumped host-cmake dependency from 3.10 to 3.15

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

Committed to 2021.02.x, thanks.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency
  2021-04-18 20:07 ` [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency Fabrice Fontaine
  2021-04-19 20:34   ` Thomas Petazzoni
@ 2021-04-26  9:51   ` Peter Korsgaard
  1 sibling, 0 replies; 8+ messages in thread
From: Peter Korsgaard @ 2021-04-26  9:51 UTC (permalink / raw)
  To: buildroot

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

 > boost date-time is not a dependency since version 4.9700 and
 > https://github.com/domoticz/domoticz/commit/a3eacbc987b59dd4aa73ada24a0a6e9a0d27b740

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

Committed to 2021.02.x, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2021-04-26  9:51 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-18 20:07 [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Fabrice Fontaine
2021-04-18 20:07 ` [Buildroot] [PATCH 2/2] package/domoticz: drop boost date-time dependency Fabrice Fontaine
2021-04-19 20:34   ` Thomas Petazzoni
2021-04-19 20:44     ` Fabrice Fontaine
2021-04-19 21:42       ` Thomas Petazzoni
2021-04-26  9:51   ` Peter Korsgaard
2021-04-19 20:34 ` [Buildroot] [PATCH 1/2] package/domoticz: drop first patch Thomas Petazzoni
2021-04-26  9:51 ` 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.