-
Notifications
You must be signed in to change notification settings - Fork 284
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
CI is red again #474
Comments
Hi, this is great thank you for this clever fix. But this also raises a new question. GNUstep code as well as application code is full of IMP caching and my impression now is that that code also would be affected of the sam issue. Should we rewrite all that code to use the proper way to get the method implementation and advise users to do the same? |
I think this is the wrong issue ^^ |
Yes, it probably is a bit of a misunderstanding, but I think the point is related to this issue:
I'm away from home at the moment, so this is from memory, but when cacheing IMPs we want to be sure we are looking them up correctly, so it is probably worth checking that the NSObject -methodForSelector: implementation is doing the right thing.
… |
These are the two remaining failures on Windows MSVC (everything else is green):
They both fail with this error:
It looks like the HTTPServer helper is not actually being built: running the tests just creates the empty folders |
I'll take a look at it tomorrow :) |
The text was updated successfully, but these errors were encountered: