> On Oct. 17, 2016, 10:20 a.m., David Edmundson wrote: > > Do you have commit access?
I don't think so, it's my first patch and I'm newbie to the KDE development team :) - Alberto ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129203/#review100077 ----------------------------------------------------------- On Oct. 17, 2016, 2:53 p.m., Alberto Curro wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129203/ > ----------------------------------------------------------- > > (Updated Oct. 17, 2016, 2:53 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > Being a newbie, following build instruction from the wiki, I could not build > plasma-framework on my Linux mint 18 (based on Ubuntu Xenial). > > Two of the changes are coding issues. > > The change in CMakeLists.txt comes from a wrong required version, also stated > here: https://git.reviewboard.kde.org/r/128763/. > > For Xenial, QT 5.5.1-2 is included, where QQuickWindow::NoStage is **not** > defined. It was introduced in Qt 5.6, so requiring Qt 5.3 won't work. > > > Diffs > ----- > > src/declarativeimports/core/windowthumbnail.cpp 0453c09 > > Diff: https://git.reviewboard.kde.org/r/129203/diff/ > > > Testing > ------- > > Building is ok now. I've had to add a second ppa for Qt 5.7 on Xenial: > https://launchpad.net/~beineri/+archive/ubuntu/opt-qt57-xenial > > > Thanks, > > Alberto Curro > >
