Package: apvlv
Version: 0.1.1-1
Severity: wishlist
Tags: patch

Hi,

currently apvlv fails to compile with Poppler >= 0.15 (which is still
only in Debian experimental), see also the upstream bug:
http://code.google.com/p/apvlv/issues/detail?id=110
I attached to that upstream bug a patch which fixes the issue, collecting
all the feedback from Fedora & Ubuntu people in that bug report, in a way
which is compatible with the Poppler currently in Debian (0.12.4).
Could you please apply the patch (attached to this bug as well),
so when we will eventually start a Poppler transition apvlv can just be
binNMUed?

Thanks,
-- 
Pino
--- a/src/ApvlvFile.cpp
+++ b/src/ApvlvFile.cpp
@@ -302,9 +302,14 @@
 
   bool ApvlvPDF::pagetext (int pn, int x1, int y1, int x2, int y2, char **out)
   {
-    PopplerRectangle rect = { x1, y1, x2, y2 };
     PopplerPage *page = poppler_document_get_page (mDoc, pn);
+#if POPPLER_CHECK_VERSION(0, 15, 1)
+    PopplerRectangle rect = { x1, y2, x2, y1 };
+    *out = poppler_page_get_selected_text (page, POPPLER_SELECTION_WORD, &rect);
+#else
+    PopplerRectangle rect = { x1, y1, x2, y2 };
     *out = poppler_page_get_text (page, POPPLER_SELECTION_WORD, &rect);
+#endif
     if (*out != NULL)
       {
         return true;

Reply via email to