Skip to content

Instantly share code, notes, and snippets.

@satwareAG-ironMike
Created December 19, 2022 15:39
Show Gist options
  • Save satwareAG-ironMike/77ec10f912e3701966dc432913c56561 to your computer and use it in GitHub Desktop.
Save satwareAG-ironMike/77ec10f912e3701966dc432913c56561 to your computer and use it in GitHub Desktop.
Build PHP 8.1 php-firebird driver for Linux
git clone --single-branch --branch fix-deprecations https://github.com/satwareAG/php-firebird.git
cd php-firebird/docker
sudo ./getmodule.sh
@satwareAG-ironMike
Copy link
Author

satwareAG-ironMike commented Dec 19, 2022

image

Artfefacts are here:

image

Failed tests:

#0 12.17 ---------------------------------------------------------------------
#0 12.17 InterBase: BLOB test [tests/004.phpt]
#0 12.17 InterBase: array handling [tests/007.phpt]
#0 12.17 =====================================================================

Test 004.phpt

Warning: main(): invalid statement handle  in /usr/src/php/ext/interbase/tests/004.php on line 112

    echo "fetch blob 3\n";
    $q = ibase_query("SELECT v_blob FROM test4 WHERE v_integer = 3"); // << this produces a Warning
    $row = ibase_fetch_object($q,IBASE_TEXT);

Test 007.phpt

 decimal(18,3) get truncated, values up to decimal(14,3) work

Test bug45373.phpt

008+ Notice: ibase_execute(): Statement expects 2 arguments, 3 given in /usr/src/php/ext/interbase/tests/bug45373.php on line 15

Just a warning, maybe it could be silenced

Test ibase_num_params_001.diff

003+ Warning: ibase_prepare(): Dynamic SQL Error SQL error code = -206 Column unknown X At line 1, column 51  in /usr/src/php/ext/interbase/tests/ibase_num_params_001.php on line 18

Nothing important, just a type I made by altering the tests for php 8.1 compatibility.

@MartinKoeditz
Copy link

Please check the latest commits. Tests 004 and 007 are valid now.

@satwareAG-ironMike
Copy link
Author

Please check the latest commits. Tests 004 and 007 are valid now.

Great news, thank you so much. Interesting PHP setting I never noticed.

@MartinKoeditz
Copy link

MartinKoeditz commented Feb 2, 2023

Didn't know about it too. Took a long time to find the reason. I put a notice in the test case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment