Re: [oe] [meta-networking][rocko][PATCH v3] civetweb: Add new recipe with git-based version

2018-05-25 Thread Krzysztof Kozlowski
On Thu, May 24, 2018 at 4:24 PM, akuster808  wrote:
>
>
> On 05/24/2018 02:03 AM, Krzysztof Kozlowski wrote:
>> Add recipe for CivetWeb: easy to use, powerful, C/C++ embeddable web
>> server with optional CGI, SSL and Lua support, under MIT license.
>>
>> CivetWeb provides C and C++ libraries and a standalone server binary.
>>
>> Choose specific post-v1.10 git revision b8148afe8fa4 which has a fixed
>> SSL support (depending on OpenSSL v1.0.2).
>>
>> Signed-off-by: Krzysztof Kozlowski 
>
> Adding new packages to a stable branch is not allowed.  Also I don't see
> it in master.

Thanks, I'll rebase and resend.

Best regards,
Krzysztof
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-networking][rocko][PATCH v3] civetweb: Add new recipe with git-based version

2018-05-24 Thread akuster808


On 05/24/2018 02:03 AM, Krzysztof Kozlowski wrote:
> Add recipe for CivetWeb: easy to use, powerful, C/C++ embeddable web
> server with optional CGI, SSL and Lua support, under MIT license.
>
> CivetWeb provides C and C++ libraries and a standalone server binary.
>
> Choose specific post-v1.10 git revision b8148afe8fa4 which has a fixed
> SSL support (depending on OpenSSL v1.0.2).
>
> Signed-off-by: Krzysztof Kozlowski 

Adding new packages to a stable branch is not allowed.  Also I don't see
it in master.

- Armin 
>
> ---
>
> Changes since v2:
> 1. Bump to post-v1.10 git revision.
> 2. Add ssl package config.
> 3. Drop cgi, cpp and debug from default package config.
>
> Changes since v1:
> 1. Use SRCREC as suggested by Khem Raj.
> 2. Drop nativesdk class.
> 3. Fix native build with patch.
> 4. Explicitly disable SSL (does not build at v1.10).
> 5. Disable ASAN (not applicable for native builds).
> ---
>  ...ssing-librt-and-libm-during-native-compil.patch | 42 +
>  .../recipes-connectivity/civetweb/civetweb_git.bb  | 43 
> ++
>  2 files changed, 85 insertions(+)
>  create mode 100644 
> meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
>  create mode 100644 
> meta-networking/recipes-connectivity/civetweb/civetweb_git.bb
>
> diff --git 
> a/meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
>  
> b/meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
> new file mode 100644
> index ..401cf39ad8df
> --- /dev/null
> +++ 
> b/meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
> @@ -0,0 +1,42 @@
> +From cd93e2c538cecb3192fd24c0fbd6e834d0c47eec Mon Sep 17 00:00:00 2001
> +From: Krzysztof Kozlowski 
> +Date: Wed, 18 Apr 2018 14:11:52 +0200
> +Subject: [PATCH] Unittest: Fix missing librt and libm during native 
> compilation
> +
> +civetweb looks for librt and libm in weird places and for native build
> +this fails:
> +
> +| ../third_party/lib/libcheck.a(check.c.o): In function `tcase_create':
> +| 
> build/tmp/work/x86_64-linux/civetweb-native/1.10-r0/build/third_party/src/check-unit-test-framework/src/check.c:145:
>  undefined reference to `floor'
> +| 
> build/tmp/work/x86_64-linux/civetweb-native/1.10-r0/build/third_party/src/check-unit-test-framework/src/check.c:148:
>  undefined reference to `floor'
> +
> +These are needed dependencies so try regular system linkage in such
> +case.
> +
> +Signed-off-by: Krzysztof Kozlowski 
> +---
> + unittest/CMakeLists.txt | 4 
> + 1 file changed, 4 insertions(+)
> +
> +diff --git a/unittest/CMakeLists.txt b/unittest/CMakeLists.txt
> +index f34b2e0e4ce4..05d25702178c 100644
> +--- a/unittest/CMakeLists.txt
>  b/unittest/CMakeLists.txt
> +@@ -60,10 +60,14 @@ endif()
> + find_package(LibM)
> + if (LIBM_FOUND)
> +   set(CHECK_LIBRARIES "${CHECK_LIBRARIES};LIBM::LIBM")
> ++else()
> ++  set(CHECK_LIBRARIES "${CHECK_LIBRARIES};-lm")
> + endif()
> + find_package(LibRt)
> + if (LIBRT_FOUND)
> +   set(CHECK_LIBRARIES "${CHECK_LIBRARIES};LIBRT::LIBRT")
> ++else()
> ++  set(CHECK_LIBRARIES "${CHECK_LIBRARIES};-lrt")
> + endif()
> + 
> + # Build the C unit tests
> +-- 
> +2.7.4
> +
> diff --git a/meta-networking/recipes-connectivity/civetweb/civetweb_git.bb 
> b/meta-networking/recipes-connectivity/civetweb/civetweb_git.bb
> new file mode 100644
> index ..ffe7282b73c9
> --- /dev/null
> +++ b/meta-networking/recipes-connectivity/civetweb/civetweb_git.bb
> @@ -0,0 +1,43 @@
> +SUMMARY = "Civetweb embedded web server"
> +HOMEPAGE = "https://github.com/civetweb/civetweb";
> +
> +LICENSE = "MIT"
> +LIC_FILES_CHKSUM = "file://LICENSE.md;md5=6f28fdcba0dda735eed62bac6a397562"
> +
> +SRCREV = "b8148afe8fa44c64f75e69655c4fdc9095565568"
> +PV = "1.10+git${SRCPV}"
> +SRC_URI = "git://github.com/civetweb/civetweb.git"
> +SRC_URI_append_class-native = " \
> +file://0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch \
> +"
> +S = "${WORKDIR}/git"
> +
> +# civetweb supports building with make or cmake (although cmake lacks few 
> features)
> +inherit cmake
> +
> +# Disable Lua and Duktape because they do not compile from CMake (as of 
> v1.8, v1.9 and v1.10).
> +# Disable ASAN as it is included only in Debug build.
> +EXTRA_OECMAKE = " \
> +-DBUILD_SHARED_LIBS=ON \
> +-DCIVETWEB_ENABLE_DUKTAPE=OFF \
> +-DCIVETWEB_ENABLE_LUA=OFF \
> +-DCIVETWEB_ENABLE_ASAN=OFF \
> +"
> +EXTRA_OECMAKE_class-native = " \
> +-DBUILD_SHARED_LIBS=ON \
> +-DCIVETWEB_ENABLE_DUKTAPE=OFF \
> +-DCIVETWEB_ENABLE_LUA=OFF \
> +-DCIVETWEB_ENABLE_ASAN=OFF \
> +"
> +
> +PACKAGECONFIG ??= "caching ipv6 server ssl websockets"
> +PACKAGECONFIG[caching] = 
> "-DCIVETWEB_DISABLE_CACHING=OFF,-DCIVETWEB_DISABLE_CACHING=ON,"
> +PACKAGECONFIG[c

[oe] [meta-networking][rocko][PATCH v3] civetweb: Add new recipe with git-based version

2018-05-24 Thread Krzysztof Kozlowski
Add recipe for CivetWeb: easy to use, powerful, C/C++ embeddable web
server with optional CGI, SSL and Lua support, under MIT license.

CivetWeb provides C and C++ libraries and a standalone server binary.

Choose specific post-v1.10 git revision b8148afe8fa4 which has a fixed
SSL support (depending on OpenSSL v1.0.2).

Signed-off-by: Krzysztof Kozlowski 

---

Changes since v2:
1. Bump to post-v1.10 git revision.
2. Add ssl package config.
3. Drop cgi, cpp and debug from default package config.

Changes since v1:
1. Use SRCREC as suggested by Khem Raj.
2. Drop nativesdk class.
3. Fix native build with patch.
4. Explicitly disable SSL (does not build at v1.10).
5. Disable ASAN (not applicable for native builds).
---
 ...ssing-librt-and-libm-during-native-compil.patch | 42 +
 .../recipes-connectivity/civetweb/civetweb_git.bb  | 43 ++
 2 files changed, 85 insertions(+)
 create mode 100644 
meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
 create mode 100644 
meta-networking/recipes-connectivity/civetweb/civetweb_git.bb

diff --git 
a/meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
 
b/meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
new file mode 100644
index ..401cf39ad8df
--- /dev/null
+++ 
b/meta-networking/recipes-connectivity/civetweb/civetweb/0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch
@@ -0,0 +1,42 @@
+From cd93e2c538cecb3192fd24c0fbd6e834d0c47eec Mon Sep 17 00:00:00 2001
+From: Krzysztof Kozlowski 
+Date: Wed, 18 Apr 2018 14:11:52 +0200
+Subject: [PATCH] Unittest: Fix missing librt and libm during native compilation
+
+civetweb looks for librt and libm in weird places and for native build
+this fails:
+
+| ../third_party/lib/libcheck.a(check.c.o): In function `tcase_create':
+| 
build/tmp/work/x86_64-linux/civetweb-native/1.10-r0/build/third_party/src/check-unit-test-framework/src/check.c:145:
 undefined reference to `floor'
+| 
build/tmp/work/x86_64-linux/civetweb-native/1.10-r0/build/third_party/src/check-unit-test-framework/src/check.c:148:
 undefined reference to `floor'
