All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Karicheri, Muralidharan" <m-karicheri2@ti.com>
Cc: "Rini, Tom" <trini@ti.com>,
	"meta-ti@yoctoproject.org" <meta-ti@yoctoproject.org>
Subject: Re: [RESEND: PATCH 2/3] keystone2: u-boot: update to add gph support
Date: Tue, 7 May 2013 19:06:34 -0400	[thread overview]
Message-ID: <20130507230634.GL12982@edge> (raw)
In-Reply-To: <3E54258959B69E4282D79E01AB1F32B7044ED22D@DFLE11.ent.ti.com>

On Tue, May 07, 2013 at 06:51:00PM -0400, Karicheri, Muralidharan wrote:
> >> > >> See below about pointing SRC_URI to an internal git server.
> >> > >>
> >> > >>
> >> > >> >  BRANCH = "master"
> >> > >> >
> >> > >> > -# DEV.MCSDK-03.00.00.07
> >> > >> > -SRCREV = "82f40e857d853165310d0753e79235aefb65d7ba"
> >> > >>
> >> > >> Please use the format above - it avoid unnecessary breakages when
> >> > >> git-ls-remote is not working...
> >> >
> >> > Adding a commit id means every time we push something to master, I
> >> > need to go and change this. This is not practical. Better point to the
> >> > tip for getting the latest for nightly. What am I missing?
> >> 
> >> The problem is that meta-ti is public and constantly used by many people outside of TI -
> >> every time they parse the recipe, it will hit the server. If you point to arago-project.org, it
> >> may bring it down, as we have seen before.
> >> If you point it to your internal git server, that is not accessible from outside of TI, it will
> >> break parsing for everyone and render meta-ti unusable!
> >> 
> >> See my next comment for details on how to do it in meta-arago instead.
> >> 
> >> 
> >> > >> > +# for nightly switch the two below #SRCREV =
> >> > >> > +"DEV.MCSDK-2013-01.10"
> >> > >> > +SRCREV = "${AUTOREV}"
> >> > >>
> >> > >> Same as before - please no AUTOREVs in meta-ti.
> >> > >>
> >> > >> If you need to track daily/nightly progress from an internal git
> >> > >> server, please use a .bbappend in meta-arago, while keeping meta-ti
> >> > >> pointing to a piblic git server and using a specific commit ID. Let
> >> > >> me know if you need existing examples - we just released Core
> >> > >> TI-SDK 2013.04.00 which used to track the latest linux-ti-staging kernel by setting
> >> AUTOREV in meta-arago.
> >> > >>
> >> >
> >> > I don't know. Please give me an example or help me figure this out.
> >> 
> >> Use something as simple as below, but for the kernel:
> >> http://arago-project.org/git/?p=meta-
> >> arago.git;a=commitdiff;h=32ec5282fd0248581d57ed2a0d66d4c47c275e8c
> 
> Hao,
> 
> Can you make an update for this in meta-arago so that our nightly points to 
> tip of the master or whatever branch?
> 
> Denys,
> 
> So for each of the repos, do we have to use a xxx.bbappend, where xxx is 
> u-boot, boot-monitor and linux-keystone? So I assume the build still uses 
> the meta-ti version of the recipe, but override the SRCREV with whatever set 
> in xxx.bbappend?

Murali,

Correct, .bbappend is the way to "append" changes to the original recipe w/o 
modifying it.

I don't know what you mean by "each of the repos", but yes, you would need to 
place a .bbappend in meta-arago-distro for every component you develop 
actively and for which you don't want to constantly update the recipe with the 
new SRCREV (or BRANCH or SRC_URI etc.)

By keeping meta-ti pointing to a public git tree with a specific SRCREV 
pointing preferably to a release tag or a stable commit ID you are making sure 
that it works for everybody. It is an official TI BSP and is distribution 
agnostic and should be parseable and buildable by anyone out there. On the 
other hand, meta-arago-distro is our own TI distribution and you have more 
freedom there.

-- 
Denys


  reply	other threads:[~2013-05-07 23:06 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-07 21:24 [RESEND: PATCH 1/3] keystone2: boot-monitor: update to pull code from tip of the master Murali Karicheri
2013-05-07 21:24 ` [RESEND: PATCH 2/3] keystone2: u-boot: update to add gph support Murali Karicheri
2013-05-07 21:52   ` Denys Dmytriyenko
2013-05-07 22:07     ` Karicheri, Muralidharan
2013-05-07 22:40       ` Denys Dmytriyenko
2013-05-07 22:51         ` Karicheri, Muralidharan
2013-05-07 23:06           ` Denys Dmytriyenko [this message]
2013-05-08 14:39             ` Karicheri, Muralidharan
2013-05-08 15:11         ` Tom Rini
2013-05-08 15:32           ` Carlos Hernandez
2013-05-08 15:55           ` Karicheri, Muralidharan
2013-05-08 16:44             ` Tom Rini
2013-05-07 21:24 ` [RESEND: PATCH 3/3] keystone2: linux: add support for building 3.8.4 based Linux kernel Murali Karicheri
2013-05-07 21:59   ` Denys Dmytriyenko
2013-05-07 21:38 ` [RESEND: PATCH 1/3] keystone2: boot-monitor: update to pull code from tip of the master Denys Dmytriyenko
2013-05-08  8:27 ` Maxin B. John
2013-05-08 12:42   ` Dmytriyenko, Denys

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=20130507230634.GL12982@edge \
    --to=denys@ti.com \
    --cc=m-karicheri2@ti.com \
    --cc=meta-ti@yoctoproject.org \
    --cc=trini@ti.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.