Summary: | [smokekde] cannot build okular bindings | ||
---|---|---|---|
Product: | [Unmaintained] bindings | Reporter: | Andrea Scarpino <scarpino> |
Component: | general | Assignee: | kde-bindings |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | arno, fabiodurso |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/smokekde/962907aa68e11d2a9d191a39c61247919db1afaa | Version Fixed In: | |
Sentry Crash Report: |
Description
Andrea Scarpino
2012-11-16 17:47:21 UTC
Git commit 962907aa68e11d2a9d191a39c61247919db1afaa by Fabio D'Urso. Committed on 17/11/2012 at 00:07. Pushed by fabiod into branch 'master'. Added missing include in okular_includes.h M +5 -0 okular/okular_includes.h http://commits.kde.org/smokekde/962907aa68e11d2a9d191a39c61247919db1afaa I've committed a quick fix so that the package can be compiled. *Note*: smokeconfig.xml (in the same dir) probably needs to be modified too (to export the new Okular::Tile class). Can someone more knowledgeable have a look please? Correct. I currently don't have access to a recent enough Okular. Could you maybe add that line to smokeconfig.xml and if it builds without errors, commit it? Git commit 676a80d5ff9a8a7fe7ff5bb03edaf8d63d920666 by Fabio D'Urso. Committed on 18/11/2012 at 00:08. Pushed by fabiod into branch 'master'. Added Okular::Tile class M +1 -0 okular/smokeconfig.xml http://commits.kde.org/smokekde/676a80d5ff9a8a7fe7ff5bb03edaf8d63d920666 Committed. Unlike in the previous patch, it seems that there's no need to check the version here, correct? I've successfully tested the patch against Okular from KDE 4.10 beta1, which obviously has the Tile class, and Okular from KDE 4.9.3, which hasn't. Correct. Thanks a lot. :) |