From: Luiz Capitulino <lcapitulino@redhat.com>
To: David Turner <digit@google.com>
Cc: Bastien ROUCARIES <roucaries.bastien@gmail.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Merge qemu android
Date: Fri, 29 Jan 2010 10:22:02 -0200 [thread overview]
Message-ID: <20100129102202.7e5117d6@doriath> (raw)
In-Reply-To: <60cad3f1001281835o1da4f17bjba67fc500411b15b@mail.gmail.com>
On Thu, 28 Jan 2010 18:35:16 -0800
David Turner <digit@google.com> wrote:
> Anthony is right, and unfortunately the Android team doesn't have the
> bandwidth to support sending patches to upstream
> at the moment.
You should also consider the benefits for yourself of merging your bits
upstream: more peer review, more testing, potential improvements and
maybe better integration.
> Note that our version of QEMU is a rather complex mix of 0.8.2 and upstream.
> I routinely cherry pick upstream improvements
> and incorporate them to the codebase. However, I'm also pretty conservative
> and try to avoid stuff we don't depend on and
> which risk breaking other stuff easily so many parts don't get in, or are
> implemented differently.
Do you ever consider rebasing someday?
> Also, many, many things in the Android emulator codebase have very little
> value for upstream qemu, imho.
>
> Bastien, what specific features are you interested in ? I can still have a
> look at what would be required to generate the
> corresponding upstream patch.
>
> On Sat, Jan 23, 2010 at 2:40 PM, Anthony Liguori <anthony@codemonkey.ws>wrote:
>
> > On 01/21/2010 10:27 AM, Bastien ROUCARIES wrote:
> >
> >> Hi,
> >>
> >> What is the step in order to get qemu android merged mainline ?
> >>
> >> http://android.git.kernel.org/?p=platform/external/qemu.git;a=summary
> >>
> >>
> >
> > Send patches.
> >
> > It's very difficult to merge downstream code unless that entire downstream
> > is committed to working through upstream. I'd suggest encouraging the
> > Android developers to commit to pushing all of the functionality upstream
> > before going down this path.
> >
> > Regards,
> >
> > Anthony Liguori
> >
> > Regards
> >>
> >> Bastien
> >>
> >>
> >>
> >>
> >
> >
> >
> >
prev parent reply other threads:[~2010-01-29 12:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-21 16:27 [Qemu-devel] Merge qemu android Bastien ROUCARIES
2010-01-21 18:13 ` [Qemu-devel] " Bastien ROUCARIES
2010-01-23 22:40 ` [Qemu-devel] " Anthony Liguori
[not found] ` <195c7a901001280244i36d09907gb1e9ea387c526255@mail.gmail.com>
2010-01-28 10:44 ` Fwd: " Bastien ROUCARIES
2010-01-28 19:43 ` Laurent Desnogues
2010-01-29 2:41 ` David Turner
2010-01-29 8:51 ` [Qemu-devel] " Jan Kiszka
2010-01-29 15:51 ` [Qemu-devel] " Anthony Liguori
2010-01-29 17:08 ` Bastien ROUCARIES
2010-01-29 2:35 ` David Turner
2010-01-29 12:22 ` Luiz Capitulino [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=20100129102202.7e5117d6@doriath \
--to=lcapitulino@redhat.com \
--cc=digit@google.com \
--cc=qemu-devel@nongnu.org \
--cc=roucaries.bastien@gmail.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.