Skip to content

Haiku fix ZTS build, disabling tls model - #8047

Closed
devnexen wants to merge 1 commit into
php:PHP-8.0from
devnexen:haiku_zts_build_fix
Closed

Haiku fix ZTS build, disabling tls model#8047
devnexen wants to merge 1 commit into
php:PHP-8.0from
devnexen:haiku_zts_build_fix

Conversation

@devnexen

Copy link
Copy Markdown
Member

No description provided.

@cmb69cmb69 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you! I don't know about Haiku, but you certainly do, so this looks good to me. :)

@cmb69cmb69 closed this in bea542aFeb 7, 2022
@mvorisek

mvorisek commented Mar 7, 2022

Copy link
Copy Markdown
Contributor

@devnexen can you see opcache ext as loaded /w JIT enabled with this, to me incomplete, fix on Haiku?

see related issue #8180 (comment) discussion

@devnexen

Copy link
Copy Markdown
MemberAuthor

afaik it is irrelevant in case of haiku as opcache does not work at all in the first place in this platform.

@mvorisek

Copy link
Copy Markdown
Contributor

afaik it is irrelevant in case of haiku as opcache does not work at all in the first place in this platform.

thanks for the reply, still, shouln't Haiku be checked here too

#elif defined(__x86_64__) && defined(__GNUC__) && !defined(__FreeBSD__) && !defined(__OpenBSD__) && !defined(__MUSL__)
?

@devnexen

Copy link
Copy Markdown
MemberAuthor

sure why not, haiku is still in beta and does not support yet tls model.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@devnexen@mvorisek@cmb69