[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: playground/pim/krss
From:       Frank Osterfeld <frank () kdab ! net>
Date:       2008-05-06 9:10:24
Message-ID: 200805061110.25174.frank () kdab ! net
[Download RAW message or body]

On Tuesday 06 May 2008 10:48:31 Dmitry Ivanov wrote:
> On 5/5/08, Frank Osterfeld <frank@kdab.net> wrote:
> > On Saturday 03 May 2008 17:18:36 Dmitry Ivanov wrote:
> > > SVN commit 803633 by divanov:
> > >
> > > Initial version of the Akonadi RSS resource:
> > >  - parses an OPML document
> > >  - builds the collection tree
> > >  - fetches feeds
> > >  - (de)serialization works
> > >
> > >
> > >  A             CMakeLists.txt
> > >  A             kfeed_private (directory)
> > >  A             kfeed_private/category.h   [License: LGPL (v2+)]
> > >  A             kfeed_private/enclosure.h   [License: LGPL (v2+)]
> > >  A             kfeed_private/item.h   [License: LGPL (v2+)]
> > >  A             kfeed_private/itemserializer.h   [License: LGPL (v2+)]
> > >  A             kfeed_private/kfeed_export.h   [License: LGPL (v2+)]
> > >  A             kfeed_private/person.h   [License: LGPL (v2+)]
> >
> > I think we should move kfeed/ to playground instead of duplicating it. It
> > won't have users before akonadi/rss is ready and in use anyway.
> > If we move it to playground, we can also install the headers.
> >
> > --
> > Frank Osterfeld -- frank@kdab.net
> > Klarälvdalens Datakonsult AB, Platform-independent software solutions
>
> So, I svn move it to playground and re-enable the cmake command
> to install the headers?

Yes, go ahead. Removing the kfeed/ directory and the respective entry in the 
toplevel CMakeLists.txt should be everything needed in kdepim/. Just make 
sure you svn copy/move it, so we don't lose the history.

-- 
Frank Osterfeld -- frank@kdab.net
Klarälvdalens Datakonsult AB, Platform-independent software solutions

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic