[Buildroot] [PATCH v2,1/1] i2pd: new package
Romain Naour
romain.naour at smile.fr
Sun Apr 1 10:26:21 UTC 2018
Hi Fabrice,
Le 27/03/2018 à 23:24, Fabrice Fontaine a écrit :
> i2pd (I2P Daemon) is a full-featured C++ implementation of I2P
> client.
>
> I2P (Invisible Internet Protocol) is a universal anonymous
> network layer.
> All communications over I2P are anonymous and end-to-end
> encrypted, participants don't reveal their real IP addresses.
>
> http://i2pd.website
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> ---
> Changes v1 -> v2 (after review of Thomas Petazzoni):
> - Use start-stop-daemon in S99i2pd
>
> DEVELOPERS | 1 +
> package/Config.in | 1 +
> ...ix-cmake-error-when-DBUILD_SHARED_LIBS-ON.patch | 28 +++++++++
> package/i2pd/0002-Install-libi2pdclient.patch | 69 +++++++++++++++++++++
> package/i2pd/Config.in | 35 +++++++++++
> package/i2pd/S99i2pd | 38 ++++++++++++
> package/i2pd/i2pd.hash | 3 +
> package/i2pd/i2pd.mk | 71 ++++++++++++++++++++++
> package/i2pd/i2pd.service | 29 +++++++++
> 9 files changed, 275 insertions(+)
> create mode 100644 package/i2pd/0001-Fix-cmake-error-when-DBUILD_SHARED_LIBS-ON.patch
> create mode 100644 package/i2pd/0002-Install-libi2pdclient.patch
> create mode 100644 package/i2pd/Config.in
> create mode 100644 package/i2pd/S99i2pd
> create mode 100644 package/i2pd/i2pd.hash
> create mode 100644 package/i2pd/i2pd.mk
> create mode 100644 package/i2pd/i2pd.service
>
> diff --git a/DEVELOPERS b/DEVELOPERS
> index 3df820e712..364b9f9047 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -652,6 +652,7 @@ F: package/gupnp/
> F: package/gupnp-dlna/
> F: package/gupnp-tools/
> F: package/hiredis/
> +F: package/i2pd/
> F: package/igd2-for-linux/
> F: package/libmpdclient/
> F: package/libupnp/
> diff --git a/package/Config.in b/package/Config.in
> index 435f188ec5..d700c11018 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1704,6 +1704,7 @@ menu "Networking applications"
> source "package/hostapd/Config.in"
> source "package/hplip/Config.in"
> source "package/httping/Config.in"
> + source "package/i2pd/Config.in"
> source "package/ibrdtn-tools/Config.in"
> source "package/ibrdtnd/Config.in"
> source "package/ifenslave/Config.in"
> diff --git a/package/i2pd/0001-Fix-cmake-error-when-DBUILD_SHARED_LIBS-ON.patch b/package/i2pd/0001-Fix-cmake-error-when-DBUILD_SHARED_LIBS-ON.patch
> new file mode 100644
> index 0000000000..d30e62f4f3
> --- /dev/null
> +++ b/package/i2pd/0001-Fix-cmake-error-when-DBUILD_SHARED_LIBS-ON.patch
> @@ -0,0 +1,28 @@
> +From 6d15be9a32989b0d00312a5b2589e36ba0ea7836 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> +Date: Sun, 4 Mar 2018 18:29:13 +0100
> +Subject: [PATCH] Fix cmake error when -DBUILD_SHARED_LIBS=ON
> +
> +Fixes "CMake Error: TARGETS given no LIBRARY DESTINATION for shared
> +library target" by adding LIBRARY parameter to INSTALL call
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> +---
> + build/CMakeLists.txt | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/build/CMakeLists.txt b/build/CMakeLists.txt
> +index 4f875b33..a1c8957b 100644
> +--- a/build/CMakeLists.txt
> ++++ b/build/CMakeLists.txt
> +@@ -97,6 +97,7 @@ set_target_properties(libi2pd PROPERTIES PREFIX "")
> + install(TARGETS libi2pd
> + EXPORT libi2pd
> + ARCHIVE DESTINATION lib
> ++ LIBRARY DESTINATION lib
> + COMPONENT Libraries)
> + # TODO Make libi2pd available to 3rd party projects via CMake as imported target
> + # FIXME This pulls stdafx
> +--
> +2.14.1
> +
> diff --git a/package/i2pd/0002-Install-libi2pdclient.patch b/package/i2pd/0002-Install-libi2pdclient.patch
> new file mode 100644
> index 0000000000..1774977965
> --- /dev/null
> +++ b/package/i2pd/0002-Install-libi2pdclient.patch
> @@ -0,0 +1,69 @@
> +From 1d934bd543360bfdd40e1501d16d8beb2016ccd8 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> +Date: Mon, 5 Mar 2018 14:55:54 +0100
> +Subject: [PATCH] Install libi2pdclient
> +
> +When building with -DBUILD_SHARED_LIBS=ON, libi2pdclient is not
> +installed on target so install it by calling install. Moreover, rename
> +i2pdclient to libi2pdclient so library is installed with correct name.
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> +---
> + build/CMakeLists.txt | 16 +++++++++++-----
> + 1 file changed, 11 insertions(+), 5 deletions(-)
> +
> +diff --git a/build/CMakeLists.txt b/build/CMakeLists.txt
> +index a1c8957b..364c3304 100644
> +--- a/build/CMakeLists.txt
> ++++ b/build/CMakeLists.txt
> +@@ -120,7 +120,13 @@ set (CLIENT_SRC
> + if(WITH_WEBSOCKETS)
> + list (APPEND CLIENT_SRC "${LIBI2PD_CLIENT_SRC_DIR}/Websocket.cpp")
> + endif ()
> +-add_library(i2pdclient ${CLIENT_SRC})
> ++add_library(libi2pdclient ${CLIENT_SRC})
> ++set_target_properties(libi2pdclient PROPERTIES PREFIX "")
> ++install(TARGETS libi2pdclient
> ++ EXPORT libi2pdclient
> ++ ARCHIVE DESTINATION lib
> ++ LIBRARY DESTINATION lib
> ++ COMPONENT Libraries)
> +
> + set(DAEMON_SRC_DIR ../daemon)
> +
> +@@ -304,7 +310,7 @@ if (WITH_PCH)
> + WORKING_DIRECTORY ${CMAKE_BINARY_DIR}
> + )
> + target_compile_options(libi2pd PRIVATE /FIstdafx.h /Yustdafx.h /Zm155 "/Fp${CMAKE_BINARY_DIR}/stdafx.dir/$<CONFIG>/stdafx.pch")
> +- target_compile_options(i2pdclient PRIVATE /FIstdafx.h /Yustdafx.h /Zm155 "/Fp${CMAKE_BINARY_DIR}/stdafx.dir/$<CONFIG>/stdafx.pch")
> ++ target_compile_options(libi2pdclient PRIVATE /FIstdafx.h /Yustdafx.h /Zm155 "/Fp${CMAKE_BINARY_DIR}/stdafx.dir/$<CONFIG>/stdafx.pch")
> + else()
> + string(TOUPPER ${CMAKE_BUILD_TYPE} BTU)
> + get_directory_property(DEFS DEFINITIONS)
> +@@ -313,12 +319,12 @@ if (WITH_PCH)
> + COMMAND ${CMAKE_CXX_COMPILER} ${FLAGS} -c ${CMAKE_CURRENT_SOURCE_DIR}/../libi2pd/stdafx.h -o ${CMAKE_BINARY_DIR}/stdafx.h.gch
> + )
> + target_compile_options(libi2pd PRIVATE -include libi2pd/stdafx.h)
> +- target_compile_options(i2pdclient PRIVATE -include libi2pd/stdafx.h)
> ++ target_compile_options(libi2pdclient PRIVATE -include libi2pd/stdafx.h)
> + endif()
> + target_link_libraries(libi2pd stdafx)
> + endif()
> +
> +-target_link_libraries(i2pdclient libi2pd)
> ++target_link_libraries(libi2pdclient libi2pd)
> +
> + find_package ( Boost COMPONENTS system filesystem program_options date_time REQUIRED )
> + if(NOT DEFINED Boost_INCLUDE_DIRS)
> +@@ -451,7 +457,7 @@ if (WITH_BINARY)
> + if (WITH_STATIC)
> + set(DL_LIB ${CMAKE_DL_LIBS})
> + endif()
> +- target_link_libraries( "${PROJECT_NAME}" libi2pd i2pdclient ${DL_LIB} ${Boost_LIBRARIES} ${OPENSSL_LIBRARIES} ${ZLIB_LIBRARY} ${CMAKE_THREAD_LIBS_INIT} ${MINGW_EXTRA} ${DL_LIB} ${CMAKE_REQUIRED_LIBRARIES})
> ++ target_link_libraries( "${PROJECT_NAME}" libi2pd libi2pdclient ${DL_LIB} ${Boost_LIBRARIES} ${OPENSSL_LIBRARIES} ${ZLIB_LIBRARY} ${CMAKE_THREAD_LIBS_INIT} ${MINGW_EXTRA} ${DL_LIB} ${CMAKE_REQUIRED_LIBRARIES})
> +
> + install(TARGETS "${PROJECT_NAME}" RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} COMPONENT Runtime)
> + set (APPS "\${CMAKE_INSTALL_PREFIX}/bin/${PROJECT_NAME}${CMAKE_EXECUTABLE_SUFFIX}")
> +--
> +2.14.1
> +
> diff --git a/package/i2pd/Config.in b/package/i2pd/Config.in
> new file mode 100644
> index 0000000000..b1a412e75a
> --- /dev/null
> +++ b/package/i2pd/Config.in
> @@ -0,0 +1,35 @@
> +config BR2_PACKAGE_I2PD
> + bool "i2pd"
> + depends on BR2_USE_MMU # fork()
> + depends on BR2_TOOLCHAIN_HAS_ATOMIC
> + # pthread_condattr_setclock
> + depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL
> + depends on BR2_INSTALL_LIBSTDCPP
> + depends on BR2_USE_WCHAR # boost
> + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # exception_ptr
> + select BR2_PACKAGE_BOOST
> + select BR2_PACKAGE_BOOST_DATE_TIME
> + select BR2_PACKAGE_BOOST_FILESYSTEM
> + select BR2_PACKAGE_BOOST_PROGRAM_OPTIONS
> + select BR2_PACKAGE_BOOST_SYSTEM
> + select BR2_PACKAGE_OPENSSL
> + select BR2_PACKAGE_ZLIB
> + help
> + i2pd (I2P Daemon) is a full-featured C++ implementation of I2P
> + client.
> +
> + I2P (Invisible Internet Protocol) is a universal anonymous
> + network layer.
> + All communications over I2P are anonymous and end-to-end
> + encrypted, participants don't reveal their real IP addresses.
> +
> + http://i2pd.website
> +
> +comment "i2pd needs a toolchain w/ C++, NPTL, wchar"
> + depends on BR2_USE_MMU
> + depends on BR2_TOOLCHAIN_HAS_ATOMIC
> + depends on !BR2_INSTALL_LIBSTDCPP || \
> + !BR2_TOOLCHAIN_HAS_THREADS_NPTL || !BR2_USE_WCHAR
> +
> +comment "i2pd needs exception_ptr"
> + depends on BR2_TOOLCHAIN_HAS_GCC_BUG_64735
> diff --git a/package/i2pd/S99i2pd b/package/i2pd/S99i2pd
> new file mode 100644
> index 0000000000..32775cdb97
> --- /dev/null
> +++ b/package/i2pd/S99i2pd
> @@ -0,0 +1,38 @@
> +#!/bin/sh
> +
> +NAME=i2pd
> +PIDFILE=/var/run/$NAME.pid
> +DAEMON=/usr/bin/$NAME
> +DAEMON_ARGS="--conf=/etc/i2pd/i2pd.conf --tunconf=/etc/i2pd/tunnels.conf --pidfile=/var/run/i2pd.pid --logfile=/var/log/i2pd.log --daemon --service"
> +
> +start() {
> + printf "Starting $NAME: "
> + start-stop-daemon -S -q -m -b -p $PIDFILE --exec $DAEMON -- $DAEMON_ARGS
> + [ $? = 0 ] && echo "OK" || echo "FAIL"
> +}
> +stop() {
> + printf "Stopping $NAME: "
> + start-stop-daemon -K -q -p $PIDFILE
> + [ $? = 0 ] && echo "OK" || echo "FAIL"
> +}
> +restart() {
> + stop
> + start
> +}
> +
> +case "$1" in
> + start)
> + start
> + ;;
> + stop)
> + stop
> + ;;
> + restart|reload)
> + restart
> + ;;
> + *)
> + echo "Usage: $0 {start|stop|restart}"
> + exit 1
> +esac
> +
> +exit $?
> diff --git a/package/i2pd/i2pd.hash b/package/i2pd/i2pd.hash
> new file mode 100644
> index 0000000000..b7a60c9164
> --- /dev/null
> +++ b/package/i2pd/i2pd.hash
> @@ -0,0 +1,3 @@
> +# Locally computed:
> +sha256 8834190418de0c4e56c46eb8b9dc4422ef14f1ad7d6591e9daae99493bf7170e i2pd-2.18.0.tar.gz
> +sha256 d147a6acdaf8ec9f6513802ef6cad4a4afcdb5ab6b98d9f6bb26d2b7f4cf454c LICENSE
> diff --git a/package/i2pd/i2pd.mk b/package/i2pd/i2pd.mk
> new file mode 100644
> index 0000000000..11e7231551
> --- /dev/null
> +++ b/package/i2pd/i2pd.mk
> @@ -0,0 +1,71 @@
> +################################################################################
> +#
> +# i2pd
> +#
> +################################################################################
> +
> +I2PD_VERSION = 2.18.0
> +I2PD_SITE = $(call github,PurpleI2P,i2pd,$(I2PD_VERSION))
> +I2PD_LICENSE = BSD-3-Clause
> +I2PD_LICENSE_FILES = LICENSE
> +I2PD_SUBDIR = build
> +I2PD_DEPENDENCIES = \
> + boost \
> + openssl \
> + zlib
> +
> +I2PD_CONF_OPTS += -DWITH_GUI=OFF
> +
> +# The following CMake variable disables a TRY_RUN call in the -pthread
> +# test which is not allowed when cross-compiling.
> +I2PD_CONF_OPTS += -DTHREADS_PTHREAD_ARG=OFF
Weird, cmake report this variable as "used by the project":
CMake Warning:
Manually-specified variables were not used by the project:
THREADS_PTHREAD_ARG
> +
> +ifeq ($(BR2_STATIC_LIBS),y)
> +I2PD_CONF_OPTS += -DWITH_STATIC=ON
> +else
> +I2PD_CONF_OPTS += -DWITH_STATIC=OFF
> +endif
> +
> +ifeq ($(BR2_PACKAGE_LIBMINIUPNPC),y)
> +I2PD_DEPENDENCIES += libminiupnpc
> +I2PD_CONF_OPTS += -DWITH_UPNP=ON
> +else
> +I2PD_CONF_OPTS += -DWITH_UPNP=OFF
> +endif
Usually optional dependencies can be added in a follow up patch.
> +
> +define I2PD_INSTALL_CONFIGURATION_FILES
> + $(INSTALL) -D -m 644 $(@D)/contrib/i2pd.conf \
> + $(TARGET_DIR)/etc/i2pd/i2pd.conf
> + $(INSTALL) -D -m 644 $(@D)/contrib/tunnels.conf \
> + $(TARGET_DIR)/etc/i2pd/tunnels.conf
> + mkdir -p $(TARGET_DIR)/var/lib/i2pd
> + cp -a $(@D)/contrib/certificates $(TARGET_DIR)/var/lib/i2pd
> +endef
> +
> +define I2PD_REMOVE_SOURCE_FILES
> + rm -f $(TARGET_DIR)/usr/LICENSE
> + rm -rf $(TARGET_DIR)/usr/src
> +endef
It would be better to remove explicitly each files installed by i2pd. We can't
know what is in TARGET_DIR/usr/src directory.
Otherwise it looks good.
Best regards,
Romain
> +
> +I2PD_POST_INSTALL_TARGET_HOOKS += \
> + I2PD_INSTALL_CONFIGURATION_FILES \
> + I2PD_REMOVE_SOURCE_FILES
> +
> +define I2PD_USERS
> + i2pd -1 i2pd -1 * /var/lib/i2pd - - I2P Daemon
> +endef
> +
> +define I2PD_INSTALL_INIT_SYSV
> + $(INSTALL) -D -m 0755 package/i2pd/S99i2pd \
> + $(TARGET_DIR)/etc/init.d/S99i2pd
> +endef
> +
> +define I2PD_INSTALL_INIT_SYSTEMD
> + $(INSTALL) -D -m 644 package/i2pd/i2pd.service \
> + $(TARGET_DIR)/usr/lib/systemd/system/i2pd.service
> + mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
> + ln -sf ../../../../usr/lib/systemd/system/i2pd.service \
> + $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/i2pd.service
> +endef
> +
> +$(eval $(cmake-package))
> diff --git a/package/i2pd/i2pd.service b/package/i2pd/i2pd.service
> new file mode 100644
> index 0000000000..0fdee59532
> --- /dev/null
> +++ b/package/i2pd/i2pd.service
> @@ -0,0 +1,29 @@
> +[Unit]
> +Description=I2P Router written in C++
> +After=network.target
> +
> +[Service]
> +User=i2pd
> +Group=i2pd
> +RuntimeDirectory=i2pd
> +RuntimeDirectoryMode=0700
> +LogsDirectory=i2pd
> +LogsDirectoryMode=0700
> +Type=simple
> +ExecStart=/usr/bin/i2pd --conf=/etc/i2pd/i2pd.conf --tunconf=/etc/i2pd/tunnels.conf --pidfile=/run/i2pd/i2pd.pid --logfile=/var/log/i2pd/i2pd.log --daemon --service
> +ExecReload=/bin/kill -HUP $MAINPID
> +PIDFile=/run/i2pd/i2pd.pid
> +### Uncomment, if auto restart needed
> +#Restart=on-failure
> +
> +### Use SIGINT for graceful stop daemon.
> +# i2pd stops accepting new tunnels and waits ~10 min while old ones do not die.
> +KillSignal=SIGINT
> +TimeoutStopSec=10m
> +
> +# If you have problems with hanging i2pd, you can try enable this
> +#LimitNOFILE=4096
> +PrivateDevices=yes
> +
> +[Install]
> +WantedBy=multi-user.target
>
More information about the buildroot
mailing list