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
autobuilds: new test 2600 failing on Solaris #10511
Comments
Continuing from #10410, half of which was fixed. |
I believe, unit2600.c does not include the necessary system header files to detect that ALARM_TIMEOUT is in use. Therefore the test runs on a platform with minimal timer resolution of 1 second and that does nor work. Since the test itself is otherwise rather platform independent, there is no need to make it work there, I believe. Could you try #10513 on your machine to see if the change works on solaris? Thanks. |
Sorry, no clue. |
Hi @MarcelRaad @bagder |
Thanks a lot @dago, the Solaris 11 machine there worked fine! (After fighting with SSH to get a connection - is OpenSSH 6.6 really the latest version available there? 😞 )
The option should be |
@icing The "expected min duration" errors are gone, but unfortunately the test is still failing:
The "remaining timeout of 2 too small to resolve via SIGALRM method" messages look suspicious, don't they? |
Hi @MarcelRaad
Unfortunately yes, building a newer version would be a major effort.
Ah yes, thanks for noticing, this should be fixed now! |
Indeed, it looks suspiciously as if I got the values wrong.😬 I pushed a fix to the PR, also taking care of the unused var. |
Looks much better now! Only test 7 is still failing:
For the first failure, the message is also buggy and always outputs Full output:
|
Thanks, that expected result (and the check) is indeed wrong. I just pushed an update. Rebased, CI is running. |
@MarcelRaad would you have time to take the PR out for a spin again? Thanks! |
@icing There's still one failure left: Full output:
This one also appears in the autotools jobs on AppVeyor, so Solaris is now as good as these :-) |
Thanks! I identified the problem with test 7 and updated the PR. |
Fixed via #10513 |
I did this
Look at the autobuilds at https://curl.se/dev/builds.html.
I expected the following
All tests successful.
On the Solaris builders, test 2600 (added in d55de24) is failing, e.g. here:
https://curl.se/dev/log.cgi?id=20230214093837-2682677#prob112
The failures are:
Info from configure
CC @icing
The text was updated successfully, but these errors were encountered: