unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Alex ter Weele <alex.ter.weele@gmail.com>
To: 34408@debbugs.gnu.org
Subject: [bug#34408] [PATCH] gnu: emacs-pdf-tools: Update to 0.90.
Date: Sat, 09 Feb 2019 17:56:10 -0600	[thread overview]
Message-ID: <874l9cjz39.fsf@librem.i-did-not-set--mail-host-address--so-tickle-me> (raw)

[-- Attachment #1: Type: text/plain, Size: 157 bytes --]


I think this is necessary because of the new version of poppler. See
https://github.com/politza/pdf-tools/commit/581b65f5babb5462f287521d13824ac803ab1fa3.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-emacs-pdf-tools-Update-to-0.90.patch --]
[-- Type: text/x-patch, Size: 4323 bytes --]

From e94414e8a6d15e3a34a086b75324f643930a50b7 Mon Sep 17 00:00:00 2001
From: Alex ter Weele <alex.ter.weele@gmail.com>
Date: Sat, 9 Feb 2019 17:14:34 -0600
Subject: [PATCH] gnu: emacs-pdf-tools: Update to 0.90.

* gnu/packages/emacs-xyz.scm (emacs-pdf-tools): Update to 0.90. [patches]
  remove.

* gnu/packages/patches/emacs-pdf-tools-poppler.patch: delete it, as the
  changes have been released in pdf-tools 0.90.

* gnu/local.mk (dist_patch_DATA): remove patch.
---
 gnu/local.mk                                  |  1 -
 gnu/packages/emacs-xyz.scm                    |  5 +--
 .../patches/emacs-pdf-tools-poppler.patch     | 41 -------------------
 3 files changed, 2 insertions(+), 45 deletions(-)
 delete mode 100644 gnu/packages/patches/emacs-pdf-tools-poppler.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index cf8d29e4ce..37083ca916 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -726,7 +726,6 @@ dist_patch_DATA =						\
   %D%/packages/patches/emacs-fix-scheme-indent-function.patch	\
   %D%/packages/patches/emacs-json-reformat-fix-tests.patch	\
   %D%/packages/patches/emacs-highlight-stages-add-gexp.patch	\
-  %D%/packages/patches/emacs-pdf-tools-poppler.patch		\
   %D%/packages/patches/emacs-scheme-complete-scheme-r5rs-info.patch	\
   %D%/packages/patches/emacs-source-date-epoch.patch		\
   %D%/packages/patches/emacs-realgud-fix-configure-ac.patch	\
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 5d8aa0e863..da27278b11 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -1532,7 +1532,7 @@ filters, new key bindings and faces.  It can be enabled by
 (define-public emacs-pdf-tools
   (package
     (name "emacs-pdf-tools")
-    (version "0.80")
+    (version "0.90")
     (home-page "https://github.com/politza/pdf-tools")
     (source (origin
               (method git-fetch)
@@ -1541,8 +1541,7 @@ filters, new key bindings and faces.  It can be enabled by
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "1i4647vax5na73basc5dz4lh9kprir00fh8ps4i0l1y3ippnjs2s"))
-              (patches (search-patches "emacs-pdf-tools-poppler.patch"))))
+                "0iv2g5kd14zk3r5dzdw7b7hk4b5w7qpbilcqkja46jgxbb6xnpl9"))))
     (build-system gnu-build-system)
     (arguments
      `(#:tests? #f ; there are no tests
diff --git a/gnu/packages/patches/emacs-pdf-tools-poppler.patch b/gnu/packages/patches/emacs-pdf-tools-poppler.patch
deleted file mode 100644
index 0477508fa9..0000000000
--- a/gnu/packages/patches/emacs-pdf-tools-poppler.patch
+++ /dev/null
@@ -1,41 +0,0 @@
-Fix build issue with recent Poppler:
-  <https://github.com/politza/pdf-tools/issues/372>.
-
-This combines upstream commits
-6cd76dec9aece2a8daa90f17ab77fbf773157a1d..50a5297b82e26cfd52f6c00645ddc1057099d6a7
-for this file.
-
-diff --git a/server/poppler-hack.cc b/server/poppler-hack.cc
-index 0c62f73..427f9df 100644
---- a/server/poppler-hack.cc
-+++ b/server/poppler-hack.cc
-@@ -51,7 +51,10 @@ GType poppler_annot_markup_get_type (void) G_GNUC_CONST;
-     double y2;
-   };
- 
--  char *_xpoppler_goo_string_to_utf8(GooString *s)
-+  // This function does not modify its argument s, but for
-+  // compatibility reasons (e.g. getLength in GooString.h before 2015)
-+  // with older poppler code, it can't be declared as such.
-+  char *_xpoppler_goo_string_to_utf8(/* const */ GooString *s)
-   {
-     char *result;
- 
-@@ -85,7 +88,7 @@ GType poppler_annot_markup_get_type (void) G_GNUC_CONST;
-   // Set the rectangle of an annotation.  It was first added in v0.26.
-   void xpoppler_annot_set_rectangle (PopplerAnnot *a, PopplerRectangle *rectangle)
-   {
--    GooString *state = a->annot->getAppearState ();
-+    GooString *state = (GooString*) a->annot->getAppearState ();
-     char *ustate = _xpoppler_goo_string_to_utf8 (state);
- 
-     a->annot->setRect (rectangle->x1, rectangle->y1,
-@@ -105,7 +108,7 @@ GType poppler_annot_markup_get_type (void) G_GNUC_CONST;
-     g_return_val_if_fail (POPPLER_IS_ANNOT_MARKUP (poppler_annot), NULL);
- 
-     annot = static_cast<AnnotMarkup *>(POPPLER_ANNOT (poppler_annot)->annot);
--    text = annot->getDate ();
-+    text = (GooString*) annot->getDate ();
- 
-     return text ? _xpoppler_goo_string_to_utf8 (text) : NULL;
-   }
-- 
2.20.1


             reply	other threads:[~2019-02-09 23:57 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-09 23:56 Alex ter Weele [this message]
2019-02-10  4:10 ` bug#34408: [PATCH] gnu: emacs-pdf-tools: Update to 0.90 Ricardo Wurmus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874l9cjz39.fsf@librem.i-did-not-set--mail-host-address--so-tickle-me \
    --to=alex.ter.weele@gmail.com \
    --cc=34408@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).