unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH 0/1] Unwinding the mupdf / cups-filters grafts
@ 2017-01-25 18:51 Leo Famulari
  2017-01-25 18:51 ` [PATCH 1/1] gnu: mupdf, cups-filters: Incorporated grafted changes Leo Famulari
  0 siblings, 1 reply; 4+ messages in thread
From: Leo Famulari @ 2017-01-25 18:51 UTC (permalink / raw)
  To: guix-devel

Please review this patch for correctness and style. Thanks in advance :)

Leo Famulari (1):
  gnu: mupdf, cups-filters: Incorporated grafted changes.

 gnu/packages/cups.scm |  8 --------
 gnu/packages/pdf.scm  | 17 +++--------------
 2 files changed, 3 insertions(+), 22 deletions(-)

-- 
2.11.0

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/1] gnu: mupdf, cups-filters: Incorporated grafted changes.
  2017-01-25 18:51 [PATCH 0/1] Unwinding the mupdf / cups-filters grafts Leo Famulari
@ 2017-01-25 18:51 ` Leo Famulari
  2017-01-26  9:56   ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Leo Famulari @ 2017-01-25 18:51 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/pdf.scm (mupdf)[replacement]: Remove field.
[source]: Add 'mupdf-mujs-CVE-2016-10132.patch' and
'mupdf-mujs-CVE-2016-10133.patch'.
(mupdf/fixed): Remove variable.
* gnu/packages/cups.scm (cups-filters)[replacement]: Remove field.
(mupdf/fixed-instead-of-mupdf),(cups-filters/fixed): Remove variables.
---
 gnu/packages/cups.scm |  8 --------
 gnu/packages/pdf.scm  | 17 +++--------------
 2 files changed, 3 insertions(+), 22 deletions(-)

diff --git a/gnu/packages/cups.scm b/gnu/packages/cups.scm
index 39ab41c19..94f8e91f1 100644
--- a/gnu/packages/cups.scm
+++ b/gnu/packages/cups.scm
@@ -52,7 +52,6 @@
 (define-public cups-filters
   (package
     (name "cups-filters")
-    (replacement cups-filters/fixed)
     (version "1.13.1")
     (source(origin
               (method url-fetch)
@@ -135,13 +134,6 @@ filters for the PDF-centric printing workflow introduced by OpenPrinting.")
                    license:lgpl2.0+
                    license:expat))))
 
-(define mupdf/fixed-instead-of-mupdf
-  (package-input-rewriting `((,mupdf . ,(@@ (gnu packages pdf) mupdf/fixed)))))
-
-;;; Fix CVE-2016-10132 and CVE-2016-10133. See mupdf/fixed for more information.
-(define cups-filters/fixed
-  (mupdf/fixed-instead-of-mupdf cups-filters))
-
 ;; CUPS on non-MacOS systems requires cups-filters.  Since cups-filters also
 ;; depends on CUPS libraries and binaries, cups-minimal has been added to
 ;; satisfy this dependency.
diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index 2ea48e599..d449b72ee 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -481,7 +481,6 @@ extracting content or merging files.")
 (define-public mupdf
   (package
     (name "mupdf")
-    (replacement mupdf/fixed)
     (version "1.10a")
     (source
       (origin
@@ -491,7 +490,9 @@ extracting content or merging files.")
         (sha256
          (base32
           "0dm8wcs8i29aibzkqkrn8kcnk4q0kd1v66pg48h5c3qqp4v1zk5a"))
-        (patches (search-patches "mupdf-build-with-openjpeg-2.1.patch"))
+        (patches (search-patches "mupdf-build-with-openjpeg-2.1.patch"
+                                 "mupdf-mujs-CVE-2016-10132.patch"
+                                 "mupdf-mujs-CVE-2016-10133.patch"))
         (modules '((guix build utils)))
         (snippet
             ;; Delete all the bundled libraries except for mujs, which is
@@ -540,18 +541,6 @@ line tools for batch rendering (pdfdraw), rewriting files (pdfclean),
 and examining the file structure (pdfshow).")
     (license license:agpl3+)))
 
-(define mupdf/fixed
-  (package
-    (inherit mupdf)
-    (source
-      (origin
-        (inherit (package-source mupdf))
-        (patches
-          (append
-            (origin-patches (package-source mupdf))
-            (search-patches "mupdf-mujs-CVE-2016-10132.patch"
-                            "mupdf-mujs-CVE-2016-10133.patch")))))))
-
 (define-public qpdf
   (package
    (name "qpdf")
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] gnu: mupdf, cups-filters: Incorporated grafted changes.
  2017-01-25 18:51 ` [PATCH 1/1] gnu: mupdf, cups-filters: Incorporated grafted changes Leo Famulari
@ 2017-01-26  9:56   ` Ludovic Courtès
  2017-01-27  0:07     ` Leo Famulari
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2017-01-26  9:56 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> skribis:

> * gnu/packages/pdf.scm (mupdf)[replacement]: Remove field.
> [source]: Add 'mupdf-mujs-CVE-2016-10132.patch' and
> 'mupdf-mujs-CVE-2016-10133.patch'.
> (mupdf/fixed): Remove variable.
> * gnu/packages/cups.scm (cups-filters)[replacement]: Remove field.
> (mupdf/fixed-instead-of-mupdf),(cups-filters/fixed): Remove variables.

LGTM!

I guess this is for ‘core-updates’, assuming we don’t do another
‘staging’ before then, right?

Thanks!

Ludo’.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] gnu: mupdf, cups-filters: Incorporated grafted changes.
  2017-01-26  9:56   ` Ludovic Courtès
@ 2017-01-27  0:07     ` Leo Famulari
  0 siblings, 0 replies; 4+ messages in thread
From: Leo Famulari @ 2017-01-27  0:07 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

On Thu, Jan 26, 2017 at 10:56:03AM +0100, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
> 
> > * gnu/packages/pdf.scm (mupdf)[replacement]: Remove field.
> > [source]: Add 'mupdf-mujs-CVE-2016-10132.patch' and
> > 'mupdf-mujs-CVE-2016-10133.patch'.
> > (mupdf/fixed): Remove variable.
> > * gnu/packages/cups.scm (cups-filters)[replacement]: Remove field.
> > (mupdf/fixed-instead-of-mupdf),(cups-filters/fixed): Remove variables.
> 
> LGTM!
> 
> I guess this is for ‘core-updates’, assuming we don’t do another
> ‘staging’ before then, right?

Yes, I just pushed it to core-updates.

There are a handful of changes on the staging branch. I think we should
bring them to core-updates if that will be built first.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-01-27  0:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-25 18:51 [PATCH 0/1] Unwinding the mupdf / cups-filters grafts Leo Famulari
2017-01-25 18:51 ` [PATCH 1/1] gnu: mupdf, cups-filters: Incorporated grafted changes Leo Famulari
2017-01-26  9:56   ` Ludovic Courtès
2017-01-27  0:07     ` Leo Famulari

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).