rpms/kdebase/devel kdebase.spec,1.386,1.387

[Rex Dieter]

On 08/30/2009 05:17 AM, Than Ngo wrote:
> it looks wrong for me with the Requires on kde-plasma-folderview in kdebase-workspace.
>
> The current dependency situation in kdebase and kdebase-workspace:
>
> kdebase-workspace requires kde-plasma-folderview that is packaged as sub package
> and kdebase needs BR on kdebase-workspace in order to build! The dependency is wrong.
>
> It still works in rawhide because kdebase only requires kdebase-workspace-devel without version,
> it's also not correct, it has to requires kdebase-workspace-devel>=%{version}
>
> is there any reason why kde-plasma-folderview is packages as subpackage? and why does kdebase-workspace
> require kde-plasma-folderview?

Well, the dep is there, because we want the default kde desktop to 
include folderview, right?

This is less bad than before, where we had kdebase-workspace Requires: 
kdebase

I understand the risk of bootstrapping problems.

I really wish upstream didn't bundle plasma applets (esp arguably 
essential ones like folderview) in kdebase, but in kdebase-workspace 
instead. oh well.

Anyway, I'm open to suggestions on how to better achive what we want 
here.   Maybe drop the kde-plasma-folderview dep, and just add to comps 
instead?

-- Rex