[Nepomuk] porting stuff to Nepomuk2

Vishesh Handa me at vhanda.in
Wed Jul 18 08:02:16 UTC 2012


On Thu, Jul 12, 2012 at 10:37 PM, Marco Martin <notmart at gmail.com> wrote:

> On Wednesday 11 July 2012, Vishesh Handa wrote:
> > > > uses nepomuk2_rcgen. Cause I don't think the packagers would like us
> > >
> > > having
> > >
> > > > 2 pacakges (nepomuk-core and kdelibs) install the same cmake file.
> > >
> > > i added it now in the nepomuk-core repo, same mart/rcgenNepomuk2 branch
> >
> > great.
> >
> > If you could just test it out a bit more, then we can merge it into
> master.
> >
>
> i ported the share-like-connect repo to nepomuk2 as well, so far so good
> (even
> tough will need setsymbols again)
>
> now only takes some volunteer porting something else that was using
> NepomukAddOntologyClasses to be really safe :p
>

Hey Marco

I decided to test out the rcgen2, and found some minor problems -

* Resource::resourceUri is now uri() - My fault. I made this change in
nepomuk-core. FIXED.
* Missing "resource.h" when trying to compile the generated classes.

Does it work for you?

This simple patch fixed it for me, but I'm not sure if I should commit it.

diff --git a/rcgen/codegenerator.cpp b/rcgen/codegenerator.cpp
index 8c5d9b5..e45c7b0 100644
--- a/rcgen/codegenerator.cpp
+++ b/rcgen/codegenerator.cpp
@@ -155,7 +155,7 @@ bool CodeGenerator::writeHeader( const ResourceClass
*resourceClass, QTextStream
         s.replace( "NEPOMUK_PARENT_INCLUDE", QString("\"%1.h\"").arg(
parent->name().toLower() ) );
     }
     else {
-        s.replace( "NEPOMUK_PARENT_INCLUDE", QString("\"resource.h\"") );
+        s.replace( "NEPOMUK_PARENT_INCLUDE",
QString("<Nepomuk2/Resource>") );
     }

     QString methods;



> Cheers,
> Marco Martin
> _______________________________________________
> Nepomuk mailing list
> Nepomuk at kde.org
> https://mail.kde.org/mailman/listinfo/nepomuk
>



-- 
Vishesh Handa
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120718/a0a5dcbc/attachment.html>


More information about the Plasma-devel mailing list