From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] Adding package janus-gateway and dependencies
Date: Mon, 1 Dec 2014 22:41:18 +0100 [thread overview]
Message-ID: <20141201214118.GD4150@free.fr> (raw)
In-Reply-To: <20141201223409.7cd48798@free-electrons.com>
Thomas, All,
On 2014-12-01 22:34 +0100, Thomas Petazzoni spake thusly:
> On Mon, 1 Dec 2014 21:47:22 +0100, Yann E. MORIN wrote:
> > On 2014-12-01 19:37 +0000, Gregory Dymarek spake thusly:
> > > This is a set of packages that are needed to compile janus-gateway:
> > > ding-libs
> > > gengetopt
> > > janus-gateway
> > > libnice
> > > libsrtp
> > > libwebsock
> > > sofia-sip
> > >
> > > Signed-off-by: gregd <gregd72002@gmail.com>
> >
> > I already started discussing this patch with Gregory on IRC:
> >
> > - this patch is broken because the mailer warped long lines.
> >
> > - it should have been split in different patches, one for each new
> > package
> >
> > - indentation is incorrect in the help texts (spaces vs. tabs)
> >
> > Gregory is resending this patch as-is so I can apply it and further work
> > on it.
> >
> > More comments on this patch can be done in the followup respin. ;-)
>
> While I certainly do appreciate a lot the fact that you're willing to
> take over the work of Gregory and do the final polishing, wouldn't it
> be better to teach people how to fish rather than fish for them? I.e
> make the comments, and let Gregory rework the patch series until it is
> OK for merging. Of course, this is only true if Gregory is
> willing/interested in pushing this set of patches up to the point where
> it becomes acceptable upstream.
I agree with you.
However, Gregory has done the base work, and that's nice. He already
incorporated comments I made earlier on IRC (in the past few days).
So, I was just gonna put the series in shape and fix the obvious that
does not take too much time, and respin so Gregory can take back the
work on its own.
Basically, a kind of reward for him to do the base work, and entice him
to continue on the series. ;-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2014-12-01 21:41 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-01 19:37 [Buildroot] [PATCH 1/1] Adding package janus-gateway and dependencies Gregory Dymarek
2014-12-01 20:43 ` Jérôme Pouiller
2014-12-01 20:47 ` Yann E. MORIN
2014-12-01 21:34 ` Thomas Petazzoni
2014-12-01 21:41 ` Yann E. MORIN [this message]
2014-12-01 20:56 ` Samuel Martin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20141201214118.GD4150@free.fr \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox