[Bug 960839] Review Request: lang-table - For guessing reasonable defaults for locale, keyboard, territory, …

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960839

--- Comment #1 from Mike FABIAN <mfabian@xxxxxxxxxx> ---
BuildRequires:  python-lxml
was missing, therefore the %check section always failed for
me when building with “mock”. Jens Petersen found the reason.
Fixed it like as below.

Spec URL: http://mfabian.fedorapeople.org/lang-table/lang-table.spec
SRPM URL:
http://mfabian.fedorapeople.org/lang-table/lang-table-0.0.1-1.fc18.src.rpm

updated.

diff --git a/lang-table.spec b/lang-table.spec
index 76f25d0..582bd73 100644
--- a/lang-table.spec
+++ b/lang-table.spec
@@ -9,6 +9,7 @@ License:        GPLv3+
 URL:            https://github.com/mike-fabian/lang-table
 Source0:       
http://mfabian.fedorapeople.org/lang-table/%{name}-%{version}.tar.gz
 BuildArch:      noarch
+BuildRequires:  python-lxml
 BuildRequires:  python-devel

 %description
@@ -23,6 +24,7 @@ Summary:        Python module to query the lang-table-data
 Group:          Development/Tools
 Requires:       %{name} = %{version}-%{release}
 Requires:       %{name}-data = %{version}-%{release}
+Requires:       python-lxml

 %description python
 This package contains a Python module to query the data
@@ -47,7 +49,7 @@ perl -pi -e "s%datadir = '(.*)'%datadir =
'%{_datadir}/lang-table'%" langtable.p
 %{__python} setup.py install --skip-build --prefix=%{_prefix}
--install-data=%{_datadir}/lang-table --root $RPM_BUILD_ROOT

 %check
-#(cd $RPM_BUILD_DIR/%{name}-%{version}; %{__python} -m doctest test_cases.txt)
+(cd $RPM_BUILD_DIR/%{name}-%{version}; %{__python} -m doctest test_cases.txt)

 %files
 %doc README COPYING ChangeLog unicode-license.txt test_cases.txt

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MfA8o9LQqK&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]