> On Okt. 29, 2014, 12:41 nachm., 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 > > ;-)
To prevent future confusion, one might want to add #define KDE_MAKE_BUGZILLA_VERSION KDE_MAKE_VERSION or sth. like this. - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120876/#review69429 ----------------------------------------------------------- On Okt. 28, 2014, 11:02 nachm., Hrvoje Senjan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120876/ > ----------------------------------------------------------- > > (Updated Okt. 28, 2014, 11:02 nachm.) > > > 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
