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

Revision history for this message
Mattias Backman (mabac) 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.

>
> --
> Tixy
>
>
> https://code.launchpad.net/~tixy/linaro-image-tools/vexpress-a9-support/+merge/82372
> You are requested to review the proposed merge of lp:~tixy/linaro-image-tools/vexpress-a9-support into lp:linaro-image-tools.
>

« Back to merge proposal