8684 2015-11-12 00:27:40 +0000 Missing python-ast package in generate-menifest-2.7.py 2016-03-18 17:40:37 +0000 1 1 7 Build System, Metadata & Runtime OE-Core devtools / tool chain 2.0 Other Multiple RESOLVED FIXED Medium major 2.1 M3 1 kei ross.burton benjamin.esquivel kei laurent.bonnans meta.mr.watcher meta.watcher ross.burton --- --- No (bug/feature does not impact docs) oldest_to_newest 56405 0 kei 2015-11-12 00:27:40 +0000 scripts/contrib/python/generate-manifest-2.7.py doesn't expose the python-ast, Abstruct Syntax Trees[1], package. One of our OpenSwitch[2] component, ops-ansible[3], requires that package to make the ansible python scripts run on our OpenSwitch target. We mirror the poky environment and want to upstream the changes as much as possible, so that we can easily downstream it into our build system. [1] https://docs.python.org/2/library/ast.html [2] http://www.openswitch.net/ [3] https://review.openswitch.net/#/c/2260/1/yocto/openswitch/meta-distro-openswitch/recipes-ops/mgmt/ops-ansible.bb 56408 1 kei 2015-11-12 01:10:25 +0000 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 56409 2 kei 2015-11-12 02:12:57 +0000 Added myself in Cc list. 56411 3 ross.burton 2015-11-12 10:05:18 +0000 That's certainly the right approach to splitting it out, but it is currently packaged in the "everything else" package, python-misc. 56449 4 kei 2015-11-12 19:09:50 +0000 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. 56450 5 ross.burton 2015-11-12 19:16:23 +0000 Patches need to go do the list to be reviewed in public, so can you resend that mail to the mailing list? 56452 6 kei 2015-11-12 22:09:01 +0000 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? 56455 7 kei 2015-11-13 02:10:33 +0000 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. 59732 8 laurent.bonnans 2016-03-04 13:42:34 +0000 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. 59797 9 ross.burton 2016-03-08 17:02:28 +0000 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. 60039 10 ross.burton 2016-03-16 21:58:48 +0000 oe-core 48bc643a4ebe74d0fa49fedbe7b0fd63fd0003e8 moves the ast libraries into -core. 60166 11 kei 2016-03-18 17:40:37 +0000 Thank you for taking care of this! Really appreciated! Cheers,