From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: "Marinescu, Bogdan A" <bogdan.a.marinescu@intel.com>
Cc: bitbake-devel@lists.openembedded.org
Subject: Re: RFC: bugfix for 3575
Date: Wed, 23 Jan 2013 15:14:48 +0000 [thread overview]
Message-ID: <1358954088.6356.61.camel@ted> (raw)
In-Reply-To: <CAMZjud1sM6e2Rd2tWsBjt_XJ7N9J6AZo5OJg+R6q47AXwdg_3A@mail.gmail.com>
On Wed, 2013-01-23 at 12:08 +0200, Marinescu, Bogdan A wrote:
> Please review my fix of bug 3575 (Ensure windows hob client
> communication with bitbake backend via xmlrpc). I'd like to get
> feedback before uploading the patches. I've pushed my fix on
> poky-contrib:
>http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=bogdanm/b3575
>
> Currently, the fix allows bitbake and hob to run on different machines
> and "talk" to each other via XMLRPC (no windows version yet, but it
> can be tested in Linux). Hopefully the commit messages will tell the
> rest of the story.
First two commits are no problem.
Third Commit:
I don't like the merging of all the extra cache data. I think when a
given UI registers, it needs to call into cooker and inform bitbake it
requires the extra caches (if it does so). With your current code, we
might as well throw away the extra cache code as it effectively works as
the greatest set. I see this only happens in server mode but I'm still
not 100% convinced its the right thing to do.
I also don't see why knotty can't work in server mode?
Fourth Commit:
I have a strong opinion that we want to support multiple clients
connected to one server. I appreciate this complicates things and
currently would be chaotic. I think the easiest way to deal with this is
to have some notion of a single "controller" which would perhaps be
represented by the token. We'd then have some kind of handoff process so
that a given UI could claim the token and then control the server. We
might be able to lose the need for the manual reclaim option too if we
can tie this to the registered handlers somehow?
Any thoughts on that approach?
Cheers,
Richard
next prev parent reply other threads:[~2013-01-23 15:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-23 10:08 RFC: bugfix for 3575 Marinescu, Bogdan A
2013-01-23 15:14 ` Richard Purdie [this message]
2013-01-24 9:14 ` Marinescu, Bogdan A
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=1358954088.6356.61.camel@ted \
--to=richard.purdie@linuxfoundation.org \
--cc=bitbake-devel@lists.openembedded.org \
--cc=bogdan.a.marinescu@intel.com \
/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.