Skip to content

Remove AC_FPM_POLL #4449

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Remove AC_FPM_POLL #4449

wants to merge 1 commit into from

Conversation

petk
Copy link
Member

@petk petk commented Jul 21, 2019 •
edited
Loading

AC_FPM_POLL defines HAVE_POLL symbol which is already checked by configure.ac and function poll in AC_CHECK_FUNCS.

@petk
Remove AC_FPM_POLL
1509ed1 
AC_FPM_POLL defines HAVE_POLL symbol which is already checked by
configure.ac and function poll in AC_CHECK_FUNCTIONS.
nikic
nikic approved these changes Jul 24, 2019
View reviewed changes
Copy link
Member

@nikic nikic left a comment

Choose a reason for hiding this comment

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

You can also drop HAVE_SELECT -- we assume elsewhere that is it available on non-win32 systems, and fpm does not support win32 anyway.

@php-pulls php-pulls closed this in a316996 Jul 24, 2019
@petk petk deleted the patch-poll-1 branch July 24, 2019 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikic nikic nikic approved these changes

Assignees
No one assigned
Labels
Category: Build System
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@petk @nikic