l-source bbclass will only create
one branch per override.
Signed-off-by: Olof Johansson
Reviewed-by: Peter Kjellerstedt
---
meta/classes/devtool-source.bbclass | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
---
v1 -> v2: Peter Kjellerstedt pointed out some issues with v1; this
l-source bbclass will only create
one branch per override.
Change-Id: I8adca0b54179793ca92478ad5b3b1b6e0448e26b
Signed-off-by: Olof Johansson
Reviewed-by: Peter Kjellerstedt
---
meta/classes/devtool-source.bbclass | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --
On 18-10-19 18:00 +0200, Olof Johansson wrote:
> For conditonally applied patches based on SRC_URI overrides, the
> devtool-source class would try to create a new branch for each override
> assignment as a postfunc to do_patch, but if the same override was used
> multiple times, it
l-source bbclass will only create
one branch per override.
Signed-off-by: Olof Johansson
---
meta/classes/devtool-source.bbclass | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
---
Peter Kjellerstedt pointed out some issues with v1; this updated change is
simpler. In addition to
On 18-10-11 10:34 +0100, Richard Purdie wrote:
> This breaks the oe-selftest devtool tests:
>
> https://autobuilder.yoctoproject.org/typhoon/api/v2/logs/46751/raw
>
> oe-selftest -r devtool
>
> should reproduce.
Thanks! Was under the impression that the DEVTOOL_EXTRA_OVERRIDES
was initialized t
On 18-10-10 17:48 +0200, Peter Kjellerstedt wrote:
> > Change-Id: I8adca0b54179793ca92478ad5b3b1b6e0448e26b
>
> You probably want to remove the Change-Id before applying this...
Sorry! :( I'll probably forget to remove that again in the
future, I should try making git format-patch automatically s
l-source bbclass will only create
one branch per override.
Change-Id: I8adca0b54179793ca92478ad5b3b1b6e0448e26b
Signed-off-by: Olof Johansson
---
meta/classes/devtool-source.bbclass | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/meta/classes/devtool-source.bbcla
The string.replace function is removed in python3. Instead, the str
method "replace" should be used instead.
Signed-off-by: Olof Johansson
---
meta/classes/spdx.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/classes/spdx.bbclass b/meta/classes/sp
In python3, passing a unicode object to hashlib will result in an
exception that encourages you to encode it first.
Signed-off-by: Olof Johansson
---
meta/classes/spdx.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/classes/spdx.bbclass b/meta/classes
The path variable is used in an error message a few lines later, but was
never defined.
Signed-off-by: Olof Johansson
---
meta/classes/spdx.bbclass | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/classes/spdx.bbclass b/meta/classes/spdx.bbclass
index ab2eaa5c0c1
The same functionality already exists within bitbake, so avoid
duplicating.
Signed-off-by: Olof Johansson
---
meta/classes/spdx.bbclass | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/meta/classes/spdx.bbclass b/meta/classes/spdx.bbclass
index 9e374d70a6c
This patch series contain small adaptions, primarily for the spdx bbclasss to
work with Python3. But with that said, I was unable to get it working when it
came to interfacing with Fossology.
Olof Johansson (4):
spdx.bbclass: Replace deprecated string.replace with str.replace
spdx.bbclass
On 18-06-30 08:37 +0200, Alexander Kanavin wrote:
> Ideally, "/" should've been replaced with rpmtsRootDir(ts), but ts is
> not accessible from the function, or from its parent, or parent
> parent. So yes, I concede this would be too invasive to pass that down
> the call stack. So if you can amend
g out of available inodes).
Normally, the temporary files would have been written to the tmp
directory of the target sysroot (which we can easily clean up), but in
this tree, you can't necessarily run the scriptlets.
Fixes [YOCTO #12792]
Signed-off-by: Olof Johansson
---
...installi
On 18-06-26 12:33 +0200, Alexander Kanavin wrote:
> I believe the culprit is likely this patch:
> http://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/meta/recipes-devtools/rpm/files/0001-When-cross-installing-execute-package-scriptlets-wit.patch
>
> As scriptlets are executed outside of rpm's chro
On 18-06-25 15:25 +0200, Alexander Kanavin wrote:
> It's better to clean them up in meta/lib/oe/rootfs.py, instead of
> forever carrying a custom patch to rpm.
I have the same hesitations, but we don't know the filenames to
clean up. We can't remove every /var/tmp/rpm-tmp.*, since they
can be crea
ator instead:
sane &= package_qa_handle_error(...)
As far as I can tell, this is not a real problem in practice, because
warnings of different levels (WARN_QA, ERROR_QA) does not seem to have
been mixed in a way that triggered this issue.
Signed-off-by: Olof Johansson
---
meta/classes/insane
If a license file referenced from LIC_FILES_CHKSUM doesn't exist,
insane.bbclass would output an error message, but would continue the
build. This change makes this error fatal (as I suspect has been the
intention).
Signed-off-by: Olof Johansson
---
meta/classes/insane.bbclass | 2 +-
1
ut of available inodes).
This is a quick fix to avoid this. It does degrade functionality for
those working with debugging pre-/postintall scripts.
Fixes [YOCTO #12792]
Signed-off-by: Olof Johansson
---
...ve-written-tmp-scriptlets-even-with-debug.patch | 37 ++
meta/recipes-devtool
On 17-12-18 12:00 +, Burton, Ross wrote:
> Just pushed ross/elf with this change. The next obvious step is to turn
> lib/oe/qa.py into a new lib/oe/elf.py which has a better API and includes
> the stripped function.
Great; I've adapted my changes on top of yours and tested it a
bit. I've publ
On 17-12-06 21:38 +, Burton, Ross wrote:
> If you are looking at this then you'll want to pull my branch again as I
> just pushed a few fixes to make ARM binaries actually work.
>
> Also, a prototype "is this binary stripped" function would be:
>
> elf = oe.elf.Elf.from_file(sys.argv[1])
> fo
On 17-12-04 12:36 +, Burton, Ross wrote:
> You might be interested in some of the patches I've got sitting in
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=ross/mutb
> specifically around "Add new ELF parser". This adds a fully-featured
> Python ELF parser to lib/oe which cou
On 17-12-01 22:13 +0100, Olof Johansson wrote:
> On 17-12-01 11:43 -0600, Mark Hatle wrote:
> > The original implementation did not include the ',' options because
> > different
> > versions of file (as well as different architectures) would return different
On 17-12-01 16:50 +0100, Olof Johansson wrote:
> is_elf/isELF had copies in both staging.bbclass and package.bbclass.
> After recent refactoring in staging.bbclass (involving breaking out the
> isELF function to is_elf in lib/oe/package.py), the implementions
> diverged. It would be b
On 17-12-01 11:43 -0600, Mark Hatle wrote:
> The original implementation did not include the ',' options because different
> versions of file (as well as different architectures) would return different
> strings.
>
> They all contains the same information, but order and inclusion of ',' changed
>
x27;s expose it here for others to use.
Signed-off-by: Olof Johansson
---
meta/lib/oe/package.py | 88 +++---
1 file changed, 54 insertions(+), 34 deletions(-)
diff --git a/meta/lib/oe/package.py b/meta/lib/oe/package.py
index 1e5c3aa8e1..f1f9333e0f 1
The isELF and is_elf function share a common ancestry, but have
diverged. Let's use the implementation from oe.package.
Signed-off-by: Olof Johansson
---
meta/classes/package.bbclass | 40 +---
1 file changed, 9 insertions(+), 31 deletions(-)
diff --
By using single quotes instead of double quotes, we don't have to worry
about escaping dangerous characters, other than ' itself.
Signed-off-by: Olof Johansson
---
meta/lib/oe/package.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/lib/oe/package.py b/m
e variable expansion.
The isELF function has been copied and was until this patchset available from
two locations, one in lib/oe/package.py and one in package.bbclass. The two
functions had diverged. This is changed so that one common implementation is
used.
Olof Johansson (5):
lib/oe/package.
oves the path prefix to the output, but the
path can still be included in shebang lines (which file will report as
something like "a /foo/bar/baz.py script").
Signed-off-by: Olof Johansson
---
meta/lib/oe/package.py | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff
e not prepend
filename to the output.
Signed-off-by: Olof Johansson
---
meta/lib/oe/package.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/lib/oe/package.py b/meta/lib/oe/package.py
index f1f9333e0f..eab94feb91 100644
--- a/meta/lib/oe/package.py
+++ b/meta/lib/oe/packa
d-off-by: Olof Johansson
---
meta/conf/licenses.conf | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/meta/conf/licenses.conf b/meta/conf/licenses.conf
index 9917c40..d210a0e 100644
--- a/meta/conf/licenses.conf
+++ b/meta/conf/licenses.conf
@@ -133,11 +133,10 @@ DATA_LI
On 16-10-27 08:46 -0400, Robert P. J. Day wrote:
>
> ack ... never mind, i see my fundamental misunderstanding. i thought
> all of the recipes being processed via the "perl-modules" package were
> being (if necessary) downloaded from CPAN, or something like that,
> then turned into installable r
If /bin/sh is a regular file (and not a symlink), we assume it's a
reasonable shell and allow it.
Signed-off-by: Olof Johansson
---
meta/classes/sanity.bbclass | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bb
On 16-05-25 16:29 +0300, Jussi Kukkonen wrote:
> https version seems more reliable and in an informal test fetching
> all gnupg recipes now takes <20% of the time it used to.
>
> Define GNUPG_MIRROR in bitbake.conf so future tweaks to this are
> easier. Replace some slower mirrors with the officia
On 16-05-23 14:09 -0400, Paul Gortmaker wrote:
> > On 16-05-20 20:02 -0400, Paul Gortmaker wrote:
> > > Several large distros are voting with their feet and actively
> > > reverting the change, as per what can be seen above for Debian.
> >
> > To me, it sounds like you're saying that the Debian ma
On 16-05-20 20:02 -0400, Paul Gortmaker wrote:
> Several large distros are voting with their feet and actively
> reverting the change, as per what can be seen above for Debian.
To me, it sounds like you're saying that the Debian maintainer is
actively taking a stand against upstream, but the chang
On 16-02-12 14:15 +0200, Dmitry Rozhkov wrote:
> man 5 os-release states that VERSION_ID is
>
> "
> a lower-case string (mostly numeric, no spaces or other
> characters outside of 0-9, a-z, ".", "_" and "-")
> identifying the operating system version
> "
>
> This becomes crucial when ClearLinux's
On 16-02-08 11:49 +, Burton, Ross wrote:
> On 5 February 2016 at 18:04, Mike Crowe wrote:
>
> > This series (sent only to openembedded-core) fixes various recipes
> > that rely on the current default value of
> > EXTRA_OEMAKE = "-e MAKEFLAGS=" to explicitly set that value in the
> > hope that
When used with --verbose, the heading for each task looks like
=== The verbose changes of example.do_do_compile:
This should instead be
=== The verbose changes of example.do_compile:
Signed-off-by: Olof Johansson
---
scripts/bitbake-whatchanged | 2 +-
1 file changed, 1 insertion(+), 1
On 16-01-21 00:11 -0800, Robert Yang wrote:
> --- a/meta/classes/insane.bbclass
> +++ b/meta/classes/insane.bbclass
> @@ -1098,6 +1098,7 @@ python do_package_qa () {
>
> do_package_qa[vardepsexclude] = "BB_TASKDEPDATA"
> do_package_qa[rdeptask] = "do_packagedata"
> +do_package_qa[vardeps] += "$
busybox ash, and it supports == as an alias for =. So in
practice, only if you use a shell like dash on target, you'll run into
issues with this.
Signed-off-by: Olof Johansson
---
meta/recipes-devtools/rpm/rpm_4.11.2.bb | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a
On 16-01-18 14:37 +, Richard Purdie wrote:
> On Mon, 2016-01-18 at 13:44 +0100, Olof Johansson wrote:
> > Hi,
> >
> > I'm currently investigating reports of bitbake not correctly
> > handling a type of change to a recipe, where the only change is a
>
Hi,
I'm currently investigating reports of bitbake not correctly
handling a type of change to a recipe, where the only change is a
filename rename to update the PV.
With bitbake-dumpsigs:
...
Variable PV value is ${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE',
False),d)[1] or '1.0'}
Va
t; +error_msg = "%s: invalid PACKAGECONFIG: %s" % (pn, pconfig)
> +package_qa_handle_error("invalid-pkgconfig", error_msg, d)
>
> I'm testing it, and will send it out sooner.
Thanks, such a patch would work nicely with my use case!
--
olo
all the affected recipes.
--
olof johansson
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
t one can
enable to get the warnings.
Signed-off-by: Olof Johansson
---
meta/classes/base.bbclass | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index e0f1053..6f652d8 100644
--- a/meta/classes/base.bbclass
+++ b
uld only premirror downloads.example.com files, fetched via http
or https. See the bitbake fetcher's uri_replace method in
bitbake/lib/bb/fetch2/__init__.py for the implementation.
--
olof johansson
--
___
Openembedded-core mailing list
Openembedded-core@list
Excerpts from Otavio Salvador's message of 2015-07-29 18:27:32 +0200:
> It allows for people to add local mirrors for example and even remap
> fetching without touching the recipes. It is a good addition IMO.
Can't you use PREMIRRORS for that?
--
Excerpts from Robert Yang's message of 2015-07-29 12:19:06 +0200:
> It is just like what GNOME_GIT, GNU_MIRROR and others did.
Tbh, I don't understand them either, but maybe that's just me. What is
the reason? I think it makes it harder to see what is happening.
f they are not interactive.
This is the behaviour of other major distros like Ubuntu and Fedora.
We also need to set it so that when executing `su -l xxx -c env' command,
/etc/profile is parsed.
[YOCTO #5359]
[YOCT
On 14-06-04 10:06 +0200, Steffen Sledz wrote:
> So what is this ^{} which comes from lib/bb/fetch2/git.py good for?
> Or is this a remnant from a typo?
>From git-rev-parse(1):
^{}, e.g. v0.99.8^{}
A suffix ^ followed by an empty brace pair means the object could be a
tag, and derefe
On 14-03-03 19:17 +0100, Khem Raj wrote:
> On Mon, Mar 3, 2014 at 6:37 AM, Olof Johansson
> wrote:
> > With this change, you can use shell like globbing expressions (as
> > supported by Python's fnmatch) for entries in SANITY_TESTED_DISTROS.
> > This makes it po
With this change, you can use shell like globbing expressions (as
supported by Python's fnmatch) for entries in SANITY_TESTED_DISTROS.
This makes it possible to say that, e.g. "all Debian 7 Wheezy releases
are supported" with the entry "Debian-7.*".
[YOCTO #5265]
Sign
On 14-02-28 16:28 +0100, Chris Larson wrote:
> Out of curiosity, why not just use fnmatch for this
> implementation?
No reason. That would probably be far superior. I'll send a
revised patch some time early next week! Thanks!
--
olofjn
___
Openembedded
With this change, you can use wildcards for entries in
SANITY_TESTED_DISTROS. This makes it possible to say that, e.g. "all
Debian 7 Wheezy releases are supported" with the entry "Debian-7.*".
[YOCTO #5265]
Signed-off-by: Olof Johansson
---
meta/classes
On 14-01-24 14:43 +0100, Richard Purdie wrote:
> On Mon, 2014-01-20 at 20:24 +0800, Robert Yang wrote:
> > The dash can't handle the or [[ in parameter expansion, for example:
Good to know. Interesting! Noting that escaping the [ would work,
but not applicable in this case.
> >
> > A=/usr/bin/[[
This change adds -f when doing rm on config.status. .config.status is
not always present when doing do_configure, and that would without this
change lead to a fatal error.
Signed-off-by: Olof Johansson
---
meta/recipes-devtools/json-c/json-c_0.11.bb | 2 +-
1 file changed, 1 insertion(+), 1
On 13-11-22 07:25 +0100, Hongxu Jia wrote:
> Hi Saul,
>
> The script could not be de-bashed, it was made by create_wrapper
> which is bashism:
> Vim image/usr/bin/qemu-mips
> ...
> #! /bin/bash
> realpath=`readlink -fn $0`
> export QEMU_RESERVED_VA=0x0
> exec -a `dirname $realpath`/qemu-mips `dir
Hi,
The syslinux recipe has the following do_compile:
do_compile() {
# Rebuild only the installer; keep precompiled bootloaders
# as per author's request (doc/distrib.txt)
oe_runmake CC="${CC} ${CFLAGS}" LDFLAGS="${LDFLAGS}" installer
}
i.e, it only builds the syslinux in
The create_wrapper functions of utils.bbclass cause implicit
dependencies on bash, which may not be suitable for deployment on
target. Besides, the wrapper doesn't seem to be necessary on target.
Signed-off-by: Olof Johansson
---
meta/recipes-devtools/flex/flex.inc |6 +-
1 file ch
On 13-08-08 13:35 +0200, Hongxu Jia wrote:
> Great, how about create a new layer named 'meta-perl', we sould do:
...
> - and than move other perl libraries already existed in oe-core
>to that layer, it includes:
>* meta/recipes-extended/perl/
>* meta/recipes-devtools/perl/
I assume per
On 13-08-08 11:17 +0200, Hongxu Jia wrote:
> On 08/08/2013 05:06 PM, Olof Johansson wrote:
> > The recipe is called libdbm-sqlite-perl, but the perl module is
> > called DBD::SQLite. Is that intentional, and if so, why?
>
> I know what your mean, we should rename the reci
On 13-08-08 11:01 +0200, Hongxu Jia wrote:
> create mode 100644 meta/recipes-extended/perl/libdbm-sqlite-perl_1.40.bb
The recipe is called libdbm-sqlite-perl, but the perl module is
called DBD::SQLite. Is that intentional, and if so, why?
Regards,
--
olofjn
_
Signed-off-by: Olof Johansson
---
meta/classes/cpan.bbclass |9 +
1 file changed, 9 insertions(+)
diff --git a/meta/classes/cpan.bbclass b/meta/classes/cpan.bbclass
index bef9e82..7088039 100644
--- a/meta/classes/cpan.bbclass
+++ b/meta/classes/cpan.bbclass
@@ -18,6 +18,15 @@ export
65 matches
Mail list logo
