Code review comment for lp:~tixy/linaro-image-tools/vexpress-a9-support

Revision history for this message
Tixy (Jon Medhurst) (tixy) wrote :

On Wed, 2011-11-16 at 14:29 +0000, Mattias Backman wrote:
> On Wed, Nov 16, 2011 at 3:04 PM, Tixy (Jon Medhurst)
> <email address hidden> wrote:
> > On Wed, 2011-11-16 at 13:56 +0000, Mattias Backman wrote:
> >> Thanks for adding the tests. We do also need the Android test cases. Can you have a look at TestGetBootCmdAndroid and TestGetSfdiskCmd and add a corresponding test method for the Vexpress as well?
> >
> > I will do that. What is the best way, commit another change and push
> > that out? Or modify the existing change?
>
> Just commit another revision and push to the branch. Bazaar should
> have remembered the target so you can do
> bzr push
> and if that does not work do
> bzr push lp:~tixy/etc/etc --remember
> so it will be remembered for next time.
>
> Then put a comment in the merge proposal since I don't think there's
> any notification for new revisions.

I've done that, thanks. For planning purposes, what is the likelyhood
that this change would be in the tools in time for a vexpress Android
release this month? (I'm guessing it's a bit late now.)

--
Tixy

« Back to merge proposal