Re: Plasma Addons build breakage - Marble wallpaper

2011-05-18 Thread Anne-Marie Mahfouf
On Wednesday 18 May 2011 05:50:29 Ben Cooksley wrote:
 Hi,
 
 Mailing both lists as i'm not sure where the maintainer for this
 component is. Currently, using both Marble master and Plasma Addons
 master, I am unable to build Plasma Addons due to the following
 compilation failure:
 
 [ 93%] Building CXX object
 wallpapers/marble/CMakeFiles/plasma_wallpaper_marble.dir/marble.o
 /home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp: In
 member function ‘virtual void Marble::MarbleWallpaper::init(const
 KConfigGroup)’:
 /home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp:116:16:
 error: ‘class Marble::MarbleMap’ has no member named ‘zoomView’
 /home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp: In
 member function ‘virtual void
 Marble::MarbleWallpaper::wheelEvent(QGraphicsSceneWheelEvent*)’:
 /home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp:211:16:
 error: ‘class Marble::MarbleMap’ has no member named ‘zoomViewBy’
 /home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp:212:25:
 error: ‘class Marble::MarbleMap’ has no member named ‘zoom’
 
 It looks like a change on the Marble side might have broken things
 somehow, can someone please take a look?
 
 Thanks,
 Ben

Hi Ben,

I reported this yesterday to Marble-devel and I think they thought I was the 
wallpaper plugin maintainer which I am not. I don't know if there is a 
maintainer as such.
In any case it would make sense for Marble developers to pass on Marble 
related changes in other part of KDE code using lxr.kde.org for example. This 
is how it's usually done in KDE.

Action should be taken as tomorrow is the first beta tagging. I do not have the 
proper knowledge to fix it unfortunately. 

Best regards,

Anne-Marie


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


Re: Plasma Addons build breakage - Marble wallpaper

2011-05-18 Thread Aaron J. Seigo
On Wednesday, May 18, 2011 15:50:29 Ben Cooksley wrote:
 Hi,
 
 Mailing both lists as i'm not sure where the maintainer for this
 component is. Currently, using both Marble master and Plasma Addons
 master, I am unable to build Plasma Addons due to the following
 compilation failure:

fixed; this really should move to the marble repository, however. is the 
marble team cool with that?

-- 
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


Plasma Addons build breakage - Marble wallpaper

2011-05-17 Thread Ben Cooksley
Hi,

Mailing both lists as i'm not sure where the maintainer for this
component is. Currently, using both Marble master and Plasma Addons
master, I am unable to build Plasma Addons due to the following
compilation failure:

[ 93%] Building CXX object
wallpapers/marble/CMakeFiles/plasma_wallpaper_marble.dir/marble.o
/home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp: In
member function ‘virtual void Marble::MarbleWallpaper::init(const
KConfigGroup)’:
/home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp:116:16:
error: ‘class Marble::MarbleMap’ has no member named ‘zoomView’
/home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp: In
member function ‘virtual void
Marble::MarbleWallpaper::wheelEvent(QGraphicsSceneWheelEvent*)’:
/home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp:211:16:
error: ‘class Marble::MarbleMap’ has no member named ‘zoomViewBy’
/home/ben/kdesvn/kdeplasma-addons/wallpapers/marble/marble.cpp:212:25:
error: ‘class Marble::MarbleMap’ has no member named ‘zoom’

It looks like a change on the Marble side might have broken things
somehow, can someone please take a look?

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