Skip to content
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

Use exec() instead of shell_exec(), use result code #264

Merged
merged 23 commits into from
Mar 31, 2022
Merged

Use exec() instead of shell_exec(), use result code #264

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
4f603b2
Altering utility execution function so to use exec(), and more.
gudmdharalds Mar 28, 2022
7c46e07
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
2a7b743
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
b2687b1
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
131c7b2
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
6b0865f
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
c08fcb7
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
e45f06d
Update usage of vipgoci_runtime_measure_exec_with_retry()
gudmdharalds Mar 28, 2022
160b9a8
Fix spacing.
gudmdharalds Mar 28, 2022
d46b7c1
Merge branch 'main' into update-exec-function
gudmdharalds Mar 28, 2022
c13983e
Renaming variable.
gudmdharalds Mar 28, 2022
e3970c8
Renaming variables.
gudmdharalds Mar 28, 2022
3d1e3f0
Renaming variable.
gudmdharalds Mar 28, 2022
d34f507
Renaming variables.
gudmdharalds Mar 28, 2022
aa341e1
Renaming variables.
gudmdharalds Mar 28, 2022
f82f789
Renaming class to fit new name of function.
gudmdharalds Mar 29, 2022
bbbadb6
Test for command returning empty output; test for command that always…
gudmdharalds Mar 29, 2022
23d03e1
Provide output even if command status code is unexpected.
gudmdharalds Mar 29, 2022
69adabe
Redefine variable type.
gudmdharalds Mar 29, 2022
dc5ec1d
Update comment; test getting standard error output.
gudmdharalds Mar 29, 2022
afb8702
Add helper script.
gudmdharalds Mar 29, 2022
f296f48
Merge branch 'main' into update-exec-function
gudmdharalds Mar 30, 2022
50b73d4
Merge branch 'main' into update-exec-function
gudmdharalds Mar 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update usage of vipgoci_runtime_measure_exec_with_retry()
  • Loading branch information
@gudmdharalds
gudmdharalds committed Mar 28, 2022
commit b2687b134ba82ee9f724abd45a854cc159e1f0f2
13 changes: 10 additions & 3 deletions other-utilities.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,14 +28,21 @@ function vipgoci_util_php_interpreter_get_version(
}

$php_cmd = sprintf(
'( %s %s 2>&1 )',
'%s %s',
escapeshellcmd( $php_path ),
escapeshellarg( '-v' )
);

$php_output = vipgoci_runtime_measure_shell_exec_with_retry(
$php_output_2 = '';
$php_result_code = -255;

$php_output = vipgoci_runtime_measure_exec_with_retry(
$php_cmd,
'php_cli'
array( 0 ),
$php_output_2,
$php_result_code,
'php_cli',
false
);

if ( null === $php_output ) {
Expand Down