| Summary: | Missing python-ast package in generate-menifest-2.7.py | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] OE-Core | Reporter: | Kei Nohguchi <kei> |
| Component: | devtools / tool chain | Assignee: | Ross Burton <ross.burton> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | major | ||
| Priority: | Medium | CC: | benjamin.esquivel, kei, laurent.bonnans, meta.mr.watcher, meta.watcher, ross.burton |
| Version: | 2.0 | ||
| Target Milestone: | 2.1 M3 | ||
| Hardware: | Other | ||
| OS: | Multiple | ||
| Whiteboard: | |||
| OS type for building Yocto: | --- | Type of Regression: | --- |
| Verified: | Documentation change: | No (bug/feature does not impact docs) | |
|
Description
Kei Nohguchi
2015-11-12 00:27:40 UTC
Here is the actual change request against our oe/poky mirror. https://review.openswitch.net/#/c/2251/ As indicated in one of the review comment, I'm now preparing to send the email to the OE maintainer but please let me know if I'm not the right track. Cheers, Kei Added myself in Cc list. That's certainly the right approach to splitting it out, but it is currently packaged in the "everything else" package, python-misc. Thank you, Ross, for confirming the change. I've also sent a patch to Richard Purdie <richard.purdie@linuxfoundation.org> of OE. I'll wait for his reply. Kei Nohguchi <kei@nohguchi.com> To: Richard Purdie <richard.purdie@linuxfoundation.org> Cc: Kei Nohguchi <kei@nohguchi.com> [PATCH] Fixes [YOCTO #8684] Add python-ast, Python Abstruct Syntax Tree package In a meanwhile, I'll play with python-misc to get things done. :) Cheers and have a wonderful day today! (In reply to comment #3) > That's certainly the right approach to splitting it out, but it is currently > packaged in the "everything else" package, python-misc. Patches need to go do the list to be reviewed in public, so can you resend that mail to the mailing list? Thank you Ross! I've sent the patch to the OE-core, as below. http://lists.openembedded.org/pipermail/openembedded-core/2015-November/112694.html Cheers, (In reply to comment #5) > Patches need to go do the list to be reviewed in public, so can you resend > that mail to the mailing list? Hi Ross, I've tried this python-misc but when I made a image with python-misc package, instead of the python-ast package, I got different error saying "ImportError: No module named compiler"... Since this is specific some ansible module issue, I think I'll work on OE-core and make the python-ast officially supported by the OE. The patch is updated, as suggested by Chris Larson, and hopefully will be approved by OE-core. http://patchwork.openembedded.org/patch/107551/ http://patchwork.openembedded.org/patch/107499/ Just FYI. debug1: client_input_global_request: rtype hostkeys-00@openssh.com want_reply 0 debug1: Sending command: /bin/sh -c 'LANG=C LC_CTYPE=C python' debug1: mux_client_request_session: master session id: 2 Traceback (most recent call last): File "<stdin>", line 242, in <module> ImportError: No module named compiler debug1: client_input_channel_req: channel 2 rtype exit-status reply 0 debug1: channel 2: free: client-session, nchannels 3 debug1: channel 1: free: mux-control, nchannels 2 (In reply to comment #3) > That's certainly the right approach to splitting it out, but it is currently > packaged in the "everything else" package, python-misc. Hello, As the 'ast' module contains a small number of helpers for the low level '_ast' which is a builtin in python (https://docs.python.org/2/library/ast.html), perhaps it would suffice to add it to the core modules ('python-core') instead of adding another small standalone package. Also, it should probably be added to python3.x as well. A package for a single small source file that is basically required to use a builtin function doesn't seem very sensible. I agree that it should be moved to core, patch incoming. oe-core 48bc643a4ebe74d0fa49fedbe7b0fd63fd0003e8 moves the ast libraries into -core. Thank you for taking care of this! Really appreciated! Cheers, |