Fix scons devDocs target (#19804)
Fixes #19803
Summary of the issue:
The devDocs target fails to build when running:
scons devDocs
SCons reports:
scons: *** Do not know how to make File target `devDocs'
This occurs because projectDocs/dev/developerGuide/sconscript exits early due to a misplaced Return() statement. As a result, the remaining build logic is never executed and the devDocs alias is not defined.
Description of user facing changes:
Developers can now successfully run:
scons devDocs
to build all developer documentation without errors.
Description of developer facing changes:
Ensures that all documentation targets (developer guide and devDocs) are properly registered.
Restores the devDocs alias so it can be resolved by SCons.
Maintains existing dependency behavior for dist by continuing to return "devGuide".
Additionally:
Explicitly imports Move from SCons.Script for clarity and robustness.
While Move may be available implicitly in some environments, relying on implicit availability can lead to issues in stricter or different SCons configurations.
This makes the script self-contained and more predictable.
Description of development approach:
The fix moves the Return(["devGuide"]) statement to the end of projectDocs/dev/developerGuide/sconscript.
Previously, the early Return() caused SCons to stop executing the script before:
nvdaHelper documentation targets were defined
Sphinx documentation targets were defined
the devDocs alias was created
By moving Return() to the end:
the full build graph is constructed
all targets and aliases are registered correctly
the intended dependency behavior is preserved
This is a minimal and non invasive change.
