> On Oct. 29, 2014, 1:41 p.m., Thomas Lübking wrote: > > The version check is actually on the bugzilla version (KDE_MAKE_VERSION is > > just a bitshifting macro) - it's the important part in the original patch > > ;-) > > Thomas Lübking wrote: > To prevent future confusion, one might want to add > > #define KDE_MAKE_BUGZILLA_VERSION KDE_MAKE_VERSION > > or sth. like this.
note to self: never blindly c/p pathes ;-) still, i wonder, how will those versions get evaluated if building with kdelibs > 4.5.x ? i guess i should read the patch and discussion more closely. - Hrvoje ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120876/#review69429 ----------------------------------------------------------- On Oct. 29, 2014, 12:02 a.m., Hrvoje Senjan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120876/ > ----------------------------------------------------------- > > (Updated Oct. 29, 2014, 12:02 a.m.) > > > Review request for Plasma, Ben Cooksley, Ian Wadham, and Thomas Lübking. > > > Bugs: 337742 > https://bugs.kde.org/show_bug.cgi?id=337742 > > > Repository: plasma-workspace > > > Description > ------- > > discussion was in https://git.reviewboard.kde.org/r/120431/ > removed the version checks, as we know we have kdelibs >= 4.5 ;-) > > > Diffs > ----- > > drkonqi/bugzillaintegration/reportassistantpages_bugzilla.h 50cf05f > drkonqi/bugzillaintegration/reportassistantpages_bugzilla.cpp 5a6096f > drkonqi/bugzillaintegration/bugzillalib.h 570169b > drkonqi/bugzillaintegration/bugzillalib.cpp 8fd8399 > > Diff: https://git.reviewboard.kde.org/r/120876/diff/ > > > Testing > ------- > > builds, succesfully reported bug via patched DrKonqi, wasn't able to do so > before. > > > Thanks, > > Hrvoje Senjan > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel