Skip to content

Fix CGI Python examples in howto docs#696

Open
deprrous wants to merge 1 commit into
apache:trunkfrom
deprrous:pr/httpd-cgi-python-examples
Open

Fix CGI Python examples in howto docs#696
deprrous wants to merge 1 commit into
apache:trunkfrom
deprrous:pr/httpd-cgi-python-examples

Conversation

@deprrous

Copy link
Copy Markdown

This fixes two small drift issues in the CGI Python examples.

The environment-dump example in both cgi.xml and cgi.xml.fr lost the indentation on the loop body, so the sample no longer parses as Python. The French page also still used first.py in the chmod example even though that page's sample script is named premier.py.

Checks:

  • xmllint --noout docs/manual/howto/cgi.xml
  • xmllint --noout docs/manual/howto/cgi.xml.fr
  • git diff --check

Copilot AI review requested due to automatic review settings July 26, 2026 12:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes drift in the CGI Python documentation examples so that the samples are consistent and syntactically valid.

Changes:

  • Restores correct Python indentation for the environment-dump loop body in both the English and French CGI howto pages.
  • Updates the French chmod example to reference premier.py (matching the French sample script name) instead of first.py.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
docs/manual/howto/cgi.xml.fr Fixes Python example indentation and corrects the chmod example to use premier.py.
docs/manual/howto/cgi.xml Fixes Python example indentation so the sample parses correctly.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants