Message ID | 20190529102706.31865-1-alexandru.ardelean@analog.com |
---|---|
State | Accepted |
Headers | show |
Series | dropbear: add build opt to provide file with localoptions.h | expand |
Hello Alexandru, Thanks for this new iteration! On Wed, 29 May 2019 13:27:06 +0300 Alexandru Ardelean <ardeleanalex@gmail.com> wrote: > From: Alexandru Ardelean <ardeleanalex@gmail.com> > > Based on discussion here: > https://patchwork.ozlabs.org/patch/1104071/ > Thomas suggested to implement an option to specify a file that gets > appended to the `localoptions.h` of dropbear, as well as providing the code > for this > > This patch implements the changes. > The patch was tested successfully with the DO_MOTD option, which the > initial patch tried to address. > > Suggested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> > Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com> > --- > Changelog v1 -> v2: > * re-implemented with suggestions from Thomas > * tested patch with `#define DO_MOTD 1` option I've applied to next, after renaming the option. To me BR2_PACKAGE_DROPBEAR_CONFIG_FILE sounded like the configuration file you would have in /etc/dropbear/ for example. I also reworked teh commit log. Thanks! Thomas
diff --git a/package/dropbear/Config.in b/package/dropbear/Config.in index 62f77bad9d..bc8d80ecf4 100644 --- a/package/dropbear/Config.in +++ b/package/dropbear/Config.in @@ -66,4 +66,11 @@ config BR2_PACKAGE_DROPBEAR_LEGACY_CRYPTO DSA public keys Diffie-Hellman Group1 key exchange +config BR2_PACKAGE_DROPBEAR_CONFIG_FILE + string "path to dropbear config file" + help + Path to a file whose contents will be appended to Dropbear + localoptions.h. It can be used to tweak the Dropbear + configuration. + endif diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk index d93faa48ea..86c7769c51 100644 --- a/package/dropbear/dropbear.mk +++ b/package/dropbear/dropbear.mk @@ -124,6 +124,14 @@ ifneq ($(BR2_PACKAGE_DROPBEAR_LASTLOG),y) DROPBEAR_CONF_OPTS += --disable-lastlog endif +DROPBEAR_CONFIG_FILE = $(call qstrip,$(BR2_PACKAGE_DROPBEAR_CONFIG_FILE)) +ifneq ($(DROPBEAR_CONFIG_FILE),) +define DROPBEAR_APPEND_CONFIG_FILE + cat $(DROPBEAR_CONFIG_FILE) >> $(@D)/localoptions.h +endef +DROPBEAR_POST_EXTRACT_HOOKS += DROPBEAR_APPEND_CONFIG_FILE +endif + define DROPBEAR_INSTALL_TARGET_CMDS $(INSTALL) -m 755 $(@D)/dropbearmulti $(TARGET_DIR)/usr/sbin/dropbear for f in $(DROPBEAR_TARGET_BINS); do \