-
-
Notifications
You must be signed in to change notification settings - Fork 659
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
[talk] Nextcloud Talk High Performance Back-end repo will change it's structure #2392
Comments
Not sure how to handle this. Adding Does the VM have any possibility to announce news or notifications to the users? (motd, nextcloud notifications, ...) |
Question is why you decided to change your repo in the first place? Is it so important that thousands of users won't be able to run 'apt-update' without an error? |
Reason was to allow the repo to be flagged properly for unattended-upgrades, which requires these fields. Also, it adheres to best practices to supply them. Unfortunately, i forgot to add them in the past, which leads to this hassle now. If a user runs It's just annoying when being used in non-interactive scripts. That's why i created this issue here, so hopefully everybody who runs into this get's the info. |
@morph027 We could automatically accept it if TURN is installeld by adding a |
Installation from scratch would work without any issues. Updating an existing installation will suffer the problem mentioned, but will work if the user runs Just using |
We can check if TURN is installed during update, and just force it with D you think you could prep a PR? |
@morph027 As far as I can tell everything is still working as expected. Do you know when this will go live? |
There is no new build since the announcement. So changes to the repo are not live. Other repos are working fine. Do we want to schedule the change? Or wait until the next upstream release? |
Fix for #2392 Signed-off-by: Daniel Hansson <[email protected]>
@morph027 Did you have time to check my PR? |
Will do tonight, was travelling yesterday... |
* Upgrade repository for Talk Fix for #2392 Signed-off-by: Daniel Hansson <[email protected]> * check for package, not for app Signed-off-by: Daniel Hansson <[email protected]> Signed-off-by: Daniel Hansson <[email protected]>
Please go ahead and push your fix @morph027 |
Steps To Reproduce
Run
apt-get update
.Expected Result
apt-get update
finishes without error.Actual Result
apt-get update
finishes with error.Screenshots, Videos, or Pastebins
No response
Additional Context
Not active yet, will happen with the next upstream or bugfix release
Announcement: https://gitlab.com/packaging/nextcloud-spreed-signaling/-/issues/5
Build Version
n/a
Environment
By using the scripts
Environment Details
No response
The text was updated successfully, but these errors were encountered: