> On Sept. 14, 2015, 8:44 a.m., Martin Gräßlin wrote: > > -2, a change for a month in the devel branch doesn't make much sense. > > David Faure wrote: > As you want. You're raising the bar for new contributors, who can't work > on your code using the latest KDE Frameworks release. > > You and me might compile everything, but you'll get more contributors if > you let people work on workspace and apps using a released frameworks (for > which there are distro packages) than if you require them to compile > frameworks first. Just like we don't require Qt from git, we shouldn't > require KF5 from git, I thought this was the general agreement. > > If you're worried about the ifdef, just use the two liner version of the > code forever, I was always a bit dubious about adding a method just for that > anyway. > > Martin Gräßlin wrote: > It's really not that uncommon to depend on latest frameworks in > workspace. It's common that I add things in KWindowSystem to make use of it > in KWin directly. Or lately I used lots of new functionalty from KGlobalAccel > directly. > > Yes it raises the entry level, but it's also rather unlikely that we are > able to a policy forbidding depending on frameworks master without CI checks. > > Sebastian Kügler wrote: > Besides, these occasional devs can use the stable branch en then forward > port? > > Marco Martin wrote: > -2 from here as well for the same reasons > > Ben Cooksley wrote: > Please note that the CI system is shifting towards only allowing usage of > released products. We'll also be imposing a dependency prohibition between > Applications and Plasma so there will no longer be any ability to have > dependencies between the two. > > Aleix Pol Gonzalez wrote: > @Ben: Really? Why? Where was this discussed? > > Martin Gräßlin wrote: > @Ben: where should libraries like kwayland and kdecoration go then? They > do not fit requirements of frameworks but might be useable to applications > (kwayland is a must have library for any wayland integration). > > Ben Cooksley wrote: > That has yet to be resolved. Likely another layer which will be released > as needed by Applications / Plasma which sits in between Frameworks and them > will be added I suspect. No binary compatibility commitments, same licensing > rules as Applications / Plasma, just a logically separate product so it > doesn't cause dependency problems. I don't expect too many libraries or > runtime components to end up there (Dr Konqi might perhaps?). > > Martin Gräßlin wrote: > Honestly: I don't see the point in that except of making everything more > complicated (yet another product which has a dedicated release cycle). If > applications want to depend on workspace libraries: that's fine. They know > what it means. We shouldn't prevent "stupid things" in the CI system.
If this patch is to be discarded, kde frameworks dep should be raised to 5.15, because it won't compile with 5.14 (which it currently requires) without this change. - Armin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125187/#review85353 ----------------------------------------------------------- On Sept. 12, 2015, 9:38 a.m., Armin K. wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125187/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2015, 9:38 a.m.) > > > Review request for Plasma and David Faure. > > > Repository: plasma-workspace > > > Description > ------- > > KDesktopFile.readMimeTypes(); hasn't made it into Frameworks 5.14, causing > plasma-workspace git master to depend on yet unreleased version of KDE > Frameworks to build. David Faure has suggested to use fix like this one until > at least Frameworks 5.15 have been released. > > I don't have commit access, so someone needs to commit this for me. > > > Diffs > ----- > > applets/icon/plugin/icon_p.cpp 97af67a > > Diff: https://git.reviewboard.kde.org/r/125187/diff/ > > > Testing > ------- > > It builds. > > > Thanks, > > Armin K. > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
