Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

PHP 7.0 - ton of segfaults in test suite #274

Open
remicollet opened this issue Jun 10, 2021 · 3 comments
Open

PHP 7.0 - ton of segfaults in test suite #274

remicollet opened this issue Jun 10, 2021 · 3 comments

Comments

@remicollet
Copy link

Using 7.0.33

TEST 1/70 [tests/annotations.phpt]
========DIFF========
001+ Termsig=11
...
=======DONE========
FAIL OpenCensus Trace: Test setting annotations [tests/annotations.phpt] 
TEST 2/70 [tests/basic_class_function.phpt]
========DIFF========
001+ Termsig=11
....
FAIL OpenCensus Trace: Basic Class Method As Function Test [tests/basic_class_function.phpt] 
PASS OpenCensus Trace: Basic Context Test [tests/basic_context.phpt] 
TEST 4/70 [tests/basic_function.phpt]
========DIFF========
001+ Termsig=11
...
========DONE========
FAIL OpenCensus Trace: Basic Function Test [tests/basic_function.phpt] 
TEST 5/70 [tests/basic_method.phpt]
========DIFF========
001+ Termsig=11
...
========DONE========
FAIL OpenCensus Trace: Basic Method Test [tests/basic_method.phpt] 
TEST 6/70 [tests/bug_001.phpt]
========DIFF========
001+ Termsig=11
...

@remicollet
Copy link
Author

Notice

  • version 0.2.2 was OK with 7.0
  • Version 0.3.0 is OK with 7.1 to 8.0 (test suite passes)

Perhaps can simple bump minimal supported version (PHP <= 7.2 are EOL)

@MrMage
Copy link
Contributor

MrMage commented Jun 10, 2021

Yes, the extension is only supported and tested on PHP 7.1+. This is documented in the package.json, but maybe not on PECL.

Could you file a PR updating the version requirements for the PECL package?

@remicollet
Copy link
Author

Could you file a PR updating the version requirements for the PECL package?

No, as I don't have any Google account (and don't want one)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants