Message ID | 20170906124735.GA31394@debian |
---|---|
State | Changes Requested |
Headers | show |
Series | Add package Prosody | expand |
Hi Dushara, On Wed, Sep 06, 2017 at 10:47:35PM +1000, Dushara Jayasinghe wrote: > Ok... take 2 Not needed. Your first attempt was seen on the list. http://lists.busybox.net/pipermail/buildroot/2017-September/201770.html > This is my first post the BR mailing list so please treat it > as a request for comments. > > I've created a BR package for Prosody, which I've tested on a > raspberry pi and was able to successfully register with an IM > client. > > Thank you. Please send patches inline, not as attachments. Using 'git send-email' for that should make it easier for you. baruch
> Not needed. Your first attempt was seen on the list. Ok sorry about the noise > Please send patches inline, not as attachments. Using 'git send-email' for > that should make it easier for you. > Ok. I'll look it up. Thanks Dushara
Hello, On Wed, 6 Sep 2017 15:54:10 +0300, Baruch Siach wrote: > On Wed, Sep 06, 2017 at 10:47:35PM +1000, Dushara Jayasinghe wrote: > > Ok... take 2 > > Not needed. Your first attempt was seen on the list. > > http://lists.busybox.net/pipermail/buildroot/2017-September/201770.html > > > This is my first post the BR mailing list so please treat it > > as a request for comments. > > > > I've created a BR package for Prosody, which I've tested on a > > raspberry pi and was able to successfully register with an IM > > client. > > > > Thank you. > > Please send patches inline, not as attachments. Using 'git send-email' for > that should make it easier for you. Let me rephrase what Baruch wanted to say here: Thanks a lot for your contribution! It's great to see new people using Buildroot and contributing to it. However, in order to ease the review process, we like to receive patches inline and not as attachments. Indeed, this allows us to simply hit "reply" and comment on each part of the patch. This is a very typical review process used in the open-source community. In order to send patches inline, we recommend you to use "git send-email" as e-mail clients very often rewrap text and therefore make patches unusable. From a quick look to your package, here are a few comments: * Adding an entry to the DEVELOPERS would be needed, so that you get notified when there are build problems with your package. * Adding a prosody.hash file is also mandatory, so that Buildroot can check the integrity of the downloaded tarball. * Do you really need Lua 5.1 specifically? Other versions of Lua, or LuaJIT are not suitable ? * We don't have any BR2_PACKAGE_LIBID option in Buildroot, so this looks like a bug in your package. * You select lots of Lua packages in Config.in, but you do not depend on them in prosody.mk. You do not even depend on lua in prosody.mk, which means you have no guarantee that all those packages will be built before Prosody. You should clarify if those packages are build dependencies or runtime dependencies. * The license information should be more specific than MIT/X11. Is it MIT *or* X11, or a combination of both MIT code and X11 code ? If so, which part is under which license ? * Why do you install prosody to staging ? It doesn't seems like Prosody is a library. If that's correct, then installing to staging is not very useful. Could you look into those comments, and send an updated version of your contribution, using git send-email ? Thanks a lot! Thomas
From c163d354f4ab731b28ef0482dc66df23e5ec4e8d Mon Sep 17 00:00:00 2001 From: Dushara Jayasinghe <silent.correspondent@gmail.com> Date: Wed, 6 Sep 2017 22:31:03 +1000 Subject: [PATCH] Add package Prosody Description from website: Prosody is a modern XMPP communication server. It aims to be easy to set up and configure, and efficient with system resources. This installs the base system with certificates for two domains: localhost and example.com --- package/Config.in | 1 + package/prosody/Config.in | 19 +++++++++++++++++++ package/prosody/prosody.mk | 45 +++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 65 insertions(+) create mode 100644 package/prosody/Config.in create mode 100644 package/prosody/prosody.mk diff --git a/package/Config.in b/package/Config.in index a21c5f0..4291d97 100644 --- a/package/Config.in +++ b/package/Config.in @@ -1716,6 +1716,7 @@ endif source "package/pptp-linux/Config.in" source "package/privoxy/Config.in" source "package/proftpd/Config.in" + source "package/prosody/Config.in" source "package/proxychains-ng/Config.in" source "package/ptpd/Config.in" source "package/ptpd2/Config.in" diff --git a/package/prosody/Config.in b/package/prosody/Config.in new file mode 100644 index 0000000..a0edc0a --- /dev/null +++ b/package/prosody/Config.in @@ -0,0 +1,19 @@ +config BR2_PACKAGE_PROSODY + bool "prosody" + depends on BR2_PACKAGE_LUA_5_1 + select BR2_PACKAGE_LUAEXPAT + select BR2_PACKAGE_LUASEC + select BR2_PACKAGE_LUASOCKET + select BR2_PACKAGE_OPENSSL + select BR2_PACKAGE_LIBID + select BR2_PACKAGE_LUAFILESYSTEM + + help + Prosody is a modern XMPP communication server. It aims to be + easy to set up and configure, and efficient with system + resources. + + https://prosody.im + +comment "prosody needs the lua interpreter" + depends on !BR2_PACKAGE_LUA_5_1 diff --git a/package/prosody/prosody.mk b/package/prosody/prosody.mk new file mode 100644 index 0000000..82cbad5 --- /dev/null +++ b/package/prosody/prosody.mk @@ -0,0 +1,45 @@ +################################################################################ +# +# prosody +# +################################################################################ + +PROSODY_VERSION = 0.9.12 +PROSODY_SOURCE = prosody-$(PROSODY_VERSION).tar.gz +PROSODY_SITE = https://prosody.im/downloads/source +PROSODY_LICENSE = MIT/X11 +PROSODY_LICENSE_FILES = COPYING +PROSODY_INSTALL_STAGING = YES + +PROSODY_DEPENDENCIES = openssl + +define PROSODY_CONFIGURE_CMDS + (cd $(@D) && \ + $(TARGET_CONFIGURE_OPTS) \ + ./configure --prefix=/usr \ + --c-compiler=$(TARGET_CC) \ + --cflags="$(TARGET_CFLAGS)" \ + --linker=$(TARGET_CC) \ + --ldflags="$(TARGET_LDFLAGS) -shared" \ + --sysconfdir=/etc/prosody \ + --with-lua=$(STAGING_DIR)/usr \ + ) +endef + +define PROSODY_BUILD_CMDS + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) +endef + +define PROSODY_INSTALL_TARGET_CMDS + $(TARGET_MAKE_ENV) $(MAKE) DESTDIR="$(TARGET_DIR)" -C $(@D) install +endef + +# make install installs a Makefile and meta data to generate certs +define PROSODY_REMOVE_CERT_GENERATOR + rm -f $(TARGET_DIR)/etc/prosody/certs/Makefile + rm -f $(TARGET_DIR)/etc/prosody/certs/*.cnf +endef + +PROSODY_POST_INSTALL_TARGET_HOOKS += PROSODY_REMOVE_CERT_GENERATOR + +$(eval $(generic-package)) -- 2.1.4