Bug 1785982 - Review Request: python-qdarkstyle - A dark stylesheet for Python and Qt applications
Summary: Review Request: python-qdarkstyle - A dark stylesheet for Python and Qt appli...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 1666545 (view as bug list)
Depends On: 1785974 1785977 1906117
Blocks: 1666513
TreeView+ depends on / blocked
 
Reported: 2019-12-22 22:55 UTC by Mukundan Ragavan
Modified: 2021-07-09 11:28 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-03-19 20:06:17 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+


Attachments (Terms of Use)

Description Mukundan Ragavan 2019-12-22 22:55:34 UTC
Spec URL: https://nonamedotc.fedorapeople.org/pkgreview/python-packages/2019-2020/qdarkstyle/rnd1/python-qdarkstyle.spec
SRPM URL: https://nonamedotc.fedorapeople.org/pkgreview/python-packages/2019-2020/qdarkstyle/rnd1/python-qdarkstyle-2.7-1.fc31.src.rpm

Description:
A dark stylesheet for Qt applications (Qt4, Qt5, PySide, PySide2, PyQt4, 
PyQt5, QtPy, PyQtGraph).

Fedora Account System Username: nonamedotc

Comment 1 Mukundan Ragavan 2019-12-22 23:03:30 UTC
This package is complicated by python-watchdog required pathtools, which has been retired.

Also, python-importlib-metadata is missing in rawhide at the moment.

https://copr.fedorainfracloud.org/coprs/nonamedotc/spyder4_rawhide/build/1134530/

Comment 2 Zbigniew Jędrzejewski-Szmek 2019-12-23 16:14:39 UTC
I'll review this once the two reviews this depends on move forward.

Comment 3 Zbigniew Jędrzejewski-Szmek 2020-03-16 17:26:53 UTC
Hmm, builds fails with:

+ /usr/bin/python3 setup.py test
running test
WARNING: Testing via this command is deprecated and will be removed in a future version. Users looking for a generic test entry point independent of test runner are encouraged to use tox.
running egg_info
writing QDarkStyle.egg-info/PKG-INFO
writing dependency_links to QDarkStyle.egg-info/dependency_links.txt
writing entry points to QDarkStyle.egg-info/entry_points.txt
writing requirements to QDarkStyle.egg-info/requires.txt
writing top-level names to QDarkStyle.egg-info/top_level.txt
reading manifest file 'QDarkStyle.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: no files found matching 'PRODUCTION.md'
writing manifest file 'QDarkStyle.egg-info/SOURCES.txt'
running build_ext
pyside_style_rc (unittest.loader._FailedTest) ... ERROR

======================================================================
ERROR: pyside_style_rc (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: pyside_style_rc
Traceback (most recent call last):
  File "/usr/lib64/python3.8/unittest/loader.py", line 154, in loadTestsFromName
    module = __import__(module_name)
  File "/builddir/build/BUILD/QDarkStyle-2.7/qdarkstyle/pyside_style_rc.py", line 10, in <module>
    from PySide import QtCore
ModuleNotFoundError: No module named 'PySide'

python3-pyside has been retired ;(
https://src.fedoraproject.org/rpms/python-pyside/c/73d32307db2c8b1d56c6705f416fc954436dc8f6?branch=master

Comment 4 Mukundan Ragavan 2020-03-17 00:20:48 UTC
Spec URL: https://nonamedotc.fedorapeople.org/pkgreview/python-packages/2019-2020/qdarkstyle/rnd2/python-qdarkstyle.spec
SRPM URL: https://nonamedotc.fedorapeople.org/pkgreview/python-packages/2019-2020/qdarkstyle/rnd2/python-qdarkstyle-2.8-1.fc31.src.rpm


- Packaged latest upstream version (v2.8)

It would be best to use tox for testing. But, not sure how to get this to work. Open to suggestions. Thanks.

Comment 5 Zbigniew Jędrzejewski-Szmek 2020-03-17 11:02:07 UTC
> Ran 0 tests in 0.000s                                                                                                                             

Yep, current test invocation doesn't seem to work.

I tried running the tests in mock, and I couldn't get it to work either.
%tox expands to "... tox --current-env ...", and tox doesn't seem to know this option.

When I remove the option, it can't find virtualenv:
+ tox -q --recreate -e py38
...
ERROR: No matching distribution found for virtualenv

but the python3-virtualenv is installed.

+ package name is OK
+ license is acceptable for Fedora (MIT)
+ license is specified correctly (according to setup.py)
+ builds and install OK
+ latest version
+ R/P/BR look OK

Package is approved.

Comment 6 Igor Raits 2020-03-18 10:15:49 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/python-qdarkstyle

Comment 7 Mukundan Ragavan 2020-03-19 20:06:17 UTC
Thanks for the review. Built on rawhide.

Comment 8 Jonny Heggheim 2021-07-09 11:28:27 UTC
*** Bug 1666545 has been marked as a duplicate of this bug. ***


Note You need to log in before you can comment on or make changes to this bug.