* [Buildroot] Add package Prosody
@ 2017-09-06 12:36 nidujay
0 siblings, 0 replies; 5+ messages in thread
From: nidujay @ 2017-09-06 12:36 UTC (permalink / raw
To: buildroot
Hi all,
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.
Dushara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20170906/b7c2e80e/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-package-Prosody.patch
Type: text/x-patch
Size: 3314 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20170906/b7c2e80e/attachment.bin>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] Add package Prosody
@ 2017-09-06 12:47 Dushara Jayasinghe
2017-09-06 12:54 ` Baruch Siach
0 siblings, 1 reply; 5+ messages in thread
From: Dushara Jayasinghe @ 2017-09-06 12:47 UTC (permalink / raw
To: buildroot
Ok... take 2
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.
Dushara
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-package-Prosody.patch
Type: text/x-diff
Size: 3315 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20170906/2fcc306b/attachment.patch>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] Add package Prosody
2017-09-06 12:47 [Buildroot] Add package Prosody Dushara Jayasinghe
@ 2017-09-06 12:54 ` Baruch Siach
2017-09-06 13:02 ` Dushara Jayasinghe
2017-09-06 19:57 ` Thomas Petazzoni
0 siblings, 2 replies; 5+ messages in thread
From: Baruch Siach @ 2017-09-06 12:54 UTC (permalink / raw
To: buildroot
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
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] Add package Prosody
2017-09-06 12:54 ` Baruch Siach
@ 2017-09-06 13:02 ` Dushara Jayasinghe
2017-09-06 19:57 ` Thomas Petazzoni
1 sibling, 0 replies; 5+ messages in thread
From: Dushara Jayasinghe @ 2017-09-06 13:02 UTC (permalink / raw
To: buildroot
> 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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] Add package Prosody
2017-09-06 12:54 ` Baruch Siach
2017-09-06 13:02 ` Dushara Jayasinghe
@ 2017-09-06 19:57 ` Thomas Petazzoni
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2017-09-06 19:57 UTC (permalink / raw
To: buildroot
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
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-09-06 19:57 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-06 12:47 [Buildroot] Add package Prosody Dushara Jayasinghe
2017-09-06 12:54 ` Baruch Siach
2017-09-06 13:02 ` Dushara Jayasinghe
2017-09-06 19:57 ` Thomas Petazzoni
-- strict thread matches above, loose matches on Subject: below --
2017-09-06 12:36 nidujay
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.