Re: [Rhythmbox-devel] [patch] devel rpm package



Hi,
Yeah, I will continue maintaining the spec unless William has a burning
desire to take over :) 

I am going away for the weekend but I look at integrating this patch
during when I come back. Unless William gets to it first :)

Christian

On Fri, 2003-08-15 at 03:07, Colin Walters wrote:
> On Thu, 2003-08-14 at 19:40, Alex wrote:
> > This patch splits off the header, the pkgconfig file, and some docs to a
> > devel rpm package.
> 
> I don't like this patch.  First of all, you moved around Rhythmbox.h,
> and not Rhythmbox.idl, which is the critical component.
> In fact, Rhythmbox.h should not be installed at all as a system header;
> for example, it is not useful for applications which use other ORBs.
> 
> Secondly, I just don't feel that there's a lot of gain by splitting off
> such a tiny package.
> 
> Anyways though, these are just my off-the-cuff opinions; I do not
> personally have strong feelings about the .spec file included in rb. 
> Christian Schaller wrote it originally, and hopefully he will be
> interested in continuing to maintain it, and will be able to comment on
> your patch.  If not, William McCann was the last person to modify it,
> and perhaps he would be willing to maintain it.
> 
> (Christian, William: I have re-attached Alex's patch for your
> convenience)
> 




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]