crmsh-2.1.0-1.x86_64, python-modules - missing Requires

[Jan Rękorajski]

On Fri, 12 Dec 2014, Bartek SQ9MEV wrote:

> Hi there,
> 
> After installatoin crmsh-2.1.0-1.x86_64, when invoking crm i got:
> 
> # crm
> Fatal error:
>      No module named lxml
>      No module named modules
> 
> There's missing python-lxml requirement, attached patch fixes this.

Added and updated crmsh to 2.1.1, thanks.

> With python-lxml installed, i got another error:
> 
> # crm
> Fatal error:
>      No module named pdb
>      No module named modules
> 
> I've installed python-devel-tools (contains pdb.py), and crmsh works
> 
> Full trace:
> # python
> Python 2.7.8 (default, Jul 18 2014, 10:11:56)
> [GCC 4.8.3 20140522 (release)] on linux2
> Type "help", "copyright", "credits" or "license" for more information.
>  >>> from crmsh import main
> Traceback (most recent call last):
>    File "<stdin>", line 1, in <module>
>    File "/usr/lib64/python2.7/site-packages/crmsh/main.py", line 33, in 
> <module>
>      import ui_root
>    File "/usr/lib64/python2.7/site-packages/crmsh/ui_root.py", line 36, 
> in <module>
>      import ui_cib
>    File "/usr/lib64/python2.7/site-packages/crmsh/ui_cib.py", line 22, 
> in <module>
>      import xmlutil
>    File "/usr/lib64/python2.7/site-packages/crmsh/xmlutil.py", line 20, 
> in <module>
>      from lxml import etree, doctestcompare
>    File "/usr/lib64/python2.7/site-packages/lxml/doctestcompare.py", 
> line 41, in <module>
>    File "/usr/share/python2.7/doctest.py", line 99, in <module>
> ImportError: No module named pdb
>  >>>
> 
> /usr/share/python2.7/doctest.pyc belongs to 
> python-modules-2.7.8-1.x86_64, so i think python-modules should include 
> /usr/share/python2.7/pdb.pyc or python-modules should require 
> python-devel-tools, or python-lxml should require python-devel-tools.
> 
> Any toughts?

I see pdb is also imported by other modules in python-modules package,
so it only makes sense to move pdb to modules. Done, thanks for catching
this.

-- 
Jan Rękorajski                    | PLD/Linux
SysAdm | baggins<at>pld-linux.org | http://www.pld-linux.org/