summaryrefslogtreecommitdiffstats
path: root/meta/classes
diff options
context:
space:
mode:
authorJoshua Watt <JPEWhacker@gmail.com>2020-07-03 13:16:37 (GMT)
committerRichard Purdie <richard.purdie@linuxfoundation.org>2020-07-16 21:28:40 (GMT)
commit2525b5f46e9ae809cf8fe535287f4d5249ffe63f (patch)
treec76fa159dbdf7d799f6a80b402d117ac0ae4b00a /meta/classes
parent854b876c25e98b21893bf39290890f68f09e30ac (diff)
downloadpoky-2525b5f46e9ae809cf8fe535287f4d5249ffe63f.tar.gz
classes/archive: do_configure should not depend on do_ar_patched
The commit d4be264061 ("classes/archiver: run do_unpack_and_patch after do_preconfigure") correctly moved do_unpack_and_patch to depend on do_preconfigure, but left do_ar_patched as a dependency of do_configure introduced from 8a7c779487 ("classes/archiver: Create patched archive before configuring"). Having do_configure depend on do_ar_patched is blatantly wrong. Firstly, doing so causes the taskhash of do_configure to change which is undesirable. Secondly, the anon python that sets up the tasks carefully skips GCC recipes that depend on gcc-source to provide their source code since running do_unpack_and_patch in them would delete the source code. Adding in the dependency effectively bypasses this and makes these recipes delete the shared gcc-source This fixes errors (for real this time) like: cat: .../gcc/defaults.h: No such file or directory when making certain configuration changes. For posterity, you could easily reproduce gcc-source being erased by running the commands (replacing aarch64 with your correct target arch): bitbake -c cleanall gcc-source gcc-cross-aarch64 bitbake -c do_deploy_archives gcc-source # The following would fail because # gcc-cross-aarch64:do_unpack_and_patch erases shared source, so # do_configure after that fails. bitbake gcc-cross-aarch64 (From OE-Core rev: 2399fdf98a6e6e4245c2e7ed1f72404ebece3eca) Signed-off-by: Joshua Watt <JPEWhacker@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> (cherry picked from commit f02c1904330da47a25cf85f0077712f584f4ee5b) Signed-off-by: Steve Sakoman <steve@sakoman.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes')
-rw-r--r--meta/classes/archiver.bbclass2
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/classes/archiver.bbclass b/meta/classes/archiver.bbclass
index bf275f4..1a3c190 100644
--- a/meta/classes/archiver.bbclass
+++ b/meta/classes/archiver.bbclass
@@ -584,7 +584,7 @@ addtask do_deploy_archives_setscene
584 584
585addtask do_ar_original after do_unpack 585addtask do_ar_original after do_unpack
586addtask do_unpack_and_patch after do_patch do_preconfigure 586addtask do_unpack_and_patch after do_patch do_preconfigure
587addtask do_ar_patched after do_unpack_and_patch before do_configure 587addtask do_ar_patched after do_unpack_and_patch
588addtask do_ar_configured after do_unpack_and_patch 588addtask do_ar_configured after do_unpack_and_patch
589addtask do_ar_mirror after do_fetch 589addtask do_ar_mirror after do_fetch
590addtask do_dumpdata 590addtask do_dumpdata