Fedora Account System
Red Hat Associate
Red Hat Customer
Spec URL: https://solomoncyj.fedorapeople.org/reviews/dds10-thumbnailer-kde/dds10-thumbnailer-kde.spec SRPM URL: https://solomoncyj.fedorapeople.org/reviews/dds10-thumbnailer-kde/dds10-thumbnailer-kde-20240316.23b2801-1.20260714git23b2801.fc45.src.rpm Description: A KDE thumbnail creator for DDS image format Fedora Account System Username: solomoncyj
Some early review notes: I believe the Version field needs to be blank here, otherwise as you can see in the naming of your srpm, it adds version information twice: https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/#_commit_example Upstream recommends getting qt6-core and qt6-gui. It's possible those are being brought in by something else, and those requirements are in the CMakeLists.txt, so I recommend uncommenting them as, if something in the stack changes and those are no longer automatically brought in, building would fail. Secondly, do consider using the cmake() names for the devel packages. For example: ``` #BuildRequires: qt6-core-devel #BuildRequires: qt6-gui-devel BuildRequires: kf6-kio-devel ``` Would become ``` #BuildRequires: cmake(Qt6Core) #BuildRequires: cmake(Qt6Gui) BuildRequires: cmake(KF6KIO) ``` Third, usually, if you build using the KDE stack and ECM, you do want to use %cmake_kf6 instead of %cmake.
Oh also, and this is rather new, consider putting the %cmake_kf6 argument in the %conf section of the spec. That should be right between %prep and %build. See: https://docs.fedoraproject.org/en-US/packaging-guidelines/KDEPackaging/#_available_macros
Once you did all that, I suspect you won't need to use -DKDE_INSTALL_USE_QT_SYS_PATHS=ON at least. Not sure about the ninja argument.
Spec URL: https://solomoncyj.fedorapeople.org/reviews/dds10-thumbnailer-kde/dds10-thumbnailer-kde.spec SRPM URL: https://solomoncyj.fedorapeople.org/reviews/dds10-thumbnailer-kde/dds10-thumbnailer-kde-0^20240316git23b2801-1.fc44.src.rpm - Uncommitted changes