Skip to content

Conversation

nielsdos
Copy link
Member

@nielsdos nielsdos commented Dec 4, 2024

No description provided.

Co-authored-by: =?UTF-8?q?=E6=AD=A6=E7=94=B0=20=E6=86=B2=E5=A4=AA=E9=83=8E?= <[email protected]>
Co-authored-by: Christoph M. Becker <[email protected]>
Copy link
Member

@SakiTakamachi SakiTakamachi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Member

@SakiTakamachi SakiTakamachi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@SakiTakamachi
Copy link
Member

Duplicate post? Please ignore...

A bug in firebird causes a memory leak when calling `isc_attach_database()`.
See https://github.com/FirebirdSQL/firebird/issues/7849
--SKIPIF--
<?php if (PHP_INT_SIZE != 8) die("skip: 64-bit only"); ?>
Copy link
Member

Choose a reason for hiding this comment

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

nit:

Suggested change
<?php if (PHP_INT_SIZE != 8) die("skip: 64-bit only"); ?>
<?php if (PHP_INT_SIZE != 8) die("skip 64-bit only"); ?>

Otherwise run-tests.php will report reason: : 64-bit only.

Copy link
Member Author

Choose a reason for hiding this comment

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

thx

@nielsdos nielsdos merged commit 912b13a into php:master Dec 6, 2024
10 checks passed
@iluuu1994
Copy link
Member

iluuu1994 commented Dec 7, 2024

This does not work in nightly, apparently. https://github.com/php/php-src/actions/runs/12208755353/job/34062517782 The changes from .github/workflows/push.yml should be ported to nightly.yml.

@nielsdos
Copy link
Member Author

nielsdos commented Dec 7, 2024

See #17080

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.

4 participants