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/platforms/mirserver/mirsingleton.cpp:1:0:
> /home/micha/Develop/reviews/multiSurfaceApp/src/platforms/mirserver/mirsingleton.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/mirsingleton.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/mirsingleton.cpp: In member function ‘void qtmir::Mir::setCursorName(const QString&)’:
> /home/micha/Develop/reviews/multiSurfaceApp/src/platforms/mirserver/mirsingleton.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.
On 04/12/2015 14:12, Michael Zanetti wrote: /fake_mirsurfac e.h:113: Error: Not a signal declaration Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingleton. cpp:1:0: Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingleton. h:33:10: error: ‘void qtmir:: Mir::setCursorN ame(const QString&)’ marked ‘override’, but does not override Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingleton. h:34:13: error: ‘QString qtmir:: Mir::cursorName () const’ marked ‘override’, but does not override Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingleton. cpp: In member function ‘void qtmir:: Mir::setCursorN ame(const QString&)’: Develop/ reviews/ multiSurfaceApp /src/platforms/ mirserver/ mirsingleton. cpp:26: 46: error: ‘cursorNameChanged’ was not declared in this scope ed(m_cursorName ); real_home/ qtmir-multiSurf aceApp/ src/modules/ Unity/Applicati on/session. cpp:315: 65: error: 'm_surface' was not declared in this scope QTMIR_SESSIONS) << "Session::close - " << name() << m_surface; real_home/ qtmir-multiSurf aceApp/ src/modules/ Unity/Applicati on/session. cpp:316: 9: error: 'm_surface' was not declared in this scope
> Review: Needs Fixing
>
> have a bunch of build failures here
>
> tests/framework
>
> trying with -DNO_TESTS:
>
> In file included from /home/micha/
> /home/micha/
> void setCursorName(const QString &cursorName) override;
> ^
> /home/micha/
> QString cursorName() const override;
> ^
> /home/micha/
> /home/micha/
> Q_EMIT cursorNameChang
>
>
>
> Also got this when trying to build on the phone
>
> /home/phablet/
> qCDebug(
> ^
> /home/phablet/
> 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?