On Thu, Jul 12, 2012 at 10:37 PM, Marco Martin <notm...@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
> nepo...@kde.org
> https://mail.kde.org/mailman/listinfo/nepomuk
>
--
Vishesh Handa
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
