Review Request 120876: Forward-port "Fix and future-proof Dr Konqi security methods on Bugzilla" from kde-runtime
Hrvoje Senjan
hrvoje.senjan at gmail.com
Wed Oct 29 19:56:02 UTC 2014
> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141029/042eef9d/attachment.html>
More information about the Plasma-devel
mailing list