Spec URL: https://heliocastro.fedorapeople.org/qt5-qtdeclarative-render2d/qt5-qtdeclarative-render2d.spec SRPM URL: https://heliocastro.fedorapeople.org/qt5-qtdeclarative-render2d/qt5-qtdeclarative-render2d-5.7.0-2.fc24.src.rpm Description: Qt5 - QtDeclarative component. Fedora Account System Username: heliocastro
Note that this will become obsolete with Qt 5.8, where the 2D renderer becomes part of qt5-qtdeclarative itself.
Also please fix your spec URL, which gives me an error 404.
The URL reflects the official URL. Packages just not moved to there yet ( will be today ) New source: SPEC URL: https://heliocastro.fedorapeople.org/qt5-qtdeclarative-render2d/qt5-qtdeclarative-render2d.spec SRPM URL: https://heliocastro.fedorapeople.org/qt5-qtdeclarative-render2d/qt5-qtdeclarative-render2d-5.7.1-1.fc25.src.rpm
naming: ok license: ok sources: ok (for now, 5.7.1 tarballs not officially released yet) macros: ok scriptlets: ok follows style/conventions of other qt5- packages 1. dir %{_qt5_plugindir}/scenegraph/ is unowned, I'd suggest changing %{_qt5_plugindir}/scenegraph/libsoftwarecontext.so to %{_qt5_plugindir}/scenegraph/ in %%files list 2. If -devel pkg is *just* for cmake files, having a separate -devel pkg is probably not needed imho. SHOULD drop -devel, and include this content in the main pkg. And maybe add something like Obsoletes: qt5-qtdeclarative-render2d-devel < 5.7.1-1 Provides: qt5-qtdeclarative-render2d-devel = %{version}-%{release} if you want APPROVED
Assigning the right approver.
confirmed imported, closing