-
Notifications
You must be signed in to change notification settings - Fork 43
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
[Linux] v1.1.8-1 test cases fail #105
Comments
Please try again with v1.1.9 as I cannot support older versions and there should be no need to stick with v1.1.8. Regardless of that I'm afraid that I cannot reproduce the problem with any version, see my build logs attached. Note that (beside the typo) there's nothing wrong with the output "Connection error: Connection configuration is insufficient.". The test simply tests whether it behaves as expected in this particular error case which is logged accordingly. I suppose the real error is syncthingtray-qt6-1.1.8-1-x86_64-build.log |
Can you apply the following patch to see whether it helps? https://github.com/Martchus/syncthingtray/commit/321bac0bf762d45e6e0b2d776450900c8e018084.patch |
v1.1.19-1 just dropped on my boxes. Same issue as before. Applied your patch (also changed all localhost lines in misctest.cpp to be sure) and now all tests passed and app installed. Thanks |
Thanks, then I'll merge that branch on master. However, if changing |
…on of `localhost` This should not be necessary but apparently helps with test failures reported in #105.
Compiling v1.1.8-1 on Arch on 3 PCs and they all throw the same error. It won't connect to the server when executing the test cases. The beginning part of the output is
The interesting part is
a) there is a minor typo "conficuration" should be configuration and b) there is no hint as to what the insufficient configuration might be so I can't fix it!
HTH
The text was updated successfully, but these errors were encountered: