Skip to content

solved #1138 by Adding missing Windows GStreamer build documentation - #1181

Open
ayushrajput6128 wants to merge 1 commit into
opencv:4.xfrom
ayushrajput6128:issue-fix-#1138
Open

solved #1138 by Adding missing Windows GStreamer build documentation#1181
ayushrajput6128 wants to merge 1 commit into
opencv:4.xfrom
ayushrajput6128:issue-fix-#1138

Conversation

@ayushrajput6128

Copy link
Copy Markdown

This PR fixes issue #1138 by adding the missing Windows GStreamer build instructions
to the opencv-python repository documentation.

Changes included:

  • Added step-by-step instructions for setting up GStreamer on Windows
  • Updated any relevant references and links
  • Ensured formatting matches existing documentation style

This should help Windows users successfully build OpenCV with GStreamer support.

@Mukller

Mukller commented Aug 24, 2026

Copy link
Copy Markdown

The section fills a real gap (#1138 — Windows GStreamer docs were indeed missing), but as written it can't be followed to completion:

  1. The actual build command is missing. The Build section says "Run the standard manual build command:" and then ends — there's no command after the colon. Since this is the entire point of the section, it needs the concrete invocation (the python setup.py bdist_wheel / pip wheel . --verbose line Windows builders actually run, plus which shell it assumes).

  2. The environment-variable snippet isn't fenced. Everywhere else in this README shell snippets use triple-backtick blocks; here set PATH=... / set GST_PLUGIN_PATH=... sit as bare text, which breaks rendering and copy-paste. They're also cmd.exe-style (set X=Y) — worth saying explicitly that these go into cmd.exe, not PowerShell ($env:PATH = ... there).

  3. Consider mentioning GST_PLUGIN_SYSTEM_PATH alongside GST_PLUGIN_PATH — for runtime plugin discovery during tests of the built wheel, the SYSTEM_PATH variant is what GStreamer actually consults when GST_PLUGIN_PATH is meant for development plugins only.

With the command restored and the block fenced this is a clear keep — the prerequisites list itself (runtime + development MSVC installers, matching Python) is accurate.

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