+
+These are needed dependencies so try regular system linkage in such
+case.
+
+Signed-off-by: Krzysztof Kozlowski 
+---
+ unittest/CMakeLists.txt | 4 
+ 1 file changed, 4 insertions(+)
+
+diff --git a/unittest/CMakeLists.txt b/unittest/CMakeLists.txt
+index f34b2e0e4ce4..05d25702178c 100644
+--- a/unittest/CMakeLists.txt
 b/unittest/CMakeLists.txt
+@@ -60,10 +60,14 @@ endif()
+ find_package(LibM)
+ if (LIBM_FOUND)
+   set(CHECK_LIBRARIES "${CHECK_LIBRARIES};LIBM::LIBM")
++else()
++  set(CHECK_LIBRARIES "${CHECK_LIBRARIES};-lm")
+ endif()
+ find_package(LibRt)
+ if (LIBRT_FOUND)
+   set(CHECK_LIBRARIES "${CHECK_LIBRARIES};LIBRT::LIBRT")
++else()
++  set(CHECK_LIBRARIES "${CHECK_LIBRARIES};-lrt")
+ endif()
+ 
+ # Build the C unit tests
+-- 
+2.7.4
+
diff --git a/meta-networking/recipes-connectivity/civetweb/civetweb_git.bb 
b/meta-networking/recipes-connectivity/civetweb/civetweb_git.bb
new file mode 100644
index ..ffe7282b73c9
--- /dev/null
+++ b/meta-networking/recipes-connectivity/civetweb/civetweb_git.bb
@@ -0,0 +1,43 @@
+SUMMARY = "Civetweb embedded web server"
+HOMEPAGE = "https://github.com/civetweb/civetweb";
+
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://LICENSE.md;md5=6f28fdcba0dda735eed62bac6a397562"
+
+SRCREV = "b8148afe8fa44c64f75e69655c4fdc9095565568"
+PV = "1.10+git${SRCPV}"
+SRC_URI = "git://github.com/civetweb/civetweb.git"
+SRC_URI_append_class-native = " \
+file://0001-Test-Fix-missing-librt-and-libm-during-native-compil.patch \
+"
+S = "${WORKDIR}/git"
+
+# civetweb supports building with make or cmake (although cmake lacks few 
features)
+inherit cmake
+
+# Disable Lua and Duktape because they do not compile from CMake (as of v1.8, 
v1.9 and v1.10).
+# Disable ASAN as it is included only in Debug build.
+EXTRA_OECMAKE = " \
+-DBUILD_SHARED_LIBS=ON \
+-DCIVETWEB_ENABLE_DUKTAPE=OFF \
+-DCIVETWEB_ENABLE_LUA=OFF \
+-DCIVETWEB_ENABLE_ASAN=OFF \
+"
+EXTRA_OECMAKE_class-native = " \
+-DBUILD_SHARED_LIBS=ON \
+-DCIVETWEB_ENABLE_DUKTAPE=OFF \
+-DCIVETWEB_ENABLE_LUA=OFF \
+-DCIVETWEB_ENABLE_ASAN=OFF \
+"
+
+PACKAGECONFIG ??= "caching ipv6 server ssl websockets"
+PACKAGECONFIG[caching] = 
"-DCIVETWEB_DISABLE_CACHING=OFF,-DCIVETWEB_DISABLE_CACHING=ON,"
+PACKAGECONFIG[cgi] = "-DCIVETWEB_DISABLE_CGI=OFF,-DCIVETWEB_DISABLE_CGI=ON,"
+PACKAGECONFIG[cpp] = "-DCIVETWEB_ENABLE_CXX=ON,-DCIVETWEB_ENABLE_CXX=OFF,"
+PACKAGECONFIG[debug] = 
"-DCIVETWEB_ENABLE_MEMORY_DEBUGGING=ON,-DCIVETWEB_ENABLE_MEMORY_DEBUGGING=OFF,"
+PACKAGECONFIG[ipv6] = "-DCIVETWEB_ENABLE_IPV6=ON,-DCIVETWEB_ENABLE_IPV6=OFF,"
+PACKAGECONFIG[server] = 
"-DCIVETWEB_INSTALL_EXECUTABLE=ON,-DCIVETWEB_INSTALL_EXECUTABLE=OFF,