On Mon, Nov 1, 2021 at 11:52 AM, Richard Purdie wrote:<br />
<blockquote>On Mon, 2021-11-01 at 03:47 -0700, Manuel Leonhardt wrote:<br />
<blockquote>I submitted a patch to fix bb.utils.lockfile to at least break the loop and<br />fail<br />when a too long filename is passed with<br /><a href="https://lists.openembedded.org/g/bitbake-devel/message/12850" target="_blank" rel="noopener">https://lists.openembedded.org/g/bitbake-devel/message/12850</a><br /><br />Nevertheless, you have a good point here: Fixing the code that constructs the<br />filename of the lock files would be the better solution. For the sstate files<br />only lockfiles of .siginfo files can overlap, because the filename of the<br />actual<br />sstate file is already 8 characters shorter than 255 characters. I have one<br />though on that:<br /><br />From what I have seen, the filename of lockfiles is mostly constructed by<br />appending ".lock" without any further check. Truncating the filename of the<br />lockfile inside bb.utils.lockfile could be confusing however, since the<br />function<br />would not use the filename that is was explicitly passed. Hence, I guess<br />bb.utils.lockfile should fail when a too long filename was passed, and the<br />function building the lockfile's filename should ensure to keep it limited;<br />that's bb.fetch2.FetchData.__init__ then if I'm not mistaken. Would you agree?<br />Or would you say, changing the filename inside bb.utils.lockfile is ok, as<br />long<br />as the function is working?<br /><br />The drive-by fix I made with using limit instead of 254 when shortening the<br />filename is still correct and necessary, right?</blockquote>
I think we just let lockfile truncate the filename if necessary since this could<br />affect other users of the funciton too.&nbsp;<br /><br />Looking at the code you changed with the limit value it isn't correct since<br />extension is already accounted for in avail (although I agree that code is<br />confusing and I had to think twice about it).<br /><br />Cheers,<br /><br />Richard</blockquote>
Ok, I'll submit a new patch.<br /><br />For the limit value, my guess was that when always using 254 the reserved characters for siginfo are not taken into account when siginfo=False. For shorter filenames it's ensured that the siginfo file is always the sstate file's name plus ".siginfo". With longer filenames it's possible that the basename of both files is different. This comes unhandy when working with the files outside of bitbake, e.g. running maintenance tasks on the sstate mirror server, e.g. copy sstate files and ensure their respective .siginfo file is also copied.