8000 [Bridge/PhpUnit] Prefer $_SERVER['argv'] over $argv · Pull Request #25304 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Bridge/PhpUnit] Prefer $_SERVER['argv'] over $argv #25304

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.

Sign up for GitHub

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

Closed
wants to merge 5 commits into from
Closed

[Bridge/PhpUnit] Prefer $_SERVER['argv'] over $argv #25304

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9f8c89b
Prefer $_SERVER['argv'] over $argv.
ricardodevries Dec 4, 2017
67cc253
Put back some code that was removed for debug purposes.
ricardodevries Dec 4, 2017
3c28b42
Put back the correct variable names.
ricardodevries Dec 4, 2017
8fbf912
Reverting changes and init the variables at the beginning.
ricardodevries Dec 4, 2017
6140de9
Add global variables.
ricardodevries Dec 4, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
10000
Diff view
Diff view
Prev Previous commit
Next Next commit
Put back some code that was removed for debug purposes.
  • Loading branch information
@ricardodevries
ricardodevries committed Dec 4, 2017
commit 67cc25360ec5bb7d77fe820d39bffaba9347c557
3 changes: 2 additions & 1 deletion src/Symfony/Bridge/PhpUnit/bin/simple-phpunit
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,8 @@ if ($components) {
class SymfonyBlacklistSimplePhpunit {}
}
array_splice($_SERVER['argv'], 1, 0, array('--colors=always'));
Copy link
Member

Choose a reason for hiding this comment

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

ok, in fact, this change introduces a bug
both $_SERVER['argv'] and $argv must be updated
this breaks the second


$_SERVER['argv'] = $argv;
$_SERVER['argc'] = ++$argc;
include "$PHPUNIT_DIR/phpunit-$PHPUNIT_VERSION/phpunit";
}

Expand Down
0