Skip to content
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

Docker cleanup #2238

Merged
merged 5 commits into from
Mar 4, 2024
Merged

Docker cleanup #2238

merged 5 commits into from
Mar 4, 2024

Conversation

julianoes
Copy link
Collaborator

See commits.

We no longer maintain and use the integration tests and SITL tests.

To do this right, we would need a lot more testing infrastructure, and
would have to keep up to date with all PX4 and ArduPilot releases.

I don't have bandwidth to do that, and I don't think it should be up to
MAVSDK to keep a docker image to run PX4 and ArduPilot.

Hence, I'm removing it all.

Signed-off-by: Julian Oes <julian@oes.ch>
This takes us from GCC 8.5 to GCC 11 which fixes issues around
stdc++fs linking.

Whoever, needs to have older libraries that run on Debian Bullseye or
older will have to build themselves.

Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
@julianoes julianoes requested a review from JonasVautherin March 4, 2024 17:19
Signed-off-by: Julian Oes <julian@oes.ch>
JonasVautherin
JonasVautherin previously approved these changes Mar 4, 2024
That's because the docker images have changed.

Signed-off-by: Julian Oes <julian@oes.ch>
Copy link

sonarqubecloud bot commented Mar 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@julianoes julianoes merged commit 52fc1c4 into main Mar 4, 2024
28 checks passed
@julianoes julianoes deleted the pr-docker-cleanup branch March 4, 2024 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants