Skip to content

Update Configure docs, again #4739

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 2, 2025
Merged

Conversation

mwichmann
Copy link
Collaborator

@mwichmann mwichmann commented Jun 27, 2025

Tweak the wordings of CheckLib, CheckLibWithHeader, CheckType, CheckTypeSize.

Add id markers to all the configure methods so they can be linked to.

Also added a docstring to the Node Attrs inner class - this relates to Configure because it adds attributes to Nodes directly, but Attrs uses __slots__ - seemed worth a brief clarification.

This is a doc-only change.

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated CHANGES.txt and RELEASE.txt (and read the README.rst).
  • I have updated the appropriate documentation

Tweak the wordings of CheckLib, CheckLibWithHeader, CheckType,
CheckTypeSize.

Add id markers to all the configure methods so they can be linked to.

Also added a docstring to the Node "Attrs" inner class - this relates to
Configure because it adds attributes to Nodes directly, but Attrs
uses __slots__ - seemed worth a brief clarification.

Signed-off-by: Mats Wichmann <mats@linux.com>
Make entries more consistent with each other.

Note when some check functions add to config.h.

Signed-off-by: Mats Wichmann <mats@linux.com>
Signed-off-by: Mats Wichmann <mats@linux.com>
@mwichmann mwichmann force-pushed the doc/configure branch 2 times, most recently from 646c17a to 41da705 Compare July 1, 2025 13:19
Signed-off-by: Mats Wichmann <mats@linux.com>
@bdbaddog bdbaddog merged commit aed426b into SCons:master Jul 2, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Next Release Jul 2, 2025
@mwichmann mwichmann added this to the NextRelease milestone Jul 2, 2025
@mwichmann mwichmann deleted the doc/configure branch July 2, 2025 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants