From 53a105a997cfbfae2525454f667b92b4efa3ffa42cc4bfa2430dadbf82df1d7f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1=C5=A1=20Chv=C3=A1tal?= <tchvatal@suse.com>
Date: Tue, 19 Feb 2019 15:10:21 +0000
Subject: [PATCH] - Format with spec-cleaner - Remove BuildRoot - Add
 BuildArch: noarch - Explain why we skip a test

OBS-URL: https://build.opensuse.org/package/show/devel:languages:python/python-eliot?expand=0&rev=2
---
 python-eliot.changes |  8 ++++++++
 python-eliot.spec    | 46 +++++++++++++++++++++-----------------------
 2 files changed, 30 insertions(+), 24 deletions(-)

diff --git a/python-eliot.changes b/python-eliot.changes
index 37045ce..b04eeff 100644
--- a/python-eliot.changes
+++ b/python-eliot.changes
@@ -1,3 +1,11 @@
+-------------------------------------------------------------------
+Tue Feb 19 15:09:53 UTC 2019 - Tomáš Chvátal <tchvatal@suse.com>
+
+- Format with spec-cleaner
+- Remove BuildRoot
+- Add BuildArch: noarch
+- Explain why we skip a test
+
 -------------------------------------------------------------------
 Thu Feb  7 10:37:52 UTC 2019 - ecsos@opensuse.org
 
diff --git a/python-eliot.spec b/python-eliot.spec
index 32d76e8..71246b3 100644
--- a/python-eliot.spec
+++ b/python-eliot.spec
@@ -12,7 +12,7 @@
 # license that conforms to the Open Source Definition (Version 1.9)
 # published by the Open Source Initiative.
 
-# Please submit bugfixes or comments via http://bugs.opensuse.org/
+# Please submit bugfixes or comments via https://bugs.opensuse.org/
 #
 
 
@@ -23,41 +23,39 @@ Release:        0
 Summary:        Eliot: the logging system that tells you *why* it happened
 License:        Apache-2.0
 Group:          Development/Languages/Python
-Url:            https://github.com/itamarst/eliot/
+URL:            https://github.com/itamarst/eliot/
 Source0:        https://github.com/itamarst/eliot/archive/%{version}/%{name}-%{version}.tar.gz
-BuildRequires:  fdupes
-BuildRequires:  python-rpm-macros
-BuildRequires:  %{python_module devel}
-BuildRequires:  %{python_module six}
-BuildRequires:  %{python_module zope.interface}
-BuildRequires:  %{python_module boltons}
-BuildRequires:  %{python_module pyrsistent >= 0.11.8}
-BuildRequires:  %{python_module boltons}
+BuildRequires:  %{python_module Sphinx}
 BuildRequires:  %{python_module Twisted}
+BuildRequires:  %{python_module boltons}
 # extra
 BuildRequires:  %{python_module cffi >= 1.1.2}
+BuildRequires:  %{python_module coverage}
+BuildRequires:  %{python_module flake8}
+BuildRequires:  %{python_module hypothesis >= 1.14.0}
+BuildRequires:  %{python_module pyrsistent >= 0.11.8}
+BuildRequires:  %{python_module pytest}
 # dev
 BuildRequires:  %{python_module setuptools >= 40}
-BuildRequires:  %{python_module twine >= 1.12.1}
-BuildRequires:  %{python_module coverage}
-BuildRequires:  %{python_module hypothesis >= 1.14.0}
-BuildRequires:  %{python_module testtools}
-BuildRequires:  %{python_module Sphinx}
+BuildRequires:  %{python_module six}
 BuildRequires:  %{python_module sphinx_rtd_theme}
-BuildRequires:  %{python_module flake8}
+BuildRequires:  %{python_module testtools}
+BuildRequires:  %{python_module twine >= 1.12.1}
 BuildRequires:  %{python_module yapf}
-BuildRequires:  %{python_module pytest}
+BuildRequires:  %{python_module zope.interface}
+BuildRequires:  fdupes
+BuildRequires:  python-rpm-macros
+BuildArch:      noarch
+Requires:       python-boltons
+Requires:       python-pyrsistent >= 0.11.8
 Requires:       python-six
 Requires:       python-zope.interface
-Requires:       python-pyrsistent >= 0.11.8
-Requires:       python-boltons
-BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 %python_subpackages
 
 %description
 Eliot: the logging system that tells you *why* it happened.
-eliot is a Python logging system that outputs causal chains of actions: actions can spawn other actions, 
-and eventually they either succeed or fail. 
+eliot is a Python logging system that outputs causal chains of actions: actions can spawn other actions,
+and eventually they either succeed or fail.
 The resulting logs tell you the story of what your software did: what happened, and what caused it.
 
 %prep
@@ -77,10 +75,10 @@ export LC_CTYPE=en_US.UTF-8
 %python_install_alternative  eliot-prettyprint
 
 %check
-%python_expand PYTHONPATH=%%{buildroot}%{$python_sitelib} py.test-%{$python_bin_suffix} eliot/tests -k 'not prettyprint'
+# skip prettyprint as it needs the binary to execute
+%python_expand PYTHONPATH=%%{buildroot}%{$python_sitelib} py.test-%{$python_bin_suffix} -v eliot/tests -k 'not prettyprint'
 
 %files %{python_files}
-%defattr(-,root,root)
 %license LICENSE
 %doc README.rst
 %python_alternative %{_bindir}/eliot-prettyprint