-
Notifications
You must be signed in to change notification settings - Fork 42
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
Smaller #268
Conversation
Upgrade synapse to 1.6.0
!testme |
ping @Josue-T |
Upgrade went well :) |
Well, I think it's really not a good idea as with this we always download all rust package at each upgrade... |
I agree with that point, it was one of my thinking when i've done that PR The point is:
What is it better:
That's why I though that few seconds added during upgrade worth the space and time saved during backup |
It depends : or some home-hosted servers, this might take a while (1MB/s on a regular ADSL takes a whole minute, fully loading the connection).
Well, on slow internet connections but quite powerful servers, I'm not even sure this is time lost (saving 660Mo to copy then compress…). It's energy and heat saved too (much less CPU load). |
Maybe a better solution is to store the rust data somewhere else (by example in /opt/yunohost/rust-build-synapse) and make just a symbolic link to this directory for the rust data. |
Problem
Solution
.rustup
(396Mo).cargo
(198Mo) folder after building matrix-synapse (159Mo)PR Status
Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)