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
next prev parent 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.