Skip to content
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.

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

Please update usocket - needed for the (already updated) drakma #268

Closed
avodonosov opened this issue Feb 12, 2012 · 7 comments
Closed

Please update usocket - needed for the (already updated) drakma #268

avodonosov opened this issue Feb 12, 2012 · 7 comments

Comments

@avodonosov
Copy link
Contributor

The recent drakma version (already included into the February quicklisp) passes new value to the :nodelay argument of usocket:socket-connect. This value is supported only by the recent usocket (not included into quicklisp yet).

@quicklisp
Copy link
Owner

On what platform is this an issue? I can't reproduce with SBCL on OS X.

@avodonosov
Copy link
Contributor Author

13.02.2012, 02:57, "Zach Beane" reply@reply.github.com:

On what platform is this an issue? I can't reproduce with SBCL on OS X.

On CLISP for example.

Some more info.

In the previous version of usocket, the socket-connect function already has
the :nodelay parameter, but with only two possible values: T or NIL.

Drakma specified :nodelay T. On the Lisps where nodelay was not
possible, usocket signalled error "Nodelay unsupporeted".

Now we added :nodelay :if-supported. With that value usocket sets the nodelay mode
for sockets if it can; but if it can not the error is not signalled. Drakma switched
to :nodelay :if-supported, which allows it to work on all lisps.

In the last QL distro we have new drakma but old usocket. I.e. drakma passes
the :nodelay :if-supported to the usocket which doesn't know this value.
It may be the case (I don't know for sure and it will vary on different Lisps)
that :if-supported has the same effect as T.

If so, the current QL is not worse than the previous one: some lisps have working
drakma, other lisps like CLISP, CMUCL and SCL have broken drakma.

But with new usocket these lisps will also have working drakma.

Therefore the update is desirable.

@quicklisp
Copy link
Owner

I don't show any newer releases of usocket. Is it only updated in git?

@avodonosov
Copy link
Contributor Author

13.02.2012, 03:52, "Zach Beane" reply@reply.github.com:

I don't show any newer releases of usocket. Is it only updated in git?

In SVN to be more precise.

@quicklisp
Copy link
Owner

I recommend encouraging usocket developers to make a new release in time for next month's update.

In the meantime, people affected by this update should go back to the previous dist.

@avodonosov
Copy link
Contributor Author

Ok. I reported this to the usocket-devel

@quicklisp
Copy link
Owner

The next dist update will include the new usocket.

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

No branches or pull requests

2 participants