> On Oct. 17, 2016, 4:51 a.m., Anthony Fieroni wrote: > > CMakeLists.txt, line 47 > > <https://git.reviewboard.kde.org/r/129203/diff/1/?file=482304#file482304line47> > > > > You can't increase minimum deps only for this framework. It's done > > automatically for all and when it's needed. Just try this patch > > https://git.reviewboard.kde.org/r/129065/ give 'ship it' if works correctly
I see, the original patch was not enough, there are 2 usings of QQuickWindow::NoStage, so I've updated the patch. - Alberto ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129203/#review100058 ----------------------------------------------------------- On Oct. 17, 2016, 1:37 a.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, 1:37 a.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 > ----- > > CMakeLists.txt 3c0eaee > src/plasmaquick/dialog.cpp eac9ee5 > src/plasmaquick/dialogshadows.cpp eb67926 > > 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 > >
