-
Notifications
You must be signed in to change notification settings - Fork 66
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.
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
ENV variables within SF4 #114
Comments
Could you reproduce the case with test in a brand new repo? |
Hmm, it seems that the problem is within Currently it contains following: return $_ENV + [
self::ENV_TEST_CHANNEL.'='.(int) $i,
self::ENV_TEST_CHANNEL_READABLE.'=test_'.(int) $i,
self::ENV_TEST_CHANNELS_NUMBER.'='.(int) $maxProcesses,
self::ENV_TEST_ARGUMENT.'='.$suite,
self::ENV_TEST_INCREMENTAL_NUMBER.'='.(int) $currentProcessCounter,
self::ENV_TEST_IS_FIRST_ON_CHANNEL.'='.(int) $isFirstOnItsThread,
'PATH='.getenv('PATH'),
]; And with that those aren't added to env variables as expected, but if modify that part like above all works as expected: return array_merge($_ENV, [
self::ENV_TEST_CHANNEL => (int) $i,
self::ENV_TEST_CHANNEL_READABLE => 'test_' . (int) $i,
self::ENV_TEST_CHANNELS_NUMBER => (int) $maxProcesses,
self::ENV_TEST_ARGUMENT => $suite,
self::ENV_TEST_INCREMENTAL_NUMBER => (int) $currentProcessCounter,
self::ENV_TEST_IS_FIRST_ON_CHANNEL => (int) $isFirstOnItsThread,
'PATH' => \getenv('PATH'),
]); And how did I test this, just simple Result with original:
So the env variable doesn't exists at all, and after that small change:
And this is what I expected to get. |
And you could easily test this with my project https://github.com/tarlepp/symfony-flex-backend just clone it run |
This issue can be closed normally. |
Hmm seems like env variables are not working within symfony 4, for some reason eg.
ENV_TEST_CHANNEL_READABLE
is not set as it was on 3.x branch.The text was updated successfully, but these errors were encountered: