Location dataengine is broken

2010-11-25 Thread Artur de Souza 
Hello :)

It seems that the "ip location tools" service does not provide the  
same API as before.
So the page that we are using:

http://iplocationtools.com/ip_query.php

Does not exist anymore.
Anybody willing to fix this? (Or is anybody the maintainer of that engine?)

Otherwise I could take a look when I have some free time (it will take  
some time for this to happen as I already have other KDE stuff on the  
queue hehe)

Cheers,

Artur

---


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

Re: Location dataengine is broken

2010-11-25 Thread Vitor Boschi 
Hi,

http://ipinfodb.com/ip_query.php is the actual address being used by the
dataengine. The problem is that they started to require an API key in order
to answer requests. The key is free, but I'm not sure that we can use it in
our application, so I'm going to send an email to ask them and will reply
here.

Vitor


On Thu, Nov 25, 2010 at 9:03 AM, Thomas Olsen  wrote:

> On Thursday 25 November 2010 10:45:09 Artur de Souza wrote:
> > Hello :)
> >
> > It seems that the "ip location tools" service does not provide the
> > same API as before.
> > So the page that we are using:
> >
> > http://iplocationtools.com/ip_query.php
>
> This could be used instead:
>
> http://ipinfodb.com/ip_location_api.php
>
> I don't have any build environment setup as my HW is too low end to even
> try to build anything :-/
>
> --
> Best regards / med venlig hilsen
>
> Thomas Olsen
>
> Dropbox and my thumb drive had an epic battle. Dropbox won.
> http://db.tt/T0p6d6L
>
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: irc meeting on the 22nd: git workflow, git and kdebase-workspace

2010-11-25 Thread Riccardo Iaconelli 
On Monday 22 November 2010 19:22:32 Aaron J. Seigo wrote:
> i will be hosting a discussion wednesday the 22nd at 18:00 UTC in #plasma 
> about git and our projects.

Do we have any logs or summary of the meeting? :)

Bye,
-Riccardo


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

summary of the Nov 24th git migration meeting

2010-11-25 Thread Aaron J. Seigo 
hi all ...

i put together a summary of yesterday's meeting here:

http://community.kde.org/Plasma/20101125

it's linked to from the usual Meetings section on the main Plasma community 
wiki page as well.

please read it for accuracy, provide corrections and additions as necessary. 
hopefully this will become the springboard for further fruitful conversations 
(leading ultimately to a conclusion on) our intended workflow.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks


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: Review Request: Kickoff application uninstaller

2010-11-25 Thread Daniel Nicoletti 

---
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5974/
---

(Updated 2010-11-25 23:44:39.182743)


Review request for Plasma and Aaron Seigo.


Summary
---

Patch to use PackageKit session interface to "Uninstall" the applications.
When kickoff starts it checks if the session interface is available, if it's 
not available the
right click option to "Uninstall" applications will not be displayed.


Diffs
-

  
/trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp
 1200785 

Diff: http://svn.reviewboard.kde.org/r/5974/diff


Testing
---


Screenshots (updated)
---

Kickoff Uninstaller
  http://svn.reviewboard.kde.org/r/5974/s/563/


Thanks,

Daniel

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

Re: Review Request: Kickoff application uninstaller

2010-11-25 Thread Aaron Seigo 

---
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5974/
---

(Updated 2010-11-25 23:48:05.415571)


Review request for Plasma.


Changes
---

am part of the plasma group; don't need two copies :)


Summary
---

Patch to use PackageKit session interface to "Uninstall" the applications.
When kickoff starts it checks if the session interface is available, if it's 
not available the
right click option to "Uninstall" applications will not be displayed.


Diffs
-

  
/trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp
 1200785 

Diff: http://svn.reviewboard.kde.org/r/5974/diff


Testing
---


Screenshots
---

Kickoff Uninstaller
  http://svn.reviewboard.kde.org/r/5974/s/563/


Thanks,

Daniel

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

Re: Review Request: Kickoff application uninstaller

2010-11-25 Thread Aaron Seigo 

---
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5974/#review8979
---


looks good (for 4.7), with the exception of using QDBusServiceWatcher.


/trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp


this would be better with a QDBusServiceWatcher (faster, also detect 
changes at runtime, not have to expose DBus internals directly in the code)


- Aaron


On 2010-11-25 23:48:05, Daniel Nicoletti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5974/
> ---
> 
> (Updated 2010-11-25 23:48:05)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> Patch to use PackageKit session interface to "Uninstall" the applications.
> When kickoff starts it checks if the session interface is available, if it's 
> not available the
> right click option to "Uninstall" applications will not be displayed.
> 
> 
> Diffs
> -
> 
>   
> /trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp
>  1200785 
> 
> Diff: http://svn.reviewboard.kde.org/r/5974/diff
> 
> 
> Testing
> ---
> 
> 
> Screenshots
> ---
> 
> Kickoff Uninstaller
>   http://svn.reviewboard.kde.org/r/5974/s/563/
> 
> 
> Thanks,
> 
> Daniel
> 
>

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

Re: Review Request: Kickoff application uninstaller

2010-11-25 Thread Aaron Seigo 


> On 2010-11-25 23:51:56, Aaron Seigo wrote:
> > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp,
> >  lines 104-117
> > 
> >
> > this would be better with a QDBusServiceWatcher (faster, also detect 
> > changes at runtime, not have to expose DBus internals directly in the code)

after further discussion on irc with dantti, it turns out that 
org.freedesktop.PackageKit may not be running and only activatable, so it needs 
to be checked for being available for activation. so this approach, while ugly 
(typical for direct dbus calls like this), is probably the best/only approach 
at hand.


- Aaron


---
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5974/#review8979
---


On 2010-11-25 23:48:05, Daniel Nicoletti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5974/
> ---
> 
> (Updated 2010-11-25 23:48:05)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> Patch to use PackageKit session interface to "Uninstall" the applications.
> When kickoff starts it checks if the session interface is available, if it's 
> not available the
> right click option to "Uninstall" applications will not be displayed.
> 
> 
> Diffs
> -
> 
>   
> /trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp
>  1200785 
> 
> Diff: http://svn.reviewboard.kde.org/r/5974/diff
> 
> 
> Testing
> ---
> 
> 
> Screenshots
> ---
> 
> Kickoff Uninstaller
>   http://svn.reviewboard.kde.org/r/5974/s/563/
> 
> 
> Thanks,
> 
> Daniel
> 
>

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

Re: Review Request: Kickoff application uninstaller

2010-11-25 Thread Aaron Seigo 

---
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5974/#review8981
---

Ship it!


a comment in the code explaining why an activatable is being checked for would 
be nice.

- Aaron


On 2010-11-25 23:48:05, Daniel Nicoletti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5974/
> ---
> 
> (Updated 2010-11-25 23:48:05)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> Patch to use PackageKit session interface to "Uninstall" the applications.
> When kickoff starts it checks if the session interface is available, if it's 
> not available the
> right click option to "Uninstall" applications will not be displayed.
> 
> 
> Diffs
> -
> 
>   
> /trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp
>  1200785 
> 
> Diff: http://svn.reviewboard.kde.org/r/5974/diff
> 
> 
> Testing
> ---
> 
> 
> Screenshots
> ---
> 
> Kickoff Uninstaller
>   http://svn.reviewboard.kde.org/r/5974/s/563/
> 
> 
> Thanks,
> 
> Daniel
> 
>

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