15026 2023-02-02 02:50:13 +0000 Unable to "devtool build <package>" in /build directory 2025-02-14 16:48:13 +0000 1 1 6 Yocto Project Subprojects eSDK eSDK unspecified x86 Multiple IN PROGRESS REVIEW Medium normal 5.99 1 weiheng.cheng unassigned randy.macleod sgw weiheng.cheng francisco.j.pedraza.gonzalez --- --- Don't know oldest_to_newest 94844 0 weiheng.cheng 2023-02-02 02:50:13 +0000 Overview: The "mv" command during devtool build <package> unable to execute as the directory is not empty if the eSDK is installed in /build directory. Steps to Reproduce: 1) Install eSDK in any directory within /build (i.e., /build/username/yocto-esdk/nodistro_sdk) 2) Source the oe environment 3) devtool modify <package> && devtool build <package> (i.e., python3-numpy) Actual Results: Subprocess output: mv: cannot move '/build/username/yocto-esdk/nodistro_sdk/tmp/work/riscv64-oe-linux/python3-numpy/1.22.3-r0/package/usr/src/debug/build/username/yocto-esdk/nodistro_sdk/workspace/sources/python3-numpy/build' to '/build/username/yocto-esdk/nodistro_sdk/tmp/work/riscv64-oe-linux/python3-numpy/1.22.3-r0/package/usr/src/debug/build': Directory not empty Expected Results: The package should be built without any error. Additional Builds and Platforms: Using kirkstone branch for openembedded-core on Linux platform. Additional Information: The line below creates this error, which exists in openembedded-core/meta/classes/package.bbclass in kirkstone & langdale branch. This line is moved to openembedded-core/meta/lib/oe/package.py in master branch. cmd = "mv %s%s%s/* %s%s" % (dvar, debugsrcdir, sdir, dvar,debugsrcdir) 94856 1 randy.macleod 2023-02-02 15:48:59 +0000 This is a bug but the obvious work-around is to use a different directory name. 94872 2 weiheng.cheng 2023-02-03 02:17:21 +0000 (In reply to comment #1) > This is a bug but the obvious work-around is to use a different directory > name. Yes, I'm aware of the workaround. However, there are systems that use /build directory. FYI, if the <package> uses a directory name same as any directory that exists in /build/username/yocto-esdk/nodistro_sdk/tmp/work/riscv64-oe-linux/<package>/<package-revision>/package/usr/src/debug/, the "devtool build <package>" will encounter the same error. Hence, I wonder if the line: cmd = "mv %s%s%s/* %s%s" % (dvar, debugsrcdir, sdir, dvar,debugsrcdir) is replaced with: cmd = "cp -r %s%s%s/* %s%s && rm -rf %s%s%s/*" % (dvar, debugsrcdir, sdir, dvar, debugsrcdir, dvar, debugsrcdir, sdir) will it work without any implication? If this fix can work, I can submit a patch for this. Thanks! 94927 3 saul.wold 2023-02-15 00:40:46 +0000 So I tried to reproduce this using a /build/<username>/sdk NOTE: python3-numpy: compiling from external source tree /build/swold/sdk/workspace/sources/python3-numpy devtool modify / devool build worked correctly. The workdir: /build/swold/sdk/tmp/work/core2-64-poky-linux/python3-numpy/1.24.1-r0 I am using the current master branch, it did not reproduce, I guess I need more details. 94961 4 weiheng.cheng 2023-02-17 07:36:06 +0000 (In reply to comment #3) > So I tried to reproduce this using a /build/<username>/sdk > > NOTE: python3-numpy: compiling from external source tree > /build/swold/sdk/workspace/sources/python3-numpy > > devtool modify / devool build worked correctly. > > The workdir: > /build/swold/sdk/tmp/work/core2-64-poky-linux/python3-numpy/1.24.1-r0 > > I am using the current master branch, it did not reproduce, I guess I need > more details. Hi Saul, FYI previously I replicated the error using Kirkstone branch for openembedded-core. I tried to replicate the error using master branch for openembedded-core but end up with no error, as what you found out. So, I think you can replicate it if you use Kirkstone branch for openembedded-core. If possible, try to use bitbake version 2.0 and Kirkstone branch for all layers. Thanks! 94965 5 saul.wold 2023-02-18 01:34:59 +0000 Ok, I missed that it was kirkstone specific, you mentioned it was a line in multiple releases including master. Please try testing that attached patch which I think helps with this specific case, yes it's a bit of a corner case due to the /build directory as Randy pointed out. This moves the internal /usr/src/debug/build to /usr/src/debug/build.build and then does the other move. I think you might have actually found an issue with master also, which I need to review more carefully. 94966 6 4931 saul.wold 2023-02-18 01:40:31 +0000 Created attachment 4931 Patch for testing /build issue 94971 7 saul.wold 2023-02-19 01:30:50 +0000 Please test patch 94972 8 weiheng.cheng 2023-02-19 06:11:08 +0000 (In reply to comment #5) > Ok, I missed that it was kirkstone specific, you mentioned it was a line in > multiple releases including master. > > Please try testing that attached patch which I think helps with this > specific case, yes it's a bit of a corner case due to the /build directory > as Randy pointed out. > > This moves the internal /usr/src/debug/build to /usr/src/debug/build.build > and then does the other move. > > I think you might have actually found an issue with master also, which I > need to review more carefully. Hi Saul, The patch works fine, no issue with the "devtool build <package>" command. I also tested the image with edited recipe to make sure everything works fine, sorry for the delay. Regarding the issue with master that you've mentioned, do you mean that the "dvar + debugsrcdir + sdir" path is missing? Thanks! 94978 9 saul.wold 2023-02-21 02:04:17 +0000 Patch sent to list 97156 10 randy.macleod 2023-10-30 15:37:32 +0000 Build move to 5.0 -- ../Randy 98813 11 randy.macleod 2024-04-25 01:13:51 +0000 Bulk move of 5.0 medium importance issues to 5.99. Move to 5.1 if you want to actively work on an issue. 98874 12 randy.macleod 2024-04-25 01:23:13 +0000 Move to 5.99 since no one is actively working on this issue. Move to 5.x if you start on a task, please. 100966 13 randy.macleod 2025-02-14 16:46:23 +0000 bulk change to add Saul's non-WR address. 101032 14 randy.macleod 2025-02-14 16:48:13 +0000 Bulk change: Remove Saul's old WR address. 4931 2023-02-18 01:40:31 +0000 2023-02-18 01:40:31 +0000 Patch for testing /build issue 0001-package.bbclase-Add-check-for-build-in-copydebugsour.patch text/plain 1707 saul.wold RnJvbSAwMGJkMGEwYzJiNWM1N2Y3ZjcxMjg5NTFiMmE1ZWQxY2U4M2JjMzUxIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBTYXVsIFdvbGQgPHNhdWwud29sZEB3aW5kcml2ZXIuY29tPgpE YXRlOiBTYXQsIDE4IEZlYiAyMDIzIDAxOjM5OjU0ICswMDAwClN1YmplY3Q6IFtQQVRDSF0gcGFj a2FnZS5iYmNsYXNlOiBBZGQgY2hlY2sgZm9yIC9idWlsZCBpbiBjb3B5ZGVidWdzb3VyY2VzKCkK ClRoaXMgaXMgbmVlZGVkIHdoZW4gdGhlIFNESyBvciBlU0RLIGlzIGluc3RhbGxlZCBpbiBhIC9i dWlsZCB0b3AgbGV2ZWwKZGlyZWN0b3J5IGFzIGl0IGNvbmZsaWN0cyB3aXRoIHRoZSBidWlsZCBk aXJlY3Rvcnkgd2l0aGluIHRoZSBleGlzdGluZwovdXNyL3NyYy9kZWJ1Zy9idWlsZCAod2hpY2gg aXMgcmVhbGx5IGEgbGluaykuIFJlbmFtZSBpdCBhbmQgdGhlbiBkbyB0aGUKY29weSwgdGhpcyBp cyBub3QgYW4gaXNzdWUgd2l0aCBtYXN0ZXIgY3VycmVudGx5IGR1ZSB0byBzb21lIG90aGVyCmNo YW5nZXMgdGhhdCBvY2N1cnJlZCBpbiBtYXN0ZXIuCgpTaWduZWQtb2ZmLWJ5OiBTYXVsIFdvbGQg PHNhdWwud29sZEB3aW5kcml2ZXIuY29tPgotLS0KIG1ldGEvY2xhc3Nlcy9wYWNrYWdlLmJiY2xh c3MgfCA3ICsrKysrKysKIDEgZmlsZSBjaGFuZ2VkLCA3IGluc2VydGlvbnMoKykKCmRpZmYgLS1n aXQgYS9tZXRhL2NsYXNzZXMvcGFja2FnZS5iYmNsYXNzIGIvbWV0YS9jbGFzc2VzL3BhY2thZ2Uu YmJjbGFzcwppbmRleCA4YjExZmRkMTU1Li4yOTUwMjE4MTQ1IDEwMDY0NAotLS0gYS9tZXRhL2Ns YXNzZXMvcGFja2FnZS5iYmNsYXNzCisrKyBiL21ldGEvY2xhc3Nlcy9wYWNrYWdlLmJiY2xhc3MK QEAgLTYzNiw2ICs2MzYsMTMgQEAgZGVmIGNvcHlkZWJ1Z3NvdXJjZXMoZGVidWdzcmNkaXIsIHNv dXJjZXMsIGQpOgogICAgICAgICAjIFNhbWUgY2hlY2sgYXMgYWJvdmUgZm9yIGV4dGVybmFsc3Jj CiAgICAgICAgIGlmIHdvcmtkaXIgbm90IGluIHNkaXI6CiAgICAgICAgICAgICBpZiBvcy5wYXRo LmV4aXN0cyhkdmFyICsgZGVidWdzcmNkaXIgKyBzZGlyKToKKyAgICAgICAgICAgICAgICAjIFNw ZWNpYWwgY2FzZSBmb3IgL2J1aWxkIHNpbmNlIHdlIG5lZWQgdG8gbW92ZSBpbnRvCisgICAgICAg ICAgICAgICAgIyAvdXNyL3NyYy9kZWJ1Zy9idWlsZCBzbyByZW5hbWUgc2RpciB0byBidWlsZC5i dWlsZAorICAgICAgICAgICAgICAgIGlmIHNkaXIuZmluZCgiL2J1aWxkIikgPT0gMDoKKyAgICAg ICAgICAgICAgICAgICAgY21kID0gIm12ICVzJXMlcyAlcyVzJXMiICUgKGR2YXIsIGRlYnVnc3Jj ZGlyLCAiL2J1aWxkIiwgZHZhciwgZGVidWdzcmNkaXIsICIvYnVpbGQuYnVpbGQiKQorICAgICAg ICAgICAgICAgICAgICBzdWJwcm9jZXNzLmNoZWNrX291dHB1dChjbWQsIHNoZWxsPVRydWUsIHN0 ZGVycj1zdWJwcm9jZXNzLlNURE9VVCkKKyAgICAgICAgICAgICAgICAgICAgc2RpciA9IHNkaXIu cmVwbGFjZSgiL2J1aWxkIiwgIi9idWlsZC5idWlsZCIsIDEpCisKICAgICAgICAgICAgICAgICBj bWQgPSAibXYgJXMlcyVzLyogJXMlcyIgJSAoZHZhciwgZGVidWdzcmNkaXIsIHNkaXIsIGR2YXIs ZGVidWdzcmNkaXIpCiAgICAgICAgICAgICAgICAgc3VicHJvY2Vzcy5jaGVja19vdXRwdXQoY21k LCBzaGVsbD1UcnVlLCBzdGRlcnI9c3VicHJvY2Vzcy5TVERPVVQpCiAKLS0gCjIuMzUuNgoK