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? -- Christopher Larson clarson at kergoth dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
