-
-
Notifications
You must be signed in to change notification settings - Fork 127
tinyproxy: cleanup config and script #596
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
Conversation
| @@ -1,12 +0,0 @@ | |||
| #!/bin/bash | |||
|
|
|||
| if [ -e /usr/sbin/tinyproxy ]; then | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/usr/sbin/tinyproxy stopped being used in Debian Buster. So dropping this by now should be fine.
(Could even revert the command -v since the path should now be fixed and static?)
|
Packages fail to build - makefile needs to be updated. |
Upstream commit b935dc8 dropped these configs.
All distros (Fedora, Debian, Arch, Ubuntu) places tinyproxy under /usr/bin, so this wrapper is no longer needed.
Thanks. I updated Makefile. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #596 +/- ##
=======================================
Coverage 71.10% 71.10%
=======================================
Files 3 3
Lines 481 481
=======================================
Hits 342 342
Misses 139 139 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025071015-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025061004-4.3&flavor=update
Failed tests14 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/142375#dependencies 10 fixed
Unstable testsPerformance TestsPerformance degradation:12 performance degradations
Remaining performance tests:60 tests
|
That may impact updates via Tor - sometimes it's pretty slow. Apt/Dnf already have their own timeouts (and they do that a bit smarter - not just absolute value, but also based on progress). So, I'd prefer to keep the proxy one high. |
installed at same path on debian/redhat/fedora. - follow-up to QubesOS#596 - QubesOS#157 - QubesOS#155
Thank you for comment. I agree with your decision. |
Additional question:
I think 60s tinyproxy timeout is too long for system updates, because there are many mirror servers to use. Should I add commit which reduces timeout to 5s or 10s?