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
searx: 0.13.1 -> 0.14.0 #36132
searx: 0.13.1 -> 0.14.0 #36132
Conversation
I think it makes more sense to wait until the geckodriver update is merged. |
geckodriver is merged, please try enabling the tests again |
@GrahamcOfBorg build searx |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
@leenaars Did you try if |
The preCheck part was there previously. I assumed it was there for a reason, so I didn't touch it... |
Please try removing it and check if the build fails. |
I did. That breaks the build. So that test has another (longer standing) issue, not related to the issue now fixed. I will report upstream. However, this does not prevent the improved Searx in nixpkgs, I presume. Because it actually works beautifully, it is that particular test (which is one of many tests) that is broken somehow upstream... So the closure should stay as is, and should be ready for merge. |
Feel free to open a PR once the issue with the tests has been resolved. |
Motivation for this change
Version bump. Note that tests are temporarily disabled because Geckodriver is broken (#35301).
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)