Skip to content

select: remove the use of POLLPRI from the poll() call#7618

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/macos-pollpri
Closed

select: remove the use of POLLPRI from the poll() call#7618
bagder wants to merge 1 commit into
masterfrom
bagder/macos-pollpri

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Aug 23, 2021

This reverts commit e21bd22.

Reported-by: marc-groundctl
Fixes #7595

This reverts commit e21bd22.

Reported-by: marc-groundctl
Fixes #7595
@bagder
Copy link
Copy Markdown
Member Author

bagder commented Aug 23, 2021

The big question is of course if this change breaks something else...

@bagder
Copy link
Copy Markdown
Member Author

bagder commented Aug 23, 2021

Nah, scrap this. We should instead avoid poll on mac for cmake builds too.

@bagder bagder closed this Aug 23, 2021
@bagder bagder deleted the bagder/macos-pollpri branch August 23, 2021 14:34
@mback2k
Copy link
Copy Markdown
Member

mback2k commented Aug 23, 2021

Yes, this would break Cygwin/msys again.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

On MacOS, if a remote socket is closed while curl is connecting to it, the program will spin until the connection timeout elapses

2 participants