on 2011-6-19 6:47, Chris Larson wrote:
On Fri, Jun 17, 2011 at 10:42 AM, Paul Eggleton<[email protected]>  wrote:
Hi Ke,

Great work. Here's my review so far:

On Monday 13 June 2011 14:15:04 Yu, Ke wrote:
--- /dev/null
+++ b/scripts/combo-layer-hook-default.sh
@@ -0,0 +1,14 @@
+#!/bin/sh

First thought on starting to read the patch: why are we home rolling a
parser when the format is clearly ini style, which the ConfigParser
module parses just fine?

Paul, Chris,

Thanks a lot for your valuable comment. I've sent out the V2 RFC patch, and has all your comments included. Please review.

Regards
Ke

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to