Possible race in CacheDir config about scons

[mwichmann]

Note the MSVC config caching recently has locking added, using the new FileLock class. We should be able to do the same for the CacheDir config.

[mtvjr]

Thank you for the quick response and my apologies for not responding myself, I tend not to check GitHub very often. Let me know if you need any help or any additional information about the issue.

In our sconstruct, we've bypassed the issue by calling CacheDir directly as soon as we know what directory our cache will use, but before spawning any jobs.

[bdbaddog]

Thank you for the quick response and my apologies for not responding myself, I tend not to check GitHub very often. Let me know if you need any help or any additional information about the issue.

In our sconstruct, we've bypassed the issue by calling CacheDir directly as soon as we know what directory our cache will use, but before spawning any jobs.

Are you spawning off other scons runs? or just regular build tasks?

[mwichmann]

Yeah, I'd say unless you're planning to have distinct CacheDir setups per environment, getting it set up early is good practice anyway.

[mtvjr]

Thank you for the quick response and my apologies for not responding myself, I tend not to check GitHub very often. Let me know if you need any help or any additional information about the issue.
In our sconstruct, we've bypassed the issue by calling CacheDir directly as soon as we know what directory our cache will use, but before spawning any jobs.

Are you spawning off other scons runs? or just regular build tasks?

It should just be regular build tasks, with custom actions.