Always rebuild it when building dist#15877
Conversation
|
This is intended behaviour, because |
Yes, but see my comment in #13372 that SCons doesn't detect changes in dependencies that trigger a new build. |
|
Would you be able to summarize (preferably with clear steps to reproduce) what are issues caused by this approach? |
Okay, I'll do that tomorrow. For the case where NVDA in dist is run maybe it's not an issue for this PR.Maybe a new issue should be created. |
|
Since #15884 has not been caused by this PR, I don't think this work should be blocked by it.
however the created folder does not contain |
Link to issue number:
Fixes #13372
Summary of the issue:
When rebuilding
dist, some code changes were not detected. SCons does not trigger a new build.Description of user facing changes
None
Description of development approach
distwill always be considered obsolete.Testing strategy:
Test steps from #13372:
Known issues with pull request:
It will always rebuild when building
dist.After running NVDA in the
distfolder, encountered a Windows error during the rebuild. Speculating that py2exe is unable to replace the occupied file.This issue, although not caused by this PR, should still be mentioned. See details in #15884.
There is no method found to only replace the modified files when rebuilding py2exe
Code Review Checklist: