[branch-24.03] snap: fix build of embedded docs when building from source#314
Open
fnordahl wants to merge 6 commits intocanonical:branch-24.03from
Open
[branch-24.03] snap: fix build of embedded docs when building from source#314fnordahl wants to merge 6 commits intocanonical:branch-24.03from
fnordahl wants to merge 6 commits intocanonical:branch-24.03from
Conversation
The commit in the fixes tag installed a newer version of the rust compiler than what was available in the core24 build base. This fixed a Python build issue at the time for most architectures, however it turned out to still break on riscv64 due to some Python dependency build systems not properly detecting the presence of the compiler. Fix build on riscv64. Fixes: 3e51f75 ("snap/docs: Bump rust version for Python dependencies.") Signed-off-by: Frode Nordahl <[email protected]> (cherry picked from commit 12bb4df)
Signed-off-by: Frode Nordahl <[email protected]> (cherry picked from commit 332cf80) (cherry picked from commit e3079c0)
This is needed for the new `--build-constraints` option. Signed-off-by: Frode Nordahl <[email protected]> (cherry picked from commit e567356) (cherry picked from commit 2b8469e)
The next commit will update constraints for Python dependencies, and as it happens there is currently an impossible set of indirect build dependencies for the direct dependencies preventing successful build from source. We resolve this situation by allowing the use of the `pip install` `--build-constraint` argument. Signed-off-by: Frode Nordahl <[email protected]> (cherry picked from commit 25b8fbb) (cherry picked from commit a2412ef)
maturin v1.10 bumped Cargo.lock version to 4 which is not supported by rust toolchain in a 24.04 build base. setuptools 82 drops the `pkg_resources` module. watchfiles 1.1.0 depends on notify v8.0.0 which is not supported by rust toolchain in a 24.04 build base. Signed-off-by: Frode Nordahl <[email protected]> (cherry picked from commit e40148e)
There has been a reorganization of the Ubuntu discourse, and the bespoke MicroOVN category is no longer to be found. While not a perfect fit, using the MicroCloud category to unblock our CI while we figure out a new home. Signed-off-by: Frode Nordahl <[email protected]> (cherry picked from commit b015343) (cherry picked from commit 9907249)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Please review/merge commit by commit.
To support review, a log from successful build of the docs/.sphinx/venv with --no-binary :all: on emulated riscv64 VM can be viewed in [0].
The instructions on this page were useful in this endeavor [1].
0: https://gist.github.com/fnordahl/b83d1bf09e1c8b1775f02a66f2020728
1: https://canonical-ubuntu-hardware-support.readthedocs-hosted.com/boards/how-to/qemu-riscv/