- Explain why the tests are conditioned around

- Specify all the runtime dependencies based on setup.py

OBS-URL: https://build.opensuse.org/package/show/devel:languages:python/python-relatorio?expand=0&rev=15

python-relatorio.changes

@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Mon Dec 11 09:55:31 UTC 2017 - tchvatal@suse.com
+
+- Explain why the tests are conditioned around
+- Specify all the runtime dependencies based on setup.py
+
 -------------------------------------------------------------------
 Sat Dec  9 16:12:18 UTC 2017 - axel.braun@gmx.de
 

python-relatorio.spec

@@ -17,10 +17,8 @@
 #
 
 
-%bcond_with tests
-
 %{?!python_module:%define python_module() python-%{**} python3-%{**}}
-
+%bcond_with tests
 %define         mod_name relatorio
 Name:           python-relatorio
 Version:        0.8.0
@@ -32,11 +30,14 @@ Url:            https://pypi.python.org/pypi/relatorio
 Source:         https://pypi.io/packages/source/r/%{mod_name}/%{mod_name}-%{version}.tar.gz
 BuildRequires:  %{python_module Genshi}
 BuildRequires:  %{python_module lxml}
+# It requires different magic for tests https://github.com/ahupp/python-magic
+#BuildRequires:  %{python_module magic}
 BuildRequires:  %{python_module setuptools}
 BuildRequires:  python-rpm-macros
+Requires:       python-Genshi
 Requires:       python-PyYAML
+Requires:       python-lxml
 Requires:       python-pycha
-BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 BuildArch:      noarch
 %python_subpackages
 
@@ -60,7 +61,6 @@ as documents (odt, ods, pdf) or images (png, svg).
 %endif
 
 %files %{python_files}
-%defattr(-,root,root)
 %doc README AUTHORS LICENSE
 %{python_sitelib}/*