https://bugzilla.redhat.com/show_bug.cgi?id=2005132

Ankur Sinha (FranciscoD) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(sanjay.ankur@gmai |
                   |l.com)                      |



--- Comment #2 from Ankur Sinha (FranciscoD) <[email protected]> ---
Hi Ben,

thanks for the review.

I've dropped the doc sub-package for the moment, and enabled the import test.

Updated spec/srpm:

https://ankursinha.fedorapeople.org/python-hdf5storage/python-hdf5storage.spec
https://ankursinha.fedorapeople.org/python-hdf5storage/python-hdf5storage-0.1.18-2.fc36.src.rpm

Cheers,
Ankur

diff --git a/python-hdf5storage.spec b/python-hdf5storage.spec
index 25a4ab2..9fce3ed 100644
--- a/python-hdf5storage.spec
+++ b/python-hdf5storage.spec
@@ -47,16 +47,6 @@ BuildRequires:  %{py3_dist nose}

 %description -n python3-hdf5storage %_description

-%package doc
-Summary:        Documentation for %{name}
-BuildRequires:  make
-BuildRequires:  %{py3_dist sphinx}
-BuildRequires:  %{py3_dist numpydoc}
-BuildRequires:  %{py3_dist sphinx-rtd-theme}
-
-%description doc
-This package provides documentation for %{name}.
-
 %prep
 %forgesetup

@@ -65,16 +55,12 @@ This package provides documentation for %{name}.
 # find . -type f -name "*.py" -exec sed -i '/^#![  ]*\/usr\/bin\/env.*$/ d' {}
2>/dev/null ';'

 %generate_buildrequires
-%pyproject_buildrequires %{?with_tests:-r}
+%pyproject_buildrequires -r


 %build
 %pyproject_wheel

-# Sometimes needed so sphinx can import the module
-PYTHONPATH=".:.." make -C doc SPHINXBUILD=sphinx-build-3 html
-rm -rf doc/build/html/{.doctrees,.buildinfo} -vf
-
 %install
 %pyproject_install
 %pyproject_save_files hdf5storage
@@ -82,15 +68,11 @@ rm -rf doc/build/html/{.doctrees,.buildinfo} -vf
 %check
 %if %{with tests}
 %{pytest}
-# or %%{tox}
 %endif
+%pyproject_check_import hdf5storage

 %files -n python3-hdf5storage -f %{pyproject_files}
 %doc README.rst

-%files doc
-%license COPYING.txt
-%doc doc/build/html
-
 %changelog
 %autochangelog


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2005132
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to