3768 2013-01-22 07:45:09 +0000 smart: --yaml option doesn't work 2013-02-16 19:23:11 +0000 1 1 11 Runtime Package Management Issues package-management-issues 1.4 x86 Multiple RESOLVED FIXED Medium normal 1.4 M4 1 yi.zhao bluelightning bluelightning bogdan.a.marinescu hongxu.jia liang.li2 mark.hatle Qi.Chen --- --- --- oldest_to_newest 29014 0 yi.zhao 2013-01-22 07:45:09 +0000 Image location: http://autobuilder.yoctoproject.org/pub/nightly/20130115-2/ Tree/Branch: poky/master Git Rev: 9eb88ceb39b7d0b8ddc6487e61ce8edadef10ec4 Some smart commands have the --yaml option, but this option doesn't work. ############## root@qemumips:~# smart channel --yaml Traceback (most recent call last): File "/usr/bin/smart", line 200, in <module> main(sys.argv[1:]) File "/usr/bin/smart", line 173, in main exitcode = iface.run(opts.command, opts.argv) File "/usr/lib/python2.7/site-packages/smart/interface.py", line 53, in run result = _command.main(self._ctrl, opts) File "/usr/lib/python2.7/site-packages/smart/commands/channel.py", line 342, in main import yaml ImportError: No module named yaml root@qemumips:~# root@qemumips:~# smart config --yaml Traceback (most recent call last): File "/usr/bin/smart", line 200, in <module> main(sys.argv[1:]) File "/usr/bin/smart", line 173, in main exitcode = iface.run(opts.command, opts.argv) File "/usr/lib/python2.7/site-packages/smart/interface.py", line 53, in run result = _command.main(self._ctrl, opts) File "/usr/lib/python2.7/site-packages/smart/commands/config.py", line 140, in main import yaml ImportError: No module named yaml root@qemumips:~# ############## 29019 1 bluelightning 2013-01-22 08:55:34 +0000 This is simply because the yaml python module isn't installed. I wouldn't consider this core functionality, so the question is do we want to add a dependency just for the sake of enabling this option? 29181 2 bluelightning 2013-01-25 10:55:34 +0000 Changing to NEEDINFO, see my above comment 29208 3 mark.hatle 2013-01-25 17:31:20 +0000 yaml is a pretty common way of exporting information from an RPM repository. We likely should package up the yaml python module.. and then add a test in smart to see if it is installed. That way it works if the user needs it, but it's disabled (to save space) for the normal case. 29209 4 bluelightning 2013-01-25 17:42:16 +0000 Sounds like a sensible approach to me - thanks. 29669 5 bluelightning 2013-02-08 09:46:49 +0000 It turns out there's a python-pyyaml recipe in meta-oe, it just needed an update; I have sent a patch to update it and the libyaml recipe that is also there. I've also sent an OE-Core patch to ask the user to install PyYAML if it can't be imported instead of printing a python backtrace. 29711 6 bluelightning 2013-02-11 15:28:52 +0000 The OE-Core/Poky part of this patch is merged: http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=0d85fa531b03f0e7b3ad3d2a5a4d92293b43b8ff The meta-oe part is still pending. 29880 7 bluelightning 2013-02-16 19:23:11 +0000 libyaml/python-pyyaml changes have now been merged into meta-oe: http://cgit.openembedded.org/meta-openembedded/commit/?id=9c824550785d96dda0e6aeb4a2a37bb2c30dc125 http://cgit.openembedded.org/meta-openembedded/commit/?id=f976a44387f45ac5ede7d74d511bef1228599597