Skip to content

Fix Windows quoting in proc_open tests - #24343

Merged
shivammathur merged 1 commit into
php:PHP-8.4from
shivammathur:fix/proc-open-windows-quoting
Oct 15, 2026
Merged

shivammathur merged 1 commit into
php:PHP-8.4from
shivammathur:fix/proc-open-windows-quoting

Conversation

@shivammathur

Copy link
Copy Markdown
Member

Follow up to GH-11669.

@ndossche

Copy link
Copy Markdown
Member

Seen #24335 ?

@shivammathur

Copy link
Copy Markdown
Member Author

@ndossche Yes, I think the since the tempfile workaround from GH-11669 was reverted, we need to fix the quoting as well.

@jorgsowa

Copy link
Copy Markdown
Contributor

What do you think about rebasing it on 8.4?

@shivammathur
shivammathur changed the base branch from master to PHP-8.4 October 15, 2026 10:02
@shivammathur

shivammathur commented Oct 15, 2026 •

Copy link
Copy Markdown
Member Author

@jorgsowa Ah, I missed selecting that, it is based on PHP-8.4

@shivammathur
shivammathur merged commit 9438a33 into php:PHP-8.4 Oct 15, 2026
18 checks passed
shivammathur added a commit to shivammathur/php-src that referenced this pull request Oct 15, 2026
* PHP-8.4:
  Fix Windows quoting in proc_open tests (php#24343)
shivammathur added a commit to shivammathur/php-src that referenced this pull request Oct 15, 2026
* PHP-8.5:
  Fix Windows quoting in proc_open tests (php#24343)
shivammathur added a commit to shivammathur/php-src that referenced this pull request Oct 15, 2026
* PHP-8.6:
  Fix Windows quoting in proc_open tests (php#24343)
arnaud-lb added a commit to heathdutton/php-src that referenced this pull request Oct 15, 2026
* up/PHP-8.4: (3595 commits)
  Fix Windows quoting in proc_open tests (php#24343)
  Pin macOS CI to OpenSSL 3 (php#24332)
  Zend: zend_call_stack_get_netbsd_pthread() pthread attribute leak.
  Revert "ext/standard/tests: work around Windows quoting with tempfiles."
  Update NEWS
  ext/standard/tests: work around Windows quoting with tempfiles.
  .github/scripts/windows: pass an extension_dir to the test runner.
  {ext/standard,sapi/cli}/tests: pass TEST_PHP_EXTRA_ARGS to php
  ext/standard/tests: pass TEST_PHP_EXTRA_ARGS to some proc_open() tests
  sapi/cli/tests/023.phpt: support TEST_PHP_EXTRA_ARGS
  std: Fix memory leak of the last response headers after a nested HTTP request (php#24201)
  Fiber: hold the object a callable resolved to in the creating frame (php#24134)
  Fix v142 ARM64 hash and floating point results (php#24317)
  Use the default archive names for Windows ARM64 builds (php#24313)
  ext/gd: Drop useless and doubtful MSVC specific code (libgd/libgd@f1480ab) (php#24316)
  ext/session: do not reuse session state a save handler tore down
  Fix phpGH-24293: Segfault with an empty password in MariaDB PAM (php#24294)
  Fix phpGH-24289: finfo::file() reports the wrong argument number
  Fix phpGH-24260: querySelectorAll() returns duplicate elements for selector lists
  Fix escape analysis for objects implementing ArrayAccess
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants