All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 00/24] lua
Date: Sat, 1 Mar 2014 17:12:53 +0100	[thread overview]
Message-ID: <20140301171253.220ce620@skate> (raw)
In-Reply-To: <1393227301-14874-1-git-send-email-francois.perrad@gadz.org>

Dear Francois Perrad,

On Mon, 24 Feb 2014 08:34:36 +0100, Francois Perrad wrote:

>   lpeg: new package
>   cosmo: new package

These two applied.

>   luajson: new package
>   lualogging: new package
>   dado: new package
>   lrandom: new package
>   lua-testmore: new package
>   lua-coat: new package
>   lua-coatpersistent: new package
>   lua-messagepack: new package
>   lua-csnappy: new package
>   lzlib: new package
>   lunit: new package
>   lpty: new package
>   lsqlite3: new package

All these applied.

If I'm correct the patches that remain to be applied are:

	[11/24] lua-iconv: new package

This one has questions from Arnout.

	[08/24] ljlinenoise: new package

Small comments from Arnout.

	[07/24] luarocks: allow to work with host-luajit

Comments from Arnout.

	[06/24] luajit: add host-luajit

Ditto.

	[05/24] luajit: refactor without shared-lib.patch

Some comments from Samuel, apparently you wanted to add another static
linking related patch to the series.

	[04/24] luajit: refactor without dont-strip.patch

Comments from Arnout.

Would you mind resending another series with those patches, taking into
account the comments?

Also, may I insist yet another time on the fact that your commit logs
are very often way too short. For anything that isn't a simple package
addition or bump, a good rule of thumb is that your commit log must
have at least 2 or 3 paragraphs detailing what's going on. If you don't
have 2 or 3 paragraphs for each of your commit logs, please re-read
them again and ask yourself whether the patch can easily be understood
by reviewers. If not, your commit log should be improved.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

      parent reply	other threads:[~2014-03-01 16:12 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-24  7:34 [Buildroot] [PATCH 00/24] lua Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 01/24] lua: remove LUA_INSTALLED_FILES Francois Perrad
2014-02-24 19:44   ` Arnout Vandecappelle
2014-02-24  7:34 ` [Buildroot] [PATCH 02/24] lua: refactor install steps Francois Perrad
2014-02-25 17:39   ` Arnout Vandecappelle
2014-02-25 20:30     ` Arnout Vandecappelle
2014-02-25 21:07     ` François Perrad
2014-02-25 21:38       ` Arnout Vandecappelle
2014-02-24  7:34 ` [Buildroot] [PATCH 03/24] luarocks: never wraps script Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 04/24] luajit: refactor without dont-strip.patch Francois Perrad
2014-02-27  6:39   ` Arnout Vandecappelle
2014-02-24  7:34 ` [Buildroot] [PATCH 05/24] luajit: refactor without shared-lib.patch Francois Perrad
2014-02-27  6:40   ` Arnout Vandecappelle
2014-02-27  9:05   ` Samuel Martin
2014-02-28  8:22     ` François Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 06/24] luajit: add host-luajit Francois Perrad
2014-02-27  6:44   ` Arnout Vandecappelle
2014-02-24  7:34 ` [Buildroot] [PATCH 07/24] luarocks: allow to work with host-luajit Francois Perrad
2014-02-27  7:02   ` Arnout Vandecappelle
2014-02-24  7:34 ` [Buildroot] [PATCH 08/24] ljlinenoise: new package Francois Perrad
2014-02-27  7:04   ` Arnout Vandecappelle
2014-02-24  7:34 ` [Buildroot] [PATCH 09/24] lpeg: " Francois Perrad
2014-02-27  7:08   ` Arnout Vandecappelle
2014-03-01 14:40   ` Thomas Petazzoni
2014-02-24  7:34 ` [Buildroot] [PATCH 10/24] cosmo: " Francois Perrad
2014-02-27  7:08   ` Arnout Vandecappelle
2014-03-01 14:42   ` Thomas Petazzoni
2014-02-24  7:34 ` [Buildroot] [PATCH 11/24] lua-iconv: " Francois Perrad
2014-02-27  7:16   ` Arnout Vandecappelle
2014-03-01 15:58     ` François Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 12/24] luajson: " Francois Perrad
2014-02-27  7:18   ` Arnout Vandecappelle
2014-03-01 15:06   ` Thomas Petazzoni
2014-02-24  7:34 ` [Buildroot] [PATCH 13/24] lualogging: " Francois Perrad
2014-03-01 15:10   ` Thomas Petazzoni
2014-02-24  7:34 ` [Buildroot] [PATCH 14/24] dado: " Francois Perrad
2014-03-01 15:12   ` Thomas Petazzoni
2014-02-24  7:34 ` [Buildroot] [PATCH 15/24] lrandom: " Francois Perrad
2014-03-01 15:15   ` Thomas Petazzoni
2014-02-24  7:34 ` [Buildroot] [PATCH 16/24] lua-testmore: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 17/24] lua-coat: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 18/24] lua-coatpersistent: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 19/24] lua-messagepack: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 20/24] lua-csnappy: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 21/24] lzlib: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 22/24] lunit: " Francois Perrad
2014-02-24  7:34 ` [Buildroot] [PATCH 23/24] lpty: " Francois Perrad
2014-02-24  7:35 ` [Buildroot] [PATCH 24/24] lsqlite3: " Francois Perrad
2014-03-01 15:09 ` [Buildroot] [PATCH 00/24] lua Thomas Petazzoni
2014-03-01 15:34   ` François Perrad
2014-03-01 15:42     ` Thomas Petazzoni
2014-03-02  7:57       ` François Perrad
2014-03-02  8:39         ` Thomas Petazzoni
2014-03-01 16:12 ` Thomas Petazzoni [this message]

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=20140301171253.220ce620@skate \
    --to=thomas.petazzoni@free-electrons.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 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.