This patch series looks really good to me in general. I know I've given feedback about some pieces that need tweaking but overall its good and I'm pleased with this. There are a few corner cases people have raised and some tweaks we need to make but nothing that can't be built from this foundation! :)
On Thu, 2011-05-19 at 18:29 +0800, Lianhao Lu wrote: > To start the network based PR service daemon, run the following command after > "sourcing" the environment file oe-init-build-env: > > bitbake-prserv --start The one other area I'm going to worry about is the "build out the box" use case. Nowhere in the quickstart guide do we mention to the user that they need to run this. I'd also argue that we shouldn't require the user to run this at all either and we should be able to autolaunch and use it in the localhost usecase. Most users shouldn't care, it should just work. Those who need to use it can either preconfigure it for their users, or will know how to setup and start the server themselves. A case in point is an autobuilder running 4 different builds. Ideally, in the default case, each automated build would setup and just use its own server on a non-conflicting free port on localhost. I'm not 100% sure how we make this happen and I can give it some thought but it is the final piece of this puzzle we need to solve. Cheers, Richard _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
