> On 04/12/2015 14:12, Michael Zanetti wrote:
> > Review: Needs Fixing
> >
> > have a bunch of build failures here
> >
> > tests/framework/fake_mirsurface.h:113: Error: Not a signal declaration
> >
> > trying with -DNO_TESTS:
> >
> > In file included from /home/micha/Develop/reviews/multiSurfaceApp/src/platfo
> rms/mirserver/mirsingleton.cpp:1:0:
> > /home/micha/Develop/reviews/multiSurfaceApp/src/platforms/mirserver/mirsingl
> eton.h:33:10: error: ‘void qtmir::Mir::setCursorName(const QString&)’ marked
> ‘override’, but does not override
> > void setCursorName(const QString &cursorName) override;
> > ^
> > /home/micha/Develop/reviews/multiSurfaceApp/src/platforms/mirserver/mirsingl
> eton.h:34:13: error: ‘QString qtmir::Mir::cursorName() const’ marked
> ‘override’, but does not override
> > QString cursorName() const override;
> > ^
> > /home/micha/Develop/reviews/multiSurfaceApp/src/platforms/mirserver/mirsingl
> eton.cpp: In member function ‘void qtmir::Mir::setCursorName(const QString&)’:
> > /home/micha/Develop/reviews/multiSurfaceApp/src/platforms/mirserver/mirsingl
> eton.cpp:26:46: error: ‘cursorNameChanged’ was not declared in this scope
> > Q_EMIT cursorNameChanged(m_cursorName);
> >
> >
> >
> > Also got this when trying to build on the phone
> >
> > /home/phablet/real_home/qtmir-
> multiSurfaceApp/src/modules/Unity/Application/session.cpp:315:65: error:
> 'm_surface' was not declared in this scope
> > qCDebug(QTMIR_SESSIONS) << "Session::close - " << name() << m_surface;
> > ^
> > /home/phablet/real_home/qtmir-
> multiSurfaceApp/src/modules/Unity/Application/session.cpp:316:9: error:
> 'm_surface' was not declared in this scope
> > if (m_surface) {
> > ^
> >
> > Not sure if the last one is just masked by the other on my desktop build or
> if you fixed that in the meantime.
>
>
>
> Just built it here. You sure you had
> lp:~dandrader/unity-api/surfaceItemFillMode2 installed?
sorry, I was building unity8 instead.
Fixed. It was due to a bad merge with polite-close.
> On 04/12/2015 14:12, Michael Zanetti wrote: /fake_mirsurfac e.h:113: Error: Not a signal declaration Develop/ reviews/ multiSurfaceApp /src/platfo mirsingleton. cpp:1:0: Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingl Mir::setCursorN ame(const QString&)’ marked Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingl Mir::cursorName () const’ marked Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingl Mir::setCursorN ame(const QString&)’: Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingl ed(m_cursorName ); real_home/ qtmir- /src/modules/ Unity/Applicati on/session. cpp:315: 65: error: QTMIR_SESSIONS) << "Session::close - " << name() << m_surface; real_home/ qtmir- /src/modules/ Unity/Applicati on/session. cpp:316: 9: error:
> > Review: Needs Fixing
> >
> > have a bunch of build failures here
> >
> > tests/framework
> >
> > trying with -DNO_TESTS:
> >
> > In file included from /home/micha/
> rms/mirserver/
> > /home/micha/
> eton.h:33:10: error: ‘void qtmir::
> ‘override’, but does not override
> > void setCursorName(const QString &cursorName) override;
> > ^
> > /home/micha/
> eton.h:34:13: error: ‘QString qtmir::
> ‘override’, but does not override
> > QString cursorName() const override;
> > ^
> > /home/micha/
> eton.cpp: In member function ‘void qtmir::
> > /home/micha/
> eton.cpp:26:46: error: ‘cursorNameChanged’ was not declared in this scope
> > Q_EMIT cursorNameChang
> >
> >
> >
> > Also got this when trying to build on the phone
> >
> > /home/phablet/
> multiSurfaceApp
> 'm_surface' was not declared in this scope
> > qCDebug(
> > ^
> > /home/phablet/
> multiSurfaceApp
> 'm_surface' was not declared in this scope
> > if (m_surface) {
> > ^
> >
> > Not sure if the last one is just masked by the other on my desktop build or
> if you fixed that in the meantime.
>
>
>
> Just built it here. You sure you had
> lp:~dandrader/unity-api/surfaceItemFillMode2 installed?
sorry, I was building unity8 instead.
Fixed. It was due to a bad merge with polite-close.