Re: QCA2 and Remote Widgets

2009-09-21 Thread Alexander Neundorf
On Sunday 20 September 2009, Artur Souza (MoRpHeUz) wrote:
 On Saturday 19 September 2009, 09:23 Alexander Neundorf wrote:
  It still doesn't build on my machine:
  Please add proper checks for QCA.

 I think that he already did that:

 if(QCA2_FOUND)
 include_directories(${QCA2_INCLUDE_DIR})
 set(ENABLE_REMOTE_WIDGETS TRUE)
 endif(QCA2_FOUND)

We're getting closer.
FindQCA2.cmake finds a QCA which is probably too old.
I have a qca 1.0.0 installed here, which consists of 
a /usr/lib/libqca.so.1.0.0 and one header, /usr/include/qca.h

What would be a better way to test for qca2 ? The current test seems to find 
also qca1.

Alex
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: QCA2 and Remote Widgets

2009-09-20 Thread Alexander Neundorf
On Sunday 06 September 2009, Dario Freddi wrote:
 In data domenica 06 settembre 2009 18:47:05, Rob Scheepmaker ha scritto:
 :  On Sunday 06 September 2009 16:49:36 Dario Freddi wrote:
 : 
   Hello people,
  
   we do have a problem! The new remote widgets in plasma are requiring
   QCA2 to make things work out. The problem is that by now, QCA2 is an
   optional package for KDELibs, and when it is not found, there are no
   checks in plasma that will prevent things needing QtCrypto (the
   remotewidgets authorizer) from being thrown off the build.
 
  Oops... Somehow I was under the impressioon QCA2 was already a required
  package.

 My fault as well, I did not notice that in the first place while helping
 you push the stuff :)

   Rob, is it possible at the moment to compile libplasma without
   QCA2/Remote Widgets? If so, please tell me how so that I can come up
   with a fix to this. Otherwise, we have to get through k-c-d requesting
   another hard dependency for KDELibs
 
  Compiling without Remote Widgets suppor is not yet possible at the
  moment, but I can make sure it won't be required. All QCA related code is
  only in one class: Credentials. I can add some cmake stuff and ifdefs to
  make sure validSignature() always returns false and canSign always
  returns false in case of a missing QCA. This makes sure
  accessRemoteApplet and publish() always just plain fails.

 This one is a valuable option, we can spit a phat warning if QCA2 is not
 installed in the optional packages. KAuth at the moment has a similar
 behavior on linux if polkit-qt is not found.

   I should probably return no remoteApplets in
   AccessManager as well so we don't list zeroconf announced plasmoids in
   places that do that (soon the new widget explorer for example), since
  you won't be able to connect anyways without QCA.

 This also would be even nicer

  I'm now dealing with some personal stuff, but I'm sure I'll have the time
  tomorrow to fix this, and make QCA2 an optional dependency for libplasma.

 Take your time, 4.4 is far away :) My primary concern was wheter to trigger
 a discussion on having yet another hard dependency on kdelibs. Just ping me
 back when you're done with this

It still doesn't build on my machine:
[ 94%] Building CXX object plasma/CMakeFiles/plasma.dir/package.o
In file included from /home/alex/src/kde4-svn/KDE 
dir/kdelibs/plasma/package.cpp:45:
/home/alex/src/kde4-svn/KDE 
dir/kdelibs/plasma/private/authorizationmanager_p.h:29:20: error: QtCrypto: 
No such file or directory
In file included from /home/alex/src/kde4-svn/KDE 
dir/kdelibs/plasma/package.cpp:45:
/home/alex/src/kde4-svn/KDE 
dir/kdelibs/plasma/private/authorizationmanager_p.h:72: error: 'QCA' has not 
been declared
/home/alex/src/kde4-svn/KDE 
dir/kdelibs/plasma/private/authorizationmanager_p.h:72: error: ISO C++ 
forbids declaration of 'Initializer' with no type
/home/alex/src/kde4-svn/KDE 
dir/kdelibs/plasma/private/authorizationmanager_p.h:72: error: expected ';' 
before 'initializer'
make[2]: *** [plasma/CMakeFiles/plasma.dir/package.o] Error 1
make[1]: *** [plasma/CMakeFiles/plasma.dir/all] Error 2
make: *** [all] Error 2


Please add proper checks for QCA.

Alex
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: QCA2 and Remote Widgets

2009-09-20 Thread Artur Souza (MoRpHeUz)
On Saturday 19 September 2009, 09:23 Alexander Neundorf wrote:
 It still doesn't build on my machine:
 Please add proper checks for QCA.

I think that he already did that:

if(QCA2_FOUND)
include_directories(${QCA2_INCLUDE_DIR})
set(ENABLE_REMOTE_WIDGETS TRUE)
endif(QCA2_FOUND)

