Solaris 5.10 has poll(), so set try_poll_syscall="yes" in configure #103

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
@tim-brown
Contributor

tim-brown commented May 29, 2012

... in order to enable it. Otherwise Solaris 5.10 has issues building.

Tim Brown
Solaris 5.10 has poll(), so set try_poll_syscall="yes" in configure
in order to enable it. Otherwise Solaris 5.10 has issues building.
@tim-brown

This comment has been minimized.

Show comment Hide comment
@tim-brown

tim-brown May 29, 2012

Contributor

I've not checked this with respect to "some fids cannot be polled" or other subtleties with the racket polling system

Contributor

tim-brown commented May 29, 2012

I've not checked this with respect to "some fids cannot be polled" or other subtleties with the racket polling system

@samth

This comment has been minimized.

Show comment Hide comment
@samth

samth Jun 2, 2012

Member

@mflatt or @tim-brown, is this pull request fully merged?

Member

samth commented Jun 2, 2012

@mflatt or @tim-brown, is this pull request fully merged?

@tim-brown

This comment has been minimized.

Show comment Hide comment
@tim-brown

tim-brown Jun 2, 2012

Contributor

Sam,

On 2 Jun 2012 17:52, "Sam Tobin-Hochstadt" <
reply@reply.github.com>
wrote:

@mflatt or @tim-brown, is this pull request fully merged?

Just checked, and it's still outstanding. It's really up to Matt and Co to
merge. As far a I'm concerned, Solaris has poll(); and there'll be no less
told by configure if this is merged.

Tim

Contributor

tim-brown commented Jun 2, 2012

Sam,

On 2 Jun 2012 17:52, "Sam Tobin-Hochstadt" <
reply@reply.github.com>
wrote:

@mflatt or @tim-brown, is this pull request fully merged?

Just checked, and it's still outstanding. It's really up to Matt and Co to
merge. As far a I'm concerned, Solaris has poll(); and there'll be no less
told by configure if this is merged.

Tim

@samth

This comment has been minimized.

Show comment Hide comment
@samth

samth Jun 2, 2012

Member

Ok, thanks @tim-brown.

Member

samth commented Jun 2, 2012

Ok, thanks @tim-brown.

@mflatt

This comment has been minimized.

Show comment Hide comment
@mflatt

mflatt Jun 2, 2012

Member

I've merged it, finally.

Member

mflatt commented Jun 2, 2012

I've merged it, finally.

@elibarzilay elibarzilay closed this Jun 8, 2012

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