| Summary: | buildhistory-diff depends on python3-git, which is not in Debian Stable | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] OE-Core | Reporter: | Jussi Kukkonen <jku> |
| Component: | Scripts and Tools | Assignee: | Ed Bartosh <eduard.bartosh> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | Medium+ | CC: | jose.perez.carranza, leonardo.sandoval.gonzalez, ross.burton, srifenbark |
| Version: | unspecified | ||
| Target Milestone: | 2.2 M1 | ||
| Hardware: | x86 | ||
| OS: | Multiple | ||
| Whiteboard: | 8 June 2016: IN PROGRESS REVIEW | ||
| OS type for building Yocto: | --- | Type of Regression: | Regression (Used to work) |
| Verified: | Documentation change: | Done (doc changes complete) | |
|
Description
Jussi Kukkonen
2016-06-06 13:36:52 UTC
This also true for OpenSUSE 13.2 and 42.1 It's even worse on OpenSUSE as it doesn't have even python-git in default installation. There are plenty of choices how to deal with this: - find and install python3-git as Jussi suggested - install GitPython with pip3: pip3 install GitPython --user - run script with python 2 explicitly: python ../scripts/buildhistory-diff - refer to python 2 in script's shebang: #!/usr/bin/env python Changing documentation looks better than going back to python 2 from my point of view. This section (http://www.yoctoproject.org/docs/2.2/mega-manual/mega-manual.html#examining-build-history-information) talks about running buildhistory-diff. Outside of a mention in the migration section, which probably does not apply, that is the only area where buildhistory-diff is specifically shown. Would a "NOTE" be sufficient here? One thing I thought was going to happen with post 2.1 is that Python 3 would be part of the release. If so, wouldn't this issue go away for future releases of YP? Scott I think NOTE should be enough. Would something like this be ok? Buildhistory-diff requries GitPython package. Please, install it with pip3: pip3 install GitPython --user Or install python3-git package using distro package manager(apt-get, dnf or zipper) > One thing I thought was going to happen with post 2.1 is that Python 3 would be part of the release. If so, wouldn't this issue go away for future releases of YP?
No it wouldn't as it was caused by the move to Python 3.
Hi, Check out the new note at http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#examining-build-history-information and let me know if this is okay. Thanks, Scott Oops... I just realized I am not the owner of this bug. I did change the state Ed. Sorry. Scott Thank you for updating the documentation! The change looks good to me. There is just one small typo in there: "Alternatively, you can install python3-3" should be "Alternatively, you can install python3-git" Jussi, are you ok with the change? (In reply to comment #7) > Jussi, are you ok with the change? Sure. a similar issue: https://bugzilla.yoctoproject.org/show_bug.cgi?id=9690 Ed, Don't know how I managed that typo :) Fixed it - see http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#examining-build-history-information Thanks, Scott Thanks... I am putting the doc flag to Done. It was set at "No" but it did involve the docs. Scott Thank you! |