So, try a clean build please...

Cheers!

--
Artur Duque de Souza
openBossa
INdT - Instituto Nokia de Tecnologia
--
Blog: http://blog.morpheuz.cc
PGP: 0xDBEEAAC3 @ wwwkeys.pgp.net
--


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


QCA2 and Remote Widgets

2009-09-06 Thread Dario Freddi
Hello people,

we do have a problem! The new remote widgets in plasma are requiring QCA2 to 
make things work out. The problem is that by now, QCA2 is an optional package 
for KDELibs, and when it is not found, there are no checks in plasma that will 
prevent things needing QtCrypto (the remotewidgets authorizer) from being 
thrown off the build.

Rob, is it possible at the moment to compile libplasma without QCA2/Remote 
Widgets? If so, please tell me how so that I can come up with a fix to this. 
Otherwise, we have to get through k-c-d requesting another hard dependency for 
KDELibs

-- 
---

Dario Freddi
KDE Developer
GPG Key Signature: 511A9A3B


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: QCA2 and Remote Widgets

2009-09-06 Thread Rob Scheepmaker
On Sunday 06 September 2009 16:49:36 Dario Freddi wrote:
 Hello people,

 we do have a problem! The new remote widgets in plasma are requiring QCA2
 to make things work out. The problem is that by now, QCA2 is an optional
 package for KDELibs, and when it is not found, there are no checks in
 plasma that will prevent things needing QtCrypto (the remotewidgets
 authorizer) from being thrown off the build.

Oops... Somehow I was under the impressioon QCA2 was already a required 
package.

 Rob, is it possible at the moment to compile libplasma without QCA2/Remote
 Widgets? If so, please tell me how so that I can come up with a fix to
 this. Otherwise, we have to get through k-c-d requesting another hard
 dependency for KDELibs

Compiling without Remote Widgets suppor is not yet possible at the moment, but 
I can make sure it won't be required. All QCA related code is only in one 
class: Credentials. I can add some cmake stuff and ifdefs to make sure 
validSignature() always returns false and canSign always returns false in case 
of a missing QCA. This makes sure accessRemoteApplet and publish() always just 
plain fails. I should probably return no remoteApplets in AccessManager as 
well so we don't list zeroconf announced plasmoids in places that do that 
(soon the new widget explorer for example), since you won't be able to connect 
anyways without QCA.
I'm now dealing with some personal stuff, but I'm sure I'll have the time 
tomorrow to fix this, and make QCA2 an optional dependency for libplasma.

Regards,
Rob
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: QCA2 and Remote Widgets

2009-09-06 Thread Dario Freddi
In data domenica 06 settembre 2009 18:47:05, Rob Scheepmaker ha scritto:
:  On Sunday 06 September 2009 16:49:36 Dario Freddi wrote:
  Hello people,
 
  we do have a problem! The new remote widgets in plasma are requiring QCA2
  to make things work out. The problem is that by now, QCA2 is an optional
  package for KDELibs, and when it is not found, there are no checks in
  plasma that will prevent things needing QtCrypto (the remotewidgets
  authorizer) from being thrown off the build.
 
 Oops... Somehow I was under the impressioon QCA2 was already a required
 package.

My fault as well, I did not notice that in the first place while helping you 
push the stuff :)

 
  Rob, is it possible at the moment to compile libplasma without
  QCA2/Remote Widgets? If so, please tell me how so that I can come up with
  a fix to this. Otherwise, we have to get through k-c-d requesting another
  hard dependency for KDELibs
 
 Compiling without Remote Widgets suppor is not yet possible at the moment,
  but I can make sure it won't be required. All QCA related code is only in
  one class: Credentials. I can add some cmake stuff and ifdefs to make sure
  validSignature() always returns false and canSign always returns false in
  case of a missing QCA. This makes sure accessRemoteApplet and publish()
  always just plain fails.

This one is a valuable option, we can spit a phat warning if QCA2 is not 
installed in the optional packages. KAuth at the moment has a similar behavior 
on linux if polkit-qt is not found.

  I should probably return no remoteApplets in
  AccessManager as well so we don't list zeroconf announced plasmoids in
  places that do that (soon the new widget explorer for example), since you
  won't be able to connect anyways without QCA.

This also would be even nicer

 I'm now dealing with some personal stuff, but I'm sure I'll have the time
 tomorrow to fix this, and make QCA2 an optional dependency for libplasma.

Take your time, 4.4 is far away :) My primary concern was wheter to trigger a 
discussion on having yet another hard dependency on kdelibs. Just ping me back 
when you're done with this

 
 Regards,
 Rob
 

-- 
---

Dario Freddi
KDE Developer
GPG Key Signature: 511A9A3B


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel