-
Notifications
You must be signed in to change notification settings - Fork 105
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
Dockerfile_base, archlinux_install_script: add pyyaml and cerberus python packages #133
Conversation
…thon packages Required for PX4/PX4-Autopilot#10337
@bkueng AppVeyor (Windows) is also complaining: https://ci.appveyor.com/project/MaEtUgR/firmware/build/1305#L158 |
I don't know what CI is doing but it looks unrelated:
and:
@MaEtUgR can you add the python packages |
The timing is a bit awkward because this is moving to Ubuntu 18.04. If this is needed immediately we may need to do it on a branch right before 65a976b. |
Side note - I've been intending to collect all of the actual python dependencies in a requirements.txt that lives in the Firmware repository, which would then be used here and in the docs. https://github.com/PX4/Firmware/blob/master/Tools/setup/requirements.txt |
I'll merge this now, but we'll either need to finishing moving armhf, snapdragon, etc to 18.04, or branch the last 16.04 version and tag. |
I already updated that in the PR as well.
Can we do the branch? I think that's the easiest & fastest way. |
Can we update https://github.com/PX4/Firmware/blob/master/.ci/Jenkinsfile-compile with the new tag? For now the latest one is called |
…thon packages (#133) Required for PX4/PX4-Autopilot#10337
@bkueng I skipped |
It's required with PX4/PX4-Autopilot#10337. It's ready to test and should get merged soon. |
I finally added the forgoten |
Required for PX4/PX4-Autopilot#10337