kdeobservatory seems to be missing a qwt min version check

[Ben Cooksley]

On Tue, Jun 4, 2013 at 12:22 PM, Sandro Andrade <sandroandrade at kde.org> wrote:
> Hi there,

Hi Sandro,

>
> That's actually intended to be moved to unmaintained, sorry for
> haven't done it yet, I'm in the last months of my thesis :( - or maybe
> :)
> Not sure if I can do that at this time in release schedule. I can fix
> it to use Qwt 6.1 or fully move it to maintained.

As the observatory plugin was shipped as part of KDE Plasma Addons in
KDE 4.9, we can't exactly remove it from stable - although this would
be an option for master (and is probably the best thing to do I
think).

I would suggest patching the build system to check the Qwt version to
ensure it is 6.0.x in the interim at least. Based on the message
"Found Qwt version 6.1.1 linked to Qt4" CMake seems to know which
version of Qwt it has found.

>
> What do you guys prefer ?
> --
> Sandro

Thanks,
Ben

>
>
> On Mon, Jun 3, 2013 at 5:22 PM, Ben Cooksley <bcooksley at kde.org> wrote:
>> On Tue, Jun 4, 2013 at 8:09 AM, Albert Astals Cid <aacid at kde.org> wrote:
>>> Hi guys, can anyone please have a look at
>>> kdeplasma-addons/applets/kdeobservatory and make sure it requires the qwt min
>>> version it needs?
>>>
>>> The build is currently failing at build.kde.org (probably a too old qwt
>>> installed) instead of just skipping the applet from being built.
>>
>> I suspect it is actually a case of a too new Qwt being installed.
>> We recently upgraded from Qwt 6.0 to 6.1, which was necessary to allow
>> the smokeqt bindings to compile successfully.
>>
>>>
>>> http://build.kde.org/view/All/job/kdeplasma-addons_master/212/console
>>>
>>> Cheers,
>>>   Albert
>>
>> Regards,
>> Ben
>>
>>>
>>> P.S: I'm *not* subscribed to plasma-devel so CC me if you want to keep me in
>>> the loop
>>> _______________________________________________
>>> Plasma-devel mailing list
>>> Plasma-devel at kde.org
>>> https://mail.kde.org/mailman/listinfo/plasma-devel