Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Kinsella, Ray <ray.kinsella@intel.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] radlib: new package
Date: Tue, 29 Mar 2016 21:35:37 +0000	[thread overview]
Message-ID: <1459287337.10145.3.camel@intel.com> (raw)
In-Reply-To: <20160329200702.01fc66d5@free-electrons.com>

> Please use git send-email to send your patches. Due to not using git
> send-email, your e-mail is badly line-wrapped and cannot be applied,
> and even less reviewed nicely.

ok - I will figure out the smtp setup. 

> On Mon, 28 Mar 2016 19:40:27 +0000, Kinsella, Ray wrote:
> > Delta's from v1:-
> > 
> >  * Complete rework of patch for the package's autotools
> > implementation
> >   * Removed references to prefix.
> >   * Reworked database support to use PKG_CHECK_MODULES and
> > AC_PATH_PROG.
> >   * Used '--with--somedb' to indicate which database backend to
> > build
> > with. (The radlib build ends up with conflicting symbols if more
> > than
> > one is enabled at the same time, we prevent this in the package's
> > .mk).
> >  * Complete rework of the Kconfig.
> >   * Database support is only enabled when BR2_PACKAGE_(mysql,
> > sqlite or
> > postgresql) are set.
> >   * Database backend is selected by 'choice' when more than one is
> > enabled. 
> >  * Other tidy up's recommended for the v1 patch. 
> >   * Included a comment on radlib.hash.
> >   * Added signed off line and description to the patch.
> >   * Removed no_sync from mysql.
> >   * etc.
> 
> This should go below the "---" sign.

ok the 'Delta's from v1:-' should be moved after the Signed-off-by
line?

> > radlib is a rapid application development library for unix
> > multi-process applications. It uses SYS V IPC facilities and
> > FIFOs to provide an RTOS-like, event-driven, distributed
> > framework. Processes may be run as daemons or have a controlling
> > terminal.
> > 
> > Signed-off-by: Ray Kinsella <ray.kinsella@intel.com>
> > ---
> >  package/Config.in                                  |   1 +
> >  ...0001-Update-to-autoconf-to-use-pkg-config.patch | 388
> > +++++++++++++++++++++
> 
> This is a fairly large patch. Have you submitted it upstream?

Not yet - will make contact. 

> 
> > diff --git a/package/radlib/0001-Update-to-autoconf-to-use-pkg
> > -config.patch b/package/radlib/0001-Update-to-autoconf-to-use-pkg
> > -config.patch
> > new file mode 100644
> > index 0000000..f98b9d5
> > --- /dev/null
> > +++ b/package/radlib/0001-Update-to-autoconf-to-use-pkg
> > -config.patch
> > @@ -0,0 +1,388 @@
> > +From d4c1b74a6624da7cc217c6b1aef59dc60b7d20c0 Mon Sep 17 00:00:00
> > 2001
> > +From: Ray Kinsella <ray.kinsella@intel.com>
> > +Date: Sun, 27 Mar 2016 16:38:16 +0100
> > +Subject: [PATCH] Update to autoconf to use pkg-config
> > +
> > +Updates to autoconf as follows;
> > +
> > + * Use pkg-config for postgresql and sqlite
> > + * Use mysql-config for mysql
> > + * Also removed unecessary conditionals from sub-makefiles
> 
> So you're doing three different things. Which calls for three
> different
> patches.

Hopefully this won't require me to re-org the patch. 
> 
> I'll do a full review once you send it with git send-email, because
> the
> line wrapping makes it really hard to follow.

ok - apologies. 

> Thanks!
> 
> Thomas

  reply	other threads:[~2016-03-29 21:35 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-28 19:40 [Buildroot] [PATCH v2] radlib: new package Kinsella, Ray
2016-03-29 18:07 ` Thomas Petazzoni
2016-03-29 21:35   ` Kinsella, Ray [this message]
2016-03-29 21:51     ` Thomas Petazzoni

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=1459287337.10145.3.camel@intel.com \
    --to=ray.kinsella@intel.com \
    --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