From kde-active Wed Jul 18 08:14:16 2012 From: Vishesh Handa Date: Wed, 18 Jul 2012 08:14:16 +0000 To: kde-active Subject: Re: [Nepomuk] porting stuff to Nepomuk2 Message-Id: X-MARC-Message: https://marc.info/?l=kde-active&m=134259881316789 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============2522077743508405533==" --===============2522077743508405533== Content-Type: multipart/alternative; boundary=bcaec54c5392dc8e1404c5161464 --bcaec54c5392dc8e1404c5161464 Content-Type: text/plain; charset=ISO-8859-1 On Thu, Jul 12, 2012 at 10:37 PM, Marco Martin 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("") ); } QString methods; > Cheers, > Marco Martin > _______________________________________________ > Nepomuk mailing list > Nepomuk@kde.org > https://mail.kde.org/mailman/listinfo/nepomuk > -- Vishesh Handa --bcaec54c5392dc8e1404c5161464 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable

On Thu, Jul 12, 2012 at 10:37 PM, Marco = Martin <notmart@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 ma= ster.
>

i ported the share-like-connect repo to nepomuk2 as well, so far so g= ood (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 proble= ms -

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

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/codeg= enerator.cpp b/rcgen/codegenerator.cpp
index 8c5d9b5..e45c7b0 100644
--- a/rcgen/codegenerator.cpp
+++ b/rcg= en/codegenerator.cpp
@@ -155,7 +155,7 @@ bool CodeGenerator::writeHeader= ( const ResourceClass *resourceClass, QTextStream
=A0=A0=A0=A0=A0=A0=A0= =A0 s.replace( "NEPOMUK_PARENT_INCLUDE", QString("\"%1.= h\"").arg( parent->name().toLower() ) );
=A0=A0=A0=A0 }
=A0=A0=A0=A0 else {
-=A0=A0=A0=A0=A0=A0=A0 s.replace( = "NEPOMUK_PARENT_INCLUDE", QString("\"resource.h\"&= quot;) );
+=A0=A0=A0=A0=A0=A0=A0 s.replace( "NEPOMUK_PARENT_INCLUDE= ", QString("<Nepomuk2/Resource>") );
=A0=A0=A0=A0 }
=A0
=A0=A0=A0=A0 QString methods;
=A0

=

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



--
Vishesh Handa

--bcaec54c5392dc8e1404c5161464-- --===============2522077743508405533== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Active mailing list Active@kde.org https://mail.kde.org/mailman/listinfo/active --===============2522077743508405533==--