Hi Andy,
On Thu, 23 Dec 2021 at 02:07, Andy Shevchenko andy.shevchenko@gmail.com wrote:
On Thu, Dec 23, 2021 at 7:08 AM Simon Glass sjg@chromium.org wrote:
On Tue, 14 Dec 2021 at 17:33, Simon Glass sjg@chromium.org wrote:
Importing libraries in Python caches the bytecode by default. Since we run scripts in source tree it ignores the current directory settings, which is $(srctree), and creates cache just in the middle of the source tree. Move cache to the current directory.
Signed-off-by: Andy Shevchenko andriy.shevchenko@linux.intel.com
v3: avoided crash (Simon), preserved tree hierarchy tools/binman/main.py | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-)
Applied to u-boot-dm/next, thanks!
I didn't notice this before, but this seems to create files like this:
./tools/binman/usr/lib/python3/dist-packages/elftools/common/construct_utils.cpython-39.pyc
We don't really want to 'recache' the common Python files. Do you think we should revert this patch, or find another fix?
I'm not sure I understand. efitools is not common, it's a separate (non-standard) module and it's cached.
The problem is that we don't want to create this file in the source directory.
For me this file is in:
/usr/lib/python3/dist-packages/elftools/
but it could be in ~/.local I suppose.
In any case, the cache file should go there, shouldn't it?
Regards, Simon