Rex Dieter wrote:
> This feels more like a "patch" operation to me, so wouldn't putting this
> in %prep (rather than %build) somewhere make more sense?
Uh, you're right, putting it in %prep makes sense. For some reason, when I
wrote this, I was thinking somehow that using the tools from our
BuildRequires (kdelibs4-devel) in %prep isn't such a good thing, but now
that I think of it, why not? There are also other packages using custom
BuildRequired tools in %prep (decompressors, sed-like tools, scripting
languages etc.), so this isn't all that different. And this is indeed a
patch-like operation.
> Thinking a bit more... would it be too evil to macro'ize this somehow, say
> hook into %cmake_kde for example, to operate automatically on all
> metadata.desktop files found in $RPM_BUILD_DIR ?
Interesting idea. But not all packages use metadata.desktop, especially the
native plasmoids sometimes use other file names.
Kevin Kofler
_______________________________________________
kde mailing list
kde@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/kde
New to KDE4? - get help from http://userbase.kde.org
