-
Notifications
You must be signed in to change notification settings - Fork 100
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
Install apt transport https #551
Conversation
This reverts commit 0d2c3a4.
Signed-off-by: Chris Lalancette <[email protected]>
I updated the comment but isn't there a good chance that the installation will already fail when we hit a https mirror? |
Signed-off-by: Chris Lalancette <[email protected]>
Yeah it should be done before the |
I've deployed the updated version of this PR to 20 of the failing packages on the buildfarm using this command:
All 20 of those that I reconfigured are now passing. As an additional piece of data, many of the downstream packages (which were not reconfigured) are still failing. So this definitely seems to be helping so far. |
All right, most of Melodic on Debian Stretch has rebuilt with this change, and seems much happier (Moveit is still building, but it will take a while). Travis keeps failing; I haven't investigated it yet, though it seems unrelated. Reviews are certainly welcome, then we can decide whether to merge with the failing CI or investigate it. |
To be considered for merging the change to the version needs to be reverted. With that commit CI will get retriggered anyway. If it still fails please look into it since it will only get merged with a green CI. For spurious error you can always retrigger the Travis builds if necessary. |
Signed-off-by: Chris Lalancette <[email protected]>
Travis has been having connexion issues today. I experienced it on several repositories. The last 5 travis run all failed at different times but all because of connection issue. Let's 🤞 for the new round build but if we don't get travis to pass I'd recommend merging before the nightlies anyway to get this deployed on all jobs. Change looks good to me with the version reverted 👍 |
WIP do not merge opening for visibility and testing.
Alternative to #549