All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Urbanec <openembedded-devel@urbanec.net>
To: Andreas Oberritter <obi@opendreambox.org>,
	"Burton, Ross" <ross.burton@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] gstreamer1.0*git: Update to build current AUTOREV from master branch.
Date: Fri, 27 Feb 2015 23:27:06 +1100	[thread overview]
Message-ID: <54F0629A.9080700@urbanec.net> (raw)
In-Reply-To: <54F03A47.8020208@opendreambox.org>

On 27/02/15 20:35, Andreas Oberritter wrote:
> as far as I remember, OE-Core policy forbids AUTOREV in recipes, because
> it would make network access mandatory.

Fair enough. If that is the policy, it would probably be appropriate to 
move the gstreamer1.01-*_git.bb recipes out of oe-core or at least fix 
them so that it is possible to use ${AUTOREV} from another layer.

> The syntax for enabling it from local.conf is:
> SRCREV_pn-gstreamer1.0 = "${AUTOREV}"

This only works for the main gstreamer1.0 recipe. It fails with infinite 
recursion on any of the gstreamer1.0-plugins-*.bb recipes as I mentioned 
before. This is because the plugins use a python method for SRC_URI 
which references PV. This blows up when you try to use SRCREV="${AUTOREV}"

I guess the question is what next? Should I just drop this and leave the 
oe-core gstreamer1.0*git.bb recipes as they are (referencing git commits 
from circa 2013) or should I put a bit more effort into cleaning up the 
issues with infinite recursion when used with AUTOREV? I suspect that 
the cleanup will just involve removing the current code and inserting a 
bunch of conditionals that translate "gstreamer1.0-" to "gst-"

In my opinion the current version of the recipes is of limited value. I 
understand that there has recently been some activity on implementing 
something that could deliver the functionality of gitpkgv at the oe-core 
level. Should I perhaps just wait and see how that pans out? 
gstreamer1.0*git.bb could be a good test case. ;-)

Cheers and beers,

	Peter



  reply	other threads:[~2015-02-27 12:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-26 17:57 [PATCH] gstreamer1.0*git: Update to build current AUTOREV from master branch Peter Urbanec
2015-02-26 21:45 ` Richard Purdie
2015-02-27  2:52   ` Peter Urbanec
2015-02-26 21:52 ` Burton, Ross
2015-02-27  3:06   ` Peter Urbanec
2015-02-27  9:35     ` Andreas Oberritter
2015-02-27 12:27       ` Peter Urbanec [this message]
2015-02-27 13:58         ` Martin Jansa

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=54F0629A.9080700@urbanec.net \
    --to=openembedded-devel@urbanec.net \
    --cc=obi@opendreambox.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.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.