All of lore.kernel.org
 help / color / mirror / Atom feed
From: Brian Avery <avery.brian@gmail.com>
To: Michael Wood <michael.g.wood@intel.com>
Cc: toaster@yoctoproject.org
Subject: Re: [PATCH 2/3] toaster: Update oe-core fixture for Morty plus master release
Date: Wed, 2 Nov 2016 07:47:30 -0700	[thread overview]
Message-ID: <CAKV_0ehyxS9abDD_UyB7k6Pin21rXc1trREtqGZcnOFpfOzQJQ@mail.gmail.com> (raw)
In-Reply-To: <f8bec757-9907-5abd-38b5-b75e1d554205@intel.com>

[-- Attachment #1: Type: text/plain, Size: 3637 bytes --]

whoops. ok.

On Wed, Nov 2, 2016 at 5:02 AM, Michael Wood <michael.g.wood@intel.com>
wrote:

> This commit subject line does not match the content of this patch. I think
> you've accidentally duplicated it from the first patch.

kind of. I should have made the subject more general since this was a patch
set to do both of them.

>
>
> On 31/10/16 21:22, brian avery wrote:
>
>> Update the poky fixture to include the master and Morty
>> releases.
>>
>> Signed-off-by: brian avery <brian.avery@intel.com>
>> ---
>>   lib/toaster/orm/fixtures/poky.xml | 13 +++++++++++++
>>   1 file changed, 13 insertions(+)
>>
>> diff --git a/lib/toaster/orm/fixtures/poky.xml
>> b/lib/toaster/orm/fixtures/poky.xml
>> index c192baa..8324071 100644
>> --- a/lib/toaster/orm/fixtures/poky.xml
>> +++ b/lib/toaster/orm/fixtures/poky.xml
>> @@ -7,6 +7,12 @@
>>     </object>
>>       <!-- Bitbake versions which correspond to the metadata release -->
>> +  <object model="orm.bitbakeversion" pk="3">
>> +    <field type="CharField" name="name">master</field>
>> +    <field type="CharField" name="giturl">git://git.yoctop
>> roject.org/poky</field>
>> +    <field type="CharField" name="branch">master</field>
>> +    <field type="CharField" name="dirpath">bitbake</field>
>> +  </object>
>>
>
> It's a small thing but please also order the objects in the fixtures
> sequentially (rather than 3 .. 1.. 2).


I found the out of order thing made it clearer to me that the order isn't
what determinies the default. Maybe we should order then correctly and add
a comment about settings.xml for default.

>
>
>     <object model="orm.bitbakeversion" pk="1">
>>       <field type="CharField" name="name">morty</field>
>>       <field type="CharField" name="giturl">git://git.yoctop
>> roject.org/poky</field>
>> @@ -21,6 +27,13 @@
>>     </object>
>>       <!-- Releases available -->
>> +  <object model="orm.release" pk="3">
>> +    <field type="CharField" name="name">master</field>
>> +    <field type="CharField" name="description">Yocto Project
>> master</field>
>> +    <field rel="ManyToOneRel" to="orm.bitbakeversion"
>> name="bitbake_version">3</field>
>> +    <field type="CharField" name="branch_name">morty</field>
>>
>
> Release master but you've got the branch_name set to "morty" this won't
> work!
>
> yep error.


>
> +    <field type="TextField" name="helptext">Toaster will run your builds
>> using the tip of the &lt;a href="http://git.yoctoproject.
>> org/cgit/cgit.cgi/poky/log/"&gt;Yocto Project master branch&lt;/a&gt;,
>> where active development takes place. This is not a stable branch, so your
>> builds might not work as expected.</field>
>> +  </object>
>>     <object model="orm.release" pk="1">
>>       <field type="CharField" name="name">morty</field>
>>       <field type="CharField" name="description">Yocto Project 2.2
>> "Morty"</field>
>>
>
> As mentioned on the Toaster call last week I've already had a go at doing
> these patches already. See https://bugzilla.yoctoproject.
> org/show_bug.cgi?id=10497 and the attachment. This bug is unresolved,
> when you create a new project it won't pick up the correct default layers,
> your patches also don't address this issue.

I'd forgotten about the layer issue. Sadly while this is unresolved we
cannot update the master toaster containers.
k, I'll leave it with you.
-b
an intel employee.

>
>
> Michael
>
>
> --
> _______________________________________________
> toaster mailing list
> toaster@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/toaster
>

[-- Attachment #2: Type: text/html, Size: 6367 bytes --]

  reply	other threads:[~2016-11-02 14:47 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-31 21:22 [PATCH 0/3] toaster: add master branch back in to releases brian avery
2016-10-31 21:22 ` [PATCH 1/3] toaster: Update oe-core fixture for Morty plus master release brian avery
2016-10-31 21:22 ` [PATCH 2/3] " brian avery
2016-11-02 12:02   ` Michael Wood
2016-11-02 14:47     ` Brian Avery [this message]
2016-10-31 21:22 ` [PATCH 3/3] toaster: Update default release to master brian avery

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=CAKV_0ehyxS9abDD_UyB7k6Pin21rXc1trREtqGZcnOFpfOzQJQ@mail.gmail.com \
    --to=avery.brian@gmail.com \
    --cc=michael.g.wood@intel.com \
    --cc=toaster@yoctoproject.org \
    /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.