* [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1
@ 2022-06-15 16:17 Francois Perrad
2022-06-15 16:17 ` [Buildroot] [PATCH 2/5] package/open62541: more options Francois Perrad
` (4 more replies)
0 siblings, 5 replies; 10+ messages in thread
From: Francois Perrad @ 2022-06-15 16:17 UTC (permalink / raw)
To: buildroot
remove patches
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
...sts.txt-project-only-contains-C-code.patch | 27 ----------
...eLists.txt-fix-build-without-threads.patch | 37 -------------
...licit-cast-to-avoid-compiler-warning.patch | 25 ---------
...4-CMakeLists.txt-add-UA_FORCE_WERROR.patch | 54 -------------------
package/open62541/open62541.hash | 2 +-
package/open62541/open62541.mk | 2 +-
6 files changed, 2 insertions(+), 145 deletions(-)
delete mode 100644 package/open62541/0001-CMakeLists.txt-project-only-contains-C-code.patch
delete mode 100644 package/open62541/0002-arch-posix-CMakeLists.txt-fix-build-without-threads.patch
delete mode 100644 package/open62541/0003-fix-core-Explicit-cast-to-avoid-compiler-warning.patch
delete mode 100644 package/open62541/0004-CMakeLists.txt-add-UA_FORCE_WERROR.patch
diff --git a/package/open62541/0001-CMakeLists.txt-project-only-contains-C-code.patch b/package/open62541/0001-CMakeLists.txt-project-only-contains-C-code.patch
deleted file mode 100644
index 52a9cbaac..000000000
--- a/package/open62541/0001-CMakeLists.txt-project-only-contains-C-code.patch
+++ /dev/null
@@ -1,27 +0,0 @@
-From 61f9440dfeacad517965451773371aca0332d700 Mon Sep 17 00:00:00 2001
-From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
-Date: Tue, 5 Jan 2021 23:08:53 +0100
-Subject: [PATCH] CMakeLists.txt: project only contains C code
-
-Make sure CMake doesn't check for a C++ compiler when it's not needed,
-as open62541 is in C.
-
-Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
----
- CMakeLists.txt | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index 94ceb127..acc41ab2 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -1,5 +1,5 @@
- cmake_minimum_required(VERSION 3.0...3.12)
--project(open62541)
-+project(open62541 C)
- # set(CMAKE_VERBOSE_MAKEFILE ON)
- if(${CMAKE_VERSION} VERSION_LESS 3.12)
- cmake_policy(VERSION ${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION})
---
-2.29.2
-
diff --git a/package/open62541/0002-arch-posix-CMakeLists.txt-fix-build-without-threads.patch b/package/open62541/0002-arch-posix-CMakeLists.txt-fix-build-without-threads.patch
deleted file mode 100644
index bdf360804..000000000
--- a/package/open62541/0002-arch-posix-CMakeLists.txt-fix-build-without-threads.patch
+++ /dev/null
@@ -1,37 +0,0 @@
-From a312e82124fb8c9f139d5a40fa5a28123bb77a33 Mon Sep 17 00:00:00 2001
-From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-Date: Sun, 10 Jan 2021 14:25:56 +0100
-Subject: [PATCH] arch/posix/CMakeLists.txt: fix build without threads
-
-Fix the following build failure without threads:
-
-[100%] Linking C shared library bin/libopen62541.so
-/home/buildroot/autobuild/instance-3/output-1/host/opt/ext-toolchain/bin/../lib/gcc/arm-buildroot-linux-uclibcgnueabihf/8.3.0/../../../../arm-buildroot-linux-uclibcgnueabihf/bin/ld: cannot find -lpthread
-collect2: error: ld returned 1 exit status
-CMakeFiles/open62541.dir/build.make:192: recipe for target 'bin/libopen62541.so.0.0.0' failed
-make[3]: *** [bin/libopen62541.so.0.0.0] Error 1
-
-Fixes:
- - http://autobuild.buildroot.org/results/87ec3c987c991f790757276316d6caf81e635fa2
-
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
----
- arch/posix/CMakeLists.txt | 2 --
- 1 file changed, 2 deletions(-)
-
-diff --git a/arch/posix/CMakeLists.txt b/arch/posix/CMakeLists.txt
-index bb24757c..a53c9f2c 100644
---- a/arch/posix/CMakeLists.txt
-+++ b/arch/posix/CMakeLists.txt
-@@ -20,8 +20,6 @@ if (${_index} GREATER -1 OR "${UA_ARCHITECTURE}" STREQUAL "posix")
- ua_architecture_append_to_library(netdb ndblib socket)
- else()
- ua_architecture_append_to_library(m)
-- #TODO - Error on first make run if pthread is included conditional?
-- ua_architecture_append_to_library(pthread)
- if(UA_MULTITHREADING OR UA_BUILD_UNIT_TESTS)
- ua_architecture_append_to_library(pthread)
- endif()
---
-2.29.2
-
diff --git a/package/open62541/0003-fix-core-Explicit-cast-to-avoid-compiler-warning.patch b/package/open62541/0003-fix-core-Explicit-cast-to-avoid-compiler-warning.patch
deleted file mode 100644
index dea325c17..000000000
--- a/package/open62541/0003-fix-core-Explicit-cast-to-avoid-compiler-warning.patch
+++ /dev/null
@@ -1,25 +0,0 @@
-From 0c53fd776af9631c915c8722e1d68ce9a69fa3f6 Mon Sep 17 00:00:00 2001
-From: Andreas Walter <andreas.walter@waltronix.de>
-Date: Fri, 26 Feb 2021 11:49:25 +0100
-Subject: [PATCH] fix(core): Explicit cast to avoid compiler warning (#4203)
-
-[Retrieved from:
-https://github.com/open62541/open62541/commit/0c53fd776af9631c915c8722e1d68ce9a69fa3f6]
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
----
- src/ua_types_encoding_binary.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/src/ua_types_encoding_binary.c b/src/ua_types_encoding_binary.c
-index 0fe88eb98e0..abbd23735d3 100644
---- a/src/ua_types_encoding_binary.c
-+++ b/src/ua_types_encoding_binary.c
-@@ -319,7 +319,7 @@ static long double
- unpack754(uint64_t i, unsigned bits, unsigned expbits) {
- unsigned significandbits = bits - expbits - 1;
- long double result = (long double)(i&(uint64_t)((1LL<<significandbits)-1));
-- result /= (1LL<<significandbits);
-+ result /= (long double)(1LL<<significandbits);
- result += 1.0f;
- unsigned bias = (unsigned)(1<<(expbits-1)) - 1;
- long long shift = (long long)((i>>significandbits) & (uint64_t)((1LL<<expbits)-1)) - bias;
diff --git a/package/open62541/0004-CMakeLists.txt-add-UA_FORCE_WERROR.patch b/package/open62541/0004-CMakeLists.txt-add-UA_FORCE_WERROR.patch
deleted file mode 100644
index 9e6dda26e..000000000
--- a/package/open62541/0004-CMakeLists.txt-add-UA_FORCE_WERROR.patch
+++ /dev/null
@@ -1,54 +0,0 @@
-From a439daafdc98391eed13e23f93ecfca81d71c731 Mon Sep 17 00:00:00 2001
-From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-Date: Mon, 2 Aug 2021 09:49:48 +0200
-Subject: [PATCH] CMakeLists.txt: add UA_FORCE_WERROR
-
-Allow the user to disable -Werror to avoid the following build failures:
-
-/tmp/instance-0/output-1/build/open62541-v1.2.2/arch/network_tcp.c: In function 'connection_recv':
-/tmp/instance-0/output-1/build/open62541-v1.2.2/arch/network_tcp.c:96:5: error: conversion to 'unsigned int' from 'int' may change the sign of the result [-Werror=sign-conversion]
- 96 | UA_fd_set(connection->sockfd, &fdset);
- | ^~~~~~~~~
-
-/tmp/instance-6/output-1/build/open62541-v1.2.2/plugins/ua_pubsub_udp.c: In function 'UA_PubSubChannelUDPMC_receive':
-/tmp/instance-6/output-1/build/open62541-v1.2.2/plugins/ua_pubsub_udp.c:477:21: error: conversion to '__suseconds_t' {aka 'int'} from 'UA_UInt32' {aka 'unsigned int'} may change the sign of the result [-Werror=sign-conversion]
- 477 | tmptv.tv_usec = (long int)(timeout % 1000000);
- | ^
-
-Fixes:
- - http://autobuild.buildroot.org/results/911811de81d8abb2a31feb8f27af1592641c6fbc
- - http://autobuild.buildroot.org/results/f0187b3f2d62e955fddeef4e90f84ba4fd642bd2
-
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-[Upstream status: https://github.com/open62541/open62541/pull/4572]
----
- CMakeLists.txt | 6 +++++-
- 1 file changed, 5 insertions(+), 1 deletion(-)
-
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index bc839f27..963460d0 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -257,6 +257,8 @@ if(UA_FORCE_CPP)
- add_definitions(-D__STDC_CONSTANT_MACROS) # We need the UINT32_C define
- endif()
-
-+option(UA_FORCE_WERROR "Force compilation with -Werror" ON)
-+
- #General PubSub setup
- option(UA_ENABLE_PUBSUB "Enable publish/subscribe" OFF)
- mark_as_advanced(UA_ENABLE_PUBSUB)
-@@ -526,7 +528,9 @@ if(NOT UA_FORCE_CPP AND (CMAKE_COMPILER_IS_GNUCC OR "x${CMAKE_C_COMPILER_ID}" ST
- check_add_cc_flag("-Wall") # Warnings
- check_add_cc_flag("-Wextra") # More warnings
- check_add_cc_flag("-Wpedantic") # Standard compliance
-- check_add_cc_flag("-Werror") # All warnings are errors
-+ if(UA_FORCE_WERROR)
-+ check_add_cc_flag("-Werror") # All warnings are errors
-+ endif()
-
- check_add_cc_flag("-Wno-static-in-inline") # Clang doesn't like the use of static inline methods inside static inline methods
- check_add_cc_flag("-Wno-overlength-strings") # May happen in the nodeset compiler when complex values are directly encoded
---
-2.30.2
-
diff --git a/package/open62541/open62541.hash b/package/open62541/open62541.hash
index 3257bfe47..c7234fd81 100644
--- a/package/open62541/open62541.hash
+++ b/package/open62541/open62541.hash
@@ -1,3 +1,3 @@
# Locally calculated
-sha256 e2e1b8e6285c4fd6d8e07f37d61a8728376d525e169fadb3d61918fa6a7c271a open62541-v1.2.4-br1.tar.gz
+sha256 b8f4bf73d50d3dc0e2e7343f03df7cd07ad169ff66056d5c392215662b1f8fc8 open62541-v1.3.1-br1.tar.gz
sha256 fab3dd6bdab226f1c08630b1dd917e11fcb4ec5e1e020e2c16f83a0a13863e85 LICENSE
diff --git a/package/open62541/open62541.mk b/package/open62541/open62541.mk
index 87bd3449b..1958befae 100644
--- a/package/open62541/open62541.mk
+++ b/package/open62541/open62541.mk
@@ -4,7 +4,7 @@
#
################################################################################
-OPEN62541_VERSION = v1.2.4
+OPEN62541_VERSION = v1.3.1
OPEN62541_SITE_METHOD = git
OPEN62541_SITE = https://github.com/open62541/open62541.git
OPEN62541_GIT_SUBMODULES = YES
--
2.34.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* [Buildroot] [PATCH 2/5] package/open62541: more options
2022-06-15 16:17 [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Francois Perrad
@ 2022-06-15 16:17 ` Francois Perrad
2022-07-20 20:58 ` Thomas Petazzoni via buildroot
2022-06-15 16:17 ` [Buildroot] [PATCH 3/5] package/open62541: more PubSub options Francois Perrad
` (3 subsequent siblings)
4 siblings, 1 reply; 10+ messages in thread
From: Francois Perrad @ 2022-06-15 16:17 UTC (permalink / raw)
To: buildroot
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
package/open62541/Config.in | 12 ++++++++++++
package/open62541/open62541.mk | 12 ++++++++++++
2 files changed, 24 insertions(+)
diff --git a/package/open62541/Config.in b/package/open62541/Config.in
index 220fb3dfa..6045a5954 100644
--- a/package/open62541/Config.in
+++ b/package/open62541/Config.in
@@ -39,8 +39,20 @@ config BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_FULL
endchoice
+config BR2_PACKAGE_OPEN62541_HISTORIZING
+ bool "historizing"
+ help
+ Enable basic support for historical access (client and server)
+
+config BR2_PACKAGE_OPEN62541_DISCOVERY
+ bool "discovery"
+ help
+ Enable Discovery Service (LDS)
+
config BR2_PACKAGE_OPEN62541_JSON_ENCODING
bool "json encoding"
+ help
+ Enable JSON encoding
config BR2_PACKAGE_OPEN62541_PUBSUB
bool "publish/subscribe"
diff --git a/package/open62541/open62541.mk b/package/open62541/open62541.mk
index 1958befae..00e7236c0 100644
--- a/package/open62541/open62541.mk
+++ b/package/open62541/open62541.mk
@@ -31,6 +31,18 @@ else ifeq ($(BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_FULL),y)
OPEN62541_CONF_OPTS += -DUA_NAMESPACE_ZERO=FULL
endif
+ifeq ($(BR2_PACKAGE_OPEN62541_HISTORIZING),y)
+OPEN62541_CONF_OPTS += -DUA_ENABLE_HISTORIZING=ON
+else
+OPEN62541_CONF_OPTS += -DUA_ENABLE_HISTORIZING=OFF
+endif
+
+ifeq ($(BR2_PACKAGE_OPEN62541_DISCOVERY),y)
+OPEN62541_CONF_OPTS += -DUA_ENABLE_DISCOVERY=ON
+else
+OPEN62541_CONF_OPTS += -DUA_ENABLE_DISCOVERY=OFF
+endif
+
ifeq ($(BR2_PACKAGE_OPEN62541_JSON_ENCODING),y)
OPEN62541_CONF_OPTS += -DUA_ENABLE_JSON_ENCODING=ON
else
--
2.34.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* [Buildroot] [PATCH 3/5] package/open62541: more PubSub options
2022-06-15 16:17 [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Francois Perrad
2022-06-15 16:17 ` [Buildroot] [PATCH 2/5] package/open62541: more options Francois Perrad
@ 2022-06-15 16:17 ` Francois Perrad
2022-07-20 20:59 ` Thomas Petazzoni via buildroot
2022-06-15 16:17 ` [Buildroot] [PATCH 4/5] package/open62541: add encryption support Francois Perrad
` (2 subsequent siblings)
4 siblings, 1 reply; 10+ messages in thread
From: Francois Perrad @ 2022-06-15 16:17 UTC (permalink / raw)
To: buildroot
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
package/open62541/Config.in | 14 ++++++++++++++
package/open62541/open62541.mk | 12 ++++++++++++
2 files changed, 26 insertions(+)
diff --git a/package/open62541/Config.in b/package/open62541/Config.in
index 6045a5954..3f5eb78d5 100644
--- a/package/open62541/Config.in
+++ b/package/open62541/Config.in
@@ -78,6 +78,20 @@ config BR2_PACKAGE_OPEN62541_PUBSUB_INFORMATIONMODEL
Enable the information model representation of the PubSub
configuration.
+if BR2_PACKAGE_OPEN62541_PUBSUB_INFORMATIONMODEL
+
+config BR2_PACKAGE_OPEN62541_PUBSUB_INFORMATIONMODEL_METHODS
+ bool "publish/subscribe information model methods"
+ help
+ Enable PubSub informationmodel methods
+
+endif
+
+config BR2_PACKAGE_OPEN62541_PUBSUB_ETH_UADP
+ bool "publish/subscribe UADP"
+ help
+ Enable publish/subscribe UADP over Ethernet
+
endif # BR2_PACKAGE_OPEN62541_PUBSUB
endif
diff --git a/package/open62541/open62541.mk b/package/open62541/open62541.mk
index 00e7236c0..9757aa18c 100644
--- a/package/open62541/open62541.mk
+++ b/package/open62541/open62541.mk
@@ -67,6 +67,18 @@ else
OPEN62541_CONF_OPTS += -DUA_ENABLE_PUBSUB_INFORMATIONMODEL=OFF
endif
+ifeq ($(BR2_PACKAGE_OPEN62541_PUBSUB_INFORMATIONMODEL_METHODS),y)
+OPEN62541_CONF_OPTS += -DUA_ENABLE_PUBSUB_INFORMATIONMODEL_METHODS=ON
+else
+OPEN62541_CONF_OPTS += -DUA_ENABLE_PUBSUB_INFORMATIONMODEL_METHODS=OFF
+endif
+
+ifeq ($(BR2_PACKAGE_OPEN62541_PUBSUB_ETH_UADP),y)
+OPEN62541_CONF_OPTS += -DUA_ENABLE_PUBSUB_ETH_UADP=ON
+else
+OPEN62541_CONF_OPTS += -DUA_ENABLE_PUBSUB_ETH_UADP=OFF
+endif
+
# Remove unneeded files
define OPEN62541_REMOVE_UNNEEDED_FILES
$(RM) -r $(TARGET_DIR)/usr/share/open62541
--
2.34.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* [Buildroot] [PATCH 4/5] package/open62541: add encryption support
2022-06-15 16:17 [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Francois Perrad
2022-06-15 16:17 ` [Buildroot] [PATCH 2/5] package/open62541: more options Francois Perrad
2022-06-15 16:17 ` [Buildroot] [PATCH 3/5] package/open62541: more PubSub options Francois Perrad
@ 2022-06-15 16:17 ` Francois Perrad
2022-07-20 21:01 ` Thomas Petazzoni via buildroot
2022-06-15 16:17 ` [Buildroot] [PATCH 5/5] package/open62541: refactor choice UA_NAMESPACE with a proper default Francois Perrad
2022-07-20 20:58 ` [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Thomas Petazzoni via buildroot
4 siblings, 1 reply; 10+ messages in thread
From: Francois Perrad @ 2022-06-15 16:17 UTC (permalink / raw)
To: buildroot
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
package/open62541/Config.in | 19 +++++++++++++++++++
package/open62541/open62541.mk | 10 ++++++++++
2 files changed, 29 insertions(+)
diff --git a/package/open62541/Config.in b/package/open62541/Config.in
index 3f5eb78d5..ceca2ca1f 100644
--- a/package/open62541/Config.in
+++ b/package/open62541/Config.in
@@ -7,6 +7,25 @@ config BR2_PACKAGE_OPEN62541
if BR2_PACKAGE_OPEN62541
+choice
+ prompt "encryption"
+ default BR2_PACKAGE_OPEN62541_ENCRYPTION_NONE
+ help
+ Encryption support
+
+config BR2_PACKAGE_OPEN62541_ENCRYPTION_NONE
+ bool "none"
+
+config BR2_PACKAGE_OPEN62541_ENCRYPTION_MBEDTLS
+ bool "mbedtls"
+ select BR2_PACKAGE_MBEDTLS
+
+config BR2_PACKAGE_OPEN62541_ENCRYPTION_OPENSSL
+ bool "openssl"
+ select BR2_PACKAGE_LIBOPENSSL
+
+endchoice
+
choice
prompt "namespace zero nodes"
help
diff --git a/package/open62541/open62541.mk b/package/open62541/open62541.mk
index 9757aa18c..0935581b0 100644
--- a/package/open62541/open62541.mk
+++ b/package/open62541/open62541.mk
@@ -23,6 +23,16 @@ OPEN62541_CONF_OPTS = \
-DUA_ENABLE_HARDENING=OFF \
-DUA_FORCE_WERROR=OFF
+ifeq ($(BR2_PACKAGE_OPEN62541_ENCRYPTION_MBEDTLS),y)
+OPEN62541_DEPENDENCIES += mbedtls
+OPEN62541_CONF_OPTS += -DUA_ENABLE_ENCRYPTION=MBEDTLS
+endif
+
+ifeq ($(BR2_PACKAGE_OPEN62541_ENCRYPTION_OPENSSL),y)
+OPEN62541_DEPENDENCIES += libopenssl
+OPEN62541_CONF_OPTS += -DUA_ENABLE_ENCRYPTION=OPENSSL
+endif
+
ifeq ($(BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_MINIMAL),y)
OPEN62541_CONF_OPTS += -DUA_NAMESPACE_ZERO=MINIMAL
else ifeq ($(BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_REDUCED),y)
--
2.34.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 4/5] package/open62541: add encryption support
2022-06-15 16:17 ` [Buildroot] [PATCH 4/5] package/open62541: add encryption support Francois Perrad
@ 2022-07-20 21:01 ` Thomas Petazzoni via buildroot
0 siblings, 0 replies; 10+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-07-20 21:01 UTC (permalink / raw)
To: Francois Perrad; +Cc: buildroot
On Wed, 15 Jun 2022 18:17:52 +0200
Francois Perrad <fperrad@gmail.com> wrote:
> +config BR2_PACKAGE_OPEN62541_ENCRYPTION_MBEDTLS
> + bool "mbedtls"
> + select BR2_PACKAGE_MBEDTLS
> +
> +config BR2_PACKAGE_OPEN62541_ENCRYPTION_OPENSSL
> + bool "openssl"
> + select BR2_PACKAGE_LIBOPENSSL
This is not the correct way to force OpenSSL. Indeed you can't select
an option that is within a choice...endchoice. Instead, you need to:
select BR2_PACKAGE_OPENSSL
select BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL
> +ifeq ($(BR2_PACKAGE_OPEN62541_ENCRYPTION_OPENSSL),y)
> +OPEN62541_DEPENDENCIES += libopenssl
and:
OPEN62541_DEPENDENCIES += openssl
so, I fixed that up. I also explained in the commit log why we are
doing this: open62541 does not build with LibreSSL.
Applied to master with those changes. Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 5/5] package/open62541: refactor choice UA_NAMESPACE with a proper default
2022-06-15 16:17 [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Francois Perrad
` (2 preceding siblings ...)
2022-06-15 16:17 ` [Buildroot] [PATCH 4/5] package/open62541: add encryption support Francois Perrad
@ 2022-06-15 16:17 ` Francois Perrad
2022-07-20 21:02 ` Thomas Petazzoni via buildroot
2022-07-20 20:58 ` [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Thomas Petazzoni via buildroot
4 siblings, 1 reply; 10+ messages in thread
From: Francois Perrad @ 2022-06-15 16:17 UTC (permalink / raw)
To: buildroot
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
package/open62541/Config.in | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/package/open62541/Config.in b/package/open62541/Config.in
index ceca2ca1f..1dad985a7 100644
--- a/package/open62541/Config.in
+++ b/package/open62541/Config.in
@@ -28,15 +28,11 @@ endchoice
choice
prompt "namespace zero nodes"
+ default BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_REDUCED
help
Namespace zero contains the standard-defined nodes. The full
namespace zero may not be required for all applications.
-config BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_DEFAULT
- bool "default"
- help
- Use the default namespace zero contents.
-
config BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_MINIMAL
bool "minimal"
help
@@ -91,8 +87,7 @@ config BR2_PACKAGE_OPEN62541_PUBSUB_DELTAFRAMES
config BR2_PACKAGE_OPEN62541_PUBSUB_INFORMATIONMODEL
bool "publish/subscribe information model"
- depends on BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_REDUCED || \
- BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_FULL
+ depends on !BR2_PACKAGE_OPEN62541_UA_NAMESPACE_ZERO_MINIMAL
help
Enable the information model representation of the PubSub
configuration.
--
2.34.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1
2022-06-15 16:17 [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Francois Perrad
` (3 preceding siblings ...)
2022-06-15 16:17 ` [Buildroot] [PATCH 5/5] package/open62541: refactor choice UA_NAMESPACE with a proper default Francois Perrad
@ 2022-07-20 20:58 ` Thomas Petazzoni via buildroot
4 siblings, 0 replies; 10+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-07-20 20:58 UTC (permalink / raw)
To: Francois Perrad; +Cc: buildroot
On Wed, 15 Jun 2022 18:17:49 +0200
Francois Perrad <fperrad@gmail.com> wrote:
> remove patches
>
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
> ...sts.txt-project-only-contains-C-code.patch | 27 ----------
> ...eLists.txt-fix-build-without-threads.patch | 37 -------------
> ...licit-cast-to-avoid-compiler-warning.patch | 25 ---------
> ...4-CMakeLists.txt-add-UA_FORCE_WERROR.patch | 54 -------------------
> package/open62541/open62541.hash | 2 +-
> package/open62541/open62541.mk | 2 +-
> 6 files changed, 2 insertions(+), 145 deletions(-)
> delete mode 100644 package/open62541/0001-CMakeLists.txt-project-only-contains-C-code.patch
> delete mode 100644 package/open62541/0002-arch-posix-CMakeLists.txt-fix-build-without-threads.patch
> delete mode 100644 package/open62541/0003-fix-core-Explicit-cast-to-avoid-compiler-warning.patch
> delete mode 100644 package/open62541/0004-CMakeLists.txt-add-UA_FORCE_WERROR.patch
"remove patches" is really a short justification, so I have expanded
that to explain why each patch is dropped. Could you try to do that for
your next version bump with patches dropped?
Applied to master with this change!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-07-20 21:02 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-06-15 16:17 [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Francois Perrad
2022-06-15 16:17 ` [Buildroot] [PATCH 2/5] package/open62541: more options Francois Perrad
2022-07-20 20:58 ` Thomas Petazzoni via buildroot
2022-06-15 16:17 ` [Buildroot] [PATCH 3/5] package/open62541: more PubSub options Francois Perrad
2022-07-20 20:59 ` Thomas Petazzoni via buildroot
2022-06-15 16:17 ` [Buildroot] [PATCH 4/5] package/open62541: add encryption support Francois Perrad
2022-07-20 21:01 ` Thomas Petazzoni via buildroot
2022-06-15 16:17 ` [Buildroot] [PATCH 5/5] package/open62541: refactor choice UA_NAMESPACE with a proper default Francois Perrad
2022-07-20 21:02 ` Thomas Petazzoni via buildroot
2022-07-20 20:58 ` [Buildroot] [PATCH 1/5] package/open62541: bump to version 1.3.1 Thomas Petazzoni via buildroot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox