GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — docs: add Windows instructions for pomerge in translating guide by pranavchoudhary-tech · Pull Request #1862 · python/devguide

Conversation

@pranavchoudhary-tech

@pranavchoudhary-tech pranavchoudhary-tech commented Jul 18, 2026 •
edited by zware
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Fixes #1783

This PR adds the missing Windows commands tab for the pomerge translation recovery tool, resolving the TODO in translating.rst. The commands for Windows use xcopy and rmdir to achieve the same result as the Unix cp and rm.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@read-the-docs-community

read-the-docs-community Bot commented Jul 18, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Documentation build overview

📚 CPython devguide | 🛠️ Build #35460729 | 📁 Comparing 84bd6c5 against latest (4d14223)

  🔍 Preview build  

44 files changed · + 1 added · ± 43 modified

+ Added

  • development-tools/sanitizers/index.html

± Modified

  • index.html
  • _static/psrt-coordinator-triage.html
  • clang/index.html
  • development-tools/index.html
  • documentation/index.html
  • genindex/index.html
  • getting-started/index.html
  • security/index.html
  • triage/index.html
  • versions/index.html
  • and 33 more...
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

StanFromIreland
StanFromIreland reviewed Jul 18, 2026
View reviewed changes
Comment thread getting-started/pull-request-lifecycle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@pranavchoudhary-tech
pranavchoudhary-tech force-pushed the fix-windows-pomerge branch from 36bbc80 to fc0c616 Compare July 18, 2026 07:06
@pranavchoudhary-tech

pranavchoudhary-tech commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

Ah, my apologies! I created this branch from my fork's main branch without syncing with upstream/main first, which accidentally dragged in some unrelated commits from another PR I was working on.

I have just rebased and force-pushed to clean it up. It should only contain the Windows instructions for pomerge now. Let me know if everything looks good!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

zware
zware previously approved these changes Jul 18, 2026
View reviewed changes

@zware zware left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

The commands look reasonable to me, assuming that pomerge is available and working the same way on Windows.

I haven't tried this out, I'm hoping you have run this successfully? :)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@pranavchoudhary-tech

pranavchoudhary-tech commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the review and approval! Yes, I can confirm the standard shell commands (xcopy, rmdir, git checkout) work perfectly in the Windows Command Prompt. Since pomerge is a Python tool, it uses Python's built-in glob which natively handles the **\*.po globbing patterns on Windows just as well as it does on Unix! 😄

zware reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@StanFromIreland StanFromIreland self-assigned this Jul 18, 2026
@tomasr8 tomasr8 added the topic-sprints label Jul 18, 2026
@StanFromIreland StanFromIreland added sprint and removed topic-sprints labels Jul 18, 2026
@StanFromIreland

StanFromIreland commented Jul 18, 2026

Copy link
Copy Markdown
Member

@tomasr8 topic-sprints is for docs on how to organise/run sprints. I’ve just created a sprint label for PRs being worked on at a sprint, like in CPython.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

StanFromIreland
StanFromIreland reviewed Jul 18, 2026
View reviewed changes

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Can you please add the .. include:: /include/activate-tab.rst to the top of the file like we have elsewhere.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@pranavchoudhary-tech

pranavchoudhary-tech commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

Done! Added the activate-tab.rst include to the top of the file.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

StanFromIreland
StanFromIreland reviewed Jul 18, 2026
View reviewed changes
Comment thread documentation/translations/translating.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@tomasr8

tomasr8 commented Jul 18, 2026

Copy link
Copy Markdown
Member

@tomasr8 topic-sprints is for docs on how to organise/run sprints. I’ve just created a sprint label for PRs being worked on at a sprint, like in CPython.

Haha yeah I figured, sorry about that! I just wanted a temporary label so I could keep track of the issues :) Thanks for adding the label!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@encukou

encukou commented Jul 18, 2026

Copy link
Copy Markdown
Member

I'll be at a Windows computer next week, if it helps.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pranavchoudhary-tech

pranavchoudhary-tech commented Jul 19, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

@StanFromIreland good catch from Claude! You're totally right, cmd.exe doesn't expand wildcards so the previous command would just crash.

I've updated the dosbatch instructions to use a standard for /R loop instead. This passes the files to pomerge one by one, which works perfectly since it saves its translation memory iteratively.

I also added shopt -s globstar to the Unix tab, since Bash actually needs that flag enabled for **/*.po to work recursively too. Thanks for pointing this out!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

StanFromIreland
StanFromIreland reviewed Jul 19, 2026
View reviewed changes
Comment thread documentation/translations/translating.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread documentation/translations/translating.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@pranavchoudhary-tech

pranavchoudhary-tech commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

@StanFromIreland Excellent catch on the translation memory overwrite! You are absolutely right; pomerge overwrites ~/.pomerge.json on every invocation, so looping through files one-by-one is completely broken.

I've pushed a fix that replaces the Windows dosbatch tab entirely with a powershell script. By using PowerShell's Get-ChildItem, we can natively expand the wildcards into an array and pass all files to pomerge in a single invocation, bypassing the overwrite issue completely while avoiding the lack of native globbing in cmd.exe.

I've also applied your suggested comment to the Unix tab!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pranavchoudhary-tech
pranavchoudhary-tech requested a review from zware July 24, 2026 05:27
@pranavchoudhary-tech
pranavchoudhary-tech force-pushed the fix-windows-pomerge branch from bd9415c to 73448bb Compare July 31, 2026 06:51
@pranavchoudhary-tech
pranavchoudhary-tech force-pushed the fix-windows-pomerge branch from 73448bb to 22700d1 Compare August 12, 2026 15:35
@pranavchoudhary-tech

pranavchoudhary-tech commented Sep 9, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Hi @zware, thanks for the approval! All 5 checks are passing and the branch merges cleanly. Could you go ahead and merge when you get a chance? Thanks!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pranavchoudhary-tech
pranavchoudhary-tech force-pushed the fix-windows-pomerge branch from 22700d1 to 058e1da Compare September 9, 2026 09:36
@zware
zware dismissed their stale review September 9, 2026 16:26

Long out of date.

@pranavchoudhary-tech
pranavchoudhary-tech force-pushed the fix-windows-pomerge branch from 058e1da to d5dd47b Compare October 4, 2026 19:23
@pranavchoudhary-tech

pranavchoudhary-tech commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

Hi @zware, I've just rebased the branch onto the latest main — everything is up to date now. Would appreciate another look when you get a chance!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pranavchoudhary-tech
docs: add Windows instructions for pomerge in translating guide (fixes …
84bd6c5
…python#1783)
@pranavchoudhary-tech
pranavchoudhary-tech force-pushed the fix-windows-pomerge branch from d5dd47b to 84bd6c5 Compare October 28, 2026 04:45
@pranavchoudhary-tech
pranavchoudhary-tech requested a review from StanFromIreland October 28, 2026 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zware zware Awaiting requested review from zware
@StanFromIreland StanFromIreland Awaiting requested review from StanFromIreland

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@StanFromIreland StanFromIreland

Labels

sprint

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Task: Add windows instructions for translations when CPython files move

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@pranavchoudhary-tech @StanFromIreland @tomasr8 @encukou @zware
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.