Fedora Account System
Red Hat Associate
Red Hat Customer
Spec URL: https://scrye.com/~kevin/fedora/review/PyQt-builder/PyQt-builder.spec SRPM URL: https://scrye.com/~kevin/fedora/review/PyQt-builder/PyQt-builder-1.6.0-1.fc34.src.rpm Description: PyQt-builder is the PEP 517 compliant build system for PyQt and projects that extend PyQt. It extends the sip build system and uses Qt's qmake to perform the actual compilation and installation of extension modules.Projects that use PyQt- builder provide an appropriate pyproject.toml file and an optional project.py. Fedora Account System Username: kevin I need this for recent versions of calibre. rpmlint says: 0 packages and 1 specfiles checked; 0 errors, 0 warnings.
The license of the package is correct and appropriated for Fedora. The spec file is simple and correct. My main issue is with the dlls bundle. What is the license of those files? Are those the preferred way to build them... Do we need them in linux and in Fedora in particular?
Good question. Those dll files are for pyqt-bundler which allows someone to bundle os stuff in their windows binary. Some of them are openssl, and some of them are microsoft visual basic (redistributable). They aren't made from source, and I don't think too many people will be using this package on linux to build and bundle windows binaries, so I am fine with just dropping them. I did look and opensuse ships them. Debian doesn't seem to package pyqt-builder at all, so no help there. New version, with dlls removed: Spec URL: https://scrye.com/~kevin/fedora/review/PyQt-builder/PyQt-builder.spec SRPM URL: https://scrye.com/~kevin/fedora/review/PyQt-builder/PyQt-builder-1.6.0-2.fc34.src.rpm
(In reply to Kevin Fenzi from comment #2) > Good question. Those dll files are for pyqt-bundler which allows someone to > bundle os stuff in their windows binary. Some of them are openssl, and some > of them are microsoft visual basic (redistributable). They aren't made from > source, and I don't think too many people will be using this package on > linux to build and bundle windows binaries, so I am fine with just dropping > them. > > I did look and opensuse ships them. Debian doesn't seem to package > pyqt-builder at all, so no help there. > > New version, with dlls removed: > > Spec URL: > https://scrye.com/~kevin/fedora/review/PyQt-builder/PyQt-builder.spec > SRPM URL: > https://scrye.com/~kevin/fedora/review/PyQt-builder/PyQt-builder-1.6.0-2. > fc34.src.rpm My view point was from the license angle, at most that status should have been declared in the spec file. :-) Something like BSD and Redistributable or something equivalent. Honestly I trust your judgement here. :-) The package is approved.
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/PyQt-builder
Built in rawhide.