https://bugzilla.redhat.com/show_bug.cgi?id=2025395
--- Comment #2 from Jakub Kadlčík jkadlcik@redhat.com --- Thank you for the review Ben,
- The installation process does not preserve timestamps. Please add the “-p”
option to “cp” in %install.
Fixed
- I’m noting that /usr/share/rofi/ and /usr/share/rofi/themes/ are owned by
rofi-themes, which is a dependency of rofi, which this package depends on. Directory ownership requirements are thus indirectly satisfied.
To be sure, I added a dependency to rofi-themes, so the directory ownership isn't satisfied transitively now. Adding /usr/share/rofi/ into %files can work as well but I would rather do it once somebody has a use-case for this package without installing rofi and rofi-themes.
Spec URL: https://download.copr.fedorainfracloud.org/results/frostyx/rofi-themes-base1... SRPM URL: https://download.copr.fedorainfracloud.org/results/frostyx/rofi-themes-base1...