3872 2013-02-09 17:08:58 +0000 Multiple scripts directories causes infinite oe-core/scripts/bitbake loop 2013-02-15 22:34:35 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core unspecified x86 Multiple RESOLVED FIXED Medium major 1.4 M4 1 fcooper sgw denis fcooper meta.mr.watcher meta.watcher --- --- --- oldest_to_newest 29700 0 fcooper 2013-02-09 17:08:58 +0000 When sourcing conf/setenv the below snippet searches all your layers for a scripts directory and adds it to your path. # check for any scripts directories in the top-level of the repos and add those # to the PATH SCRIPTS=`find ${OEBASE}/sources -maxdepth 2 -name "scripts" -type d` for s in ${SCRIPTS} do PATH="${s}:$PATH" done Usually oe-core is the only layer that contains a scripts directory but it is possible for other layers to define their own especially if they plan on overriding a script contained within oe-core's scripts directory. oe-core/scripts/bitbake contains the below snippet: export PATH=`echo $PATH | sed s#[^:]*/scripts:##` This portion is meant to remove any scripts directory from the PATH temporarily. However, this assumes that only one scripts directory has been defined. In the event that more scripts directory is located in the path only once instance will be removed. For whatever reason the bitbake script freaks out and results in a recursive call to itself spawning multiple instances of itself until after a long time it crashes. The simple fix is to add the g option to the regular expression so all instances of the scripts directory within the path is removed. 29701 1 1068 fcooper 2013-02-09 17:13:22 +0000 Created attachment 1068 Bug fix 29718 2 denis 2013-02-11 19:43:19 +0000 Franklin, 1. Do you realize that conf/setenv as well as the code you cited below are specific to our own layer setup tool from the Arago Project? 2. And even with that, I don't seem to experience any problems with setup code and multiple instances of scripts/ directory... 29734 3 denis 2013-02-11 23:19:43 +0000 Ok, I was able to reproduce the issue on Franklin's machine... The "find" command below is from our own setenv script, but it adds all occurrences of <layer>/scripts to the PATH variable. The order of layers is not guaranteed, so if oe-core/scripts is not the first in the list, it won't be processed by the below sed command within scripts/bitbake wrapper and filtered out from the PATH variable before re-calling the actual bitbake binary. And since it only filters out the first occurrence (which could have been e.g. meta-arago/scripts), it ends up calling bitbake wrapper from oe-core/scripts again and again. 29874 4 sgw 2013-02-15 22:34:35 +0000 Thanks for the patch! http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=a9608dd3e5d32a784c13d2b379d36b9046226172 1068 2013-02-09 17:13:22 +0000 2013-02-09 17:13:22 +0000 Bug fix 0001-scripts-bitbake-Remove-all-instances-of-paths-to-a-l.patch text/plain 1358 fcooper RnJvbSBjMGJjYmM2OWZkYWU3YjIwMTgwZDZlZjY1NzZiOGZlYzgyODZhZWIwIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBGcmFua2xpbiBTLiBDb29wZXIgSnIgPGZjb29wZXJAdGkuY29t PgpEYXRlOiBTYXQsIDkgRmViIDIwMTMgMTE6MDg6MTAgLTA2MDAKU3ViamVjdDogW1BBVENIXSBz Y3JpcHRzL2JpdGJha2U6IFJlbW92ZSBhbGwgaW5zdGFuY2VzIG9mIHBhdGhzIHRvIGEgbGF5ZXIn cyBzY3JpcHRzIGRpcmVjdG9yeS4KCiogQ3VycmVudGx5IHRoZSBhc3N1bXB0aW9uIGlzIG1hZGUg dGhhdCBvbmx5IG9lLWNvcmUgY2FuIGluY2x1ZGUgYSBzY3JpcHRzCiAgZGlyZWN0b3J5LgoKKiBI b3dldmVyLCB3aGVuIG90aGVyIGxheWVycyBjcmVhdGUgYSBzY3JpcHRzIGRpcmVjdG9yeSB0aGUg Yml0YmFrZSBzY3JpcHQKICBmcmVha3Mgb3V0IGNhdXNpbmcgYSBpbmZpbml0ZSByZWN1cnNpdmUg bG9vcCB1bnRpbCBpdCBjcmFzaGVzLgoKKiBTaW1wbHkgY2hhbmdpbmcgdGhlIHJlZ3VsYXIgZXhw cmVzc2lvbiB0byByZW1vdmUgYWxsIGluc3RhbmNlcyBvZiBzY3JpcHRzIHBhdGgKICBpbnN0ZWFk IG9mIGp1c3QgdGhlIGZpcnN0IG9uZSBmaXhlcyB0aGlzIHByb2JsZW0uCgogW1lvY3RvIEJ1ZyAz ODcyXQoKU2lnbmVkLW9mZi1ieTogRnJhbmtsaW4gUy4gQ29vcGVyIEpyIDxmY29vcGVyQHRpLmNv bT4KLS0tCiBzY3JpcHRzL2JpdGJha2UgfCAgICAyICstCiAxIGZpbGVzIGNoYW5nZWQsIDEgaW5z ZXJ0aW9ucygrKSwgMSBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9zY3JpcHRzL2JpdGJha2Ug Yi9zY3JpcHRzL2JpdGJha2UKaW5kZXggNzlhODFlYS4uY2EyYmM4MiAxMDA3NTUKLS0tIGEvc2Ny aXB0cy9iaXRiYWtlCisrKyBiL3NjcmlwdHMvYml0YmFrZQpAQCAtMTI2LDcgKzEyNiw3IEBAIGlm IFsgJG5lZWRwc2V1ZG8gIT0gIjAiIC1hICRidWlsZHBzZXVkbyAtZXEgMCBdOyB0aGVuCiBmaQog CiBPTERQQVRIPSRQQVRICi1leHBvcnQgUEFUSD1gZWNobyAkUEFUSCB8IHNlZCBzI1teOl0qL3Nj cmlwdHM6IyNgCitleHBvcnQgUEFUSD1gZWNobyAkUEFUSCB8IHNlZCBzI1teOl0qL3NjcmlwdHM6 IyNnYAogaWYgWyAkYnVpbGRwc2V1ZG8gLWd0IDAgXTsgdGhlbgogICAgIFsgJGJ1aWxkcHNldWRv IC1lcSAxIF0gJiYgZWNobyAiUHNldWRvIGlzIG5vdCBwcmVzZW50IGJ1dCBpcyByZXF1aXJlZCwg YnVpbGRpbmcgdGhpcyBmaXJzdCBiZWZvcmUgdGhlIG1haW4gYnVpbGQiCiAgICAgWyAkYnVpbGRw c2V1ZG8gLWVxIDIgXSAmJiBlY2hvICJQc2V1ZG8gbWF5IGJlIG91dCBvZiBkYXRlLCByZWJ1aWxk aW5nIHBzZXVkbyBiZWZvcmUgdGhlIG1haW4gYnVpbGQiCi0tIAoxLjcuMC40Cgo=