kdeobservatory seems to be missing a qwt min version check
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. http://build.kde.org/view/All/job/kdeplasma-addons_master/212/console Cheers, Albert 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@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: kdeobservatory seems to be missing a qwt min version check
On Tue, Jun 4, 2013 at 8:09 AM, Albert Astals Cid 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@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: kdeobservatory seems to be missing a qwt min version check
Hi there, 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. What do you guys prefer ? -- Sandro On Mon, Jun 3, 2013 at 5:22 PM, Ben Cooksley wrote: > On Tue, Jun 4, 2013 at 8:09 AM, Albert Astals Cid 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@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: kdeobservatory seems to be missing a qwt min version check
On Tue, Jun 4, 2013 at 12:22 PM, Sandro Andrade 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 wrote: >> On Tue, Jun 4, 2013 at 8:09 AM, Albert Astals Cid 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@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: kdeobservatory seems to be missing a qwt min version check
El Dilluns, 3 de juny de 2013, a les 21:22:42, Sandro Andrade va escriure: > Hi there, > > 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. It's unmaintained, but does it work? We ship lots of unmaintained stuff that "works". So if fixing it for Qwt 6.1 makes it work I'd just keep it and try to find a new maintainer. Cheers, Albert > > What do you guys prefer ? > -- > Sandro > > On Mon, Jun 3, 2013 at 5:22 PM, Ben Cooksley wrote: > > On Tue, Jun 4, 2013 at 8:09 AM, Albert Astals Cid 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@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: kdeobservatory seems to be missing a qwt min version check
It used to work on the SVN, needs to be ported to Git. /me downloading it to see if I can revive it. On Tue, Jun 4, 2013 at 2:22 PM, Albert Astals Cid wrote: > El Dilluns, 3 de juny de 2013, a les 21:22:42, Sandro Andrade va escriure: > > Hi there, > > > > 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. > > It's unmaintained, but does it work? We ship lots of unmaintained stuff > that > "works". So if fixing it for Qwt 6.1 makes it work I'd just keep it and > try to > find a new maintainer. > > Cheers, > Albert > > > > > What do you guys prefer ? > > -- > > Sandro > > > > On Mon, Jun 3, 2013 at 5:22 PM, Ben Cooksley wrote: > > > On Tue, Jun 4, 2013 at 8:09 AM, Albert Astals Cid > 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@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 > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
