GNU bug report logs - #28235
[PATCH] Update calibre to 3.6.0.

Previous Next

Package: guix-patches;

Reported by: Roel Janssen <roel <at> gnu.org>

Date: Fri, 25 Aug 2017 18:24:02 UTC

Severity: normal

Tags: patch

Done: Roel Janssen <roel <at> gnu.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 28235 in the body.
You can then email your comments to 28235 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#28235; Package guix-patches. (Fri, 25 Aug 2017 18:24:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Roel Janssen <roel <at> gnu.org>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Fri, 25 Aug 2017 18:24:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: guix-patches <guix-patches <at> gnu.org>
Subject: [PATCH] Update calibre to 3.6.0.
Date: Fri, 25 Aug 2017 20:22:49 +0200
Dear Guix,

The following patches update calibre to 3.6.0.  This version doesn't
include the unrar code anymore.  Instead, it uses a python library
called "unrardll", which is not included in these patches.  We removed
the unrar code before, so there's no lost functionality to the current
recipe.

I had to add 'python2-html5-parser' as a new input.  It seems that it is
used in favor of 'python2-html5lib', but that input hadn't been added in
the current recipe.

Thanks for your time!

Kind regards,
Roel Janssen




Information forwarded to guix-patches <at> gnu.org:
bug#28235; Package guix-patches. (Fri, 25 Aug 2017 18:27:02 GMT) Full text and rfc822 format available.

Message #8 received at 28235 <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: 28235 <at> debbugs.gnu.org
Subject: [PATCH 1/3] gnu: Update python-lxml to 3.8.0.
Date: Fri, 25 Aug 2017 20:25:55 +0200
* gnu/packages/python.scm (python-lxml): Update to 3.8.0.
---
 gnu/packages/python.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 4bd777c09..9bf46fb6f 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -6117,14 +6117,14 @@ translate the complete SQLite API into Python.")
 (define-public python-lxml
   (package
     (name "python-lxml")
-    (version "3.6.0")
+    (version "3.8.0")
     (source
       (origin
         (method url-fetch)
         (uri (pypi-uri "lxml" version))
         (sha256
          (base32
-          "1pvbmiy2m7jwv493kilbghhj2pkh8wy1na3ji350vhzhlwlclx4w"))))
+          "15nvf6n285n282682qyw3wihsncb0x5amdhyi4b83bfa2nz74vvk"))))
     (build-system python-build-system)
     (inputs
       `(("libxml2" ,libxml2)
-- 
2.14.1





Information forwarded to guix-patches <at> gnu.org:
bug#28235; Package guix-patches. (Fri, 25 Aug 2017 18:27:02 GMT) Full text and rfc822 format available.

Message #11 received at 28235 <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: 28235 <at> debbugs.gnu.org
Subject: [PATCH 2/3] gnu: Add python-html5-parser, python2-html5-parser
Date: Fri, 25 Aug 2017 20:26:43 +0200
* gnu/packages/python.scm (python-html5-parser): New variable.
  (python2-html5-parser: New variable.
---
 gnu/packages/python.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 9bf46fb6f..8629228db 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -5868,6 +5868,35 @@ and written in Python.")
 (define-public python2-html5lib-0.9
   (package-with-python2 python-html5lib-0.9))
 
+(define-public python-html5-parser
+  (package
+    (name "python-html5-parser")
+    (version "0.4.4")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "html5-parser" version))
+              (sha256
+               (base32
+                "1d8sxhl41ffh7qlk7wlsy17xw6slzx5v1yna9s72wx5qrpaa3wxr"))))
+    (build-system python-build-system)
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("libxml2" ,libxml2)))
+    (propagated-inputs
+     `(("python-lxml" ,python-lxml)
+       ("python-beautifulsoup4" ,python-beautifulsoup4)))
+    (home-page "https://html5-parser.readthedocs.io")
+    (synopsis "Fast C-based HTML5 parsing for Python")
+    (description "This package provides a fast implementation of the HTML5
+parsing spec for Python.  Parsing is done in C using a variant of the gumbo
+parser.  The gumbo parse tree is then transformed into an lxml tree, also in
+C, yielding parse times that can be a thirtieth of the html5lib parse times.")
+    (license license:asl2.0)))
+
+(define-public python2-html5-parser
+  (package-with-python2 python-html5-parser))
+
 (define-public python-webencodings
   (package
     (name "python-webencodings")
-- 
2.14.1




Information forwarded to guix-patches <at> gnu.org:
bug#28235; Package guix-patches. (Fri, 25 Aug 2017 18:28:01 GMT) Full text and rfc822 format available.

Message #14 received at 28235 <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: 28235 <at> debbugs.gnu.org
Subject: [PATCH 3/3] gnu: Update calibre to 3.6.0.
Date: Fri, 25 Aug 2017 20:27:18 +0200
* gnu/local.mk: Remove calibre-drop-unrar.patch
* gnu/packages/ebook.scm (calibre): Update to 3.6.0.
  Add python-html5-parser input.
* gnu/packages/patches/calibre-drop-unrar.patch: Remove file.
---
 gnu/local.mk                                  |  1 -
 gnu/packages/ebook.scm                        | 12 ++++---
 gnu/packages/patches/calibre-drop-unrar.patch | 49 ---------------------------
 3 files changed, 7 insertions(+), 55 deletions(-)
 delete mode 100644 gnu/packages/patches/calibre-drop-unrar.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index 71f1cb244..631ab0b95 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -537,7 +537,6 @@ dist_patch_DATA =						\
   %D%/packages/patches/blast+-fix-makefile.patch		\
   %D%/packages/patches/byobu-writable-status.patch		\
   %D%/packages/patches/cairo-CVE-2016-9082.patch			\
-  %D%/packages/patches/calibre-drop-unrar.patch			\
   %D%/packages/patches/calibre-no-updates-dialog.patch		\
   %D%/packages/patches/calibre-use-packaged-feedparser.patch	\
   %D%/packages/patches/catdoc-CVE-2017-11110.patch		\
diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index 38abf47c8..4bcaa0abf 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -72,7 +72,7 @@
 (define-public calibre
   (package
     (name "calibre")
-    (version "3.0.0")
+    (version "3.6.0")
     (source
       (origin
         (method url-fetch)
@@ -81,21 +81,19 @@
                             version ".tar.xz"))
         (sha256
          (base32
-          "1zhk7bvgr973dd18x4wp48kzai29qqqi5qcy72sxc4wcbk2sbnkw"))
+          "0vp2nds4b5xbchsh1rpc1q7093gd26dnw7mgbnax97dcchvlc4sc"))
         ;; Remove non-free or doubtful code, see
         ;; https://lists.gnu.org/archive/html/guix-devel/2015-02/msg00478.html
         (modules '((guix build utils)))
         (snippet
           '(begin
             (delete-file-recursively "src/calibre/ebooks/markdown")
-            (delete-file-recursively "src/unrar")
             (delete-file "src/odf/thumbnail.py")
             (delete-file-recursively "resources/fonts/liberation")
             (substitute* (find-files "." "\\.py")
               (("calibre\\.ebooks\\.markdown") "markdown"))
             #t))
-        (patches (search-patches "calibre-drop-unrar.patch"
-                                 "calibre-use-packaged-feedparser.patch"
+        (patches (search-patches "calibre-use-packaged-feedparser.patch"
                                  "calibre-no-updates-dialog.patch"))))
     (build-system python-build-system)
     (native-inputs
@@ -129,6 +127,7 @@
        ("python2-dbus" ,python2-dbus)
        ("python2-dnspython" ,python2-dnspython)
        ("python2-feedparser" ,python2-feedparser)
+       ("python2-html5-parser" ,python2-html5-parser)
        ("python2-lxml" ,python2-lxml)
        ("python2-markdown" ,python2-markdown)
        ("python2-mechanize" ,python2-mechanize)
@@ -140,6 +139,9 @@
        ("python2-pyqt" ,python2-pyqt)
        ("python2-sip" ,python2-sip)
        ("python2-regex" ,python2-regex)
+       ;; python2-unrardll is needed for decompressing RAR files.
+       ;; A program called 'pdf2html' is needed for reading PDF books
+       ;; in the web interface.
        ("sqlite" ,sqlite)))
     (arguments
      `(#:python ,python-2
diff --git a/gnu/packages/patches/calibre-drop-unrar.patch b/gnu/packages/patches/calibre-drop-unrar.patch
deleted file mode 100644
index adf977b18..000000000
--- a/gnu/packages/patches/calibre-drop-unrar.patch
+++ /dev/null
@@ -1,49 +0,0 @@
-Recreated old debian patch on the latest calibre version
-
-From 6764e4c211e50d4f4633dbabfba7cbc3089c51dc Mon Sep 17 00:00:00 2001
-From: Brendan Tildesley <brendan.tildesley <at> openmailbox.org>
-Date: Sat, 13 May 2017 21:12:12 +1000
-Subject: [PATCH] Remove unrar extension
-
----
- setup/extensions.json                  | 11 -----------
- src/calibre/ebooks/metadata/archive.py |  2 +-
- 2 files changed, 1 insertion(+), 12 deletions(-)
-
-diff --git a/setup/extensions.json b/setup/extensions.json
-index 1f6d1fb5fd..127390450f 100644
---- a/setup/extensions.json
-+++ b/setup/extensions.json
-@@ -211,16 +211,5 @@
-         "sources": "calibre/devices/mtp/unix/devices.c calibre/devices/mtp/unix/libmtp.c",
-         "headers": "calibre/devices/mtp/unix/devices.h calibre/devices/mtp/unix/upstream/music-players.h calibre/devices/mtp/unix/upstream/device-flags.h",
-         "libraries": "mtp"
--    },
--    {
--        "name": "unrar",
--        "sources": "unrar/rar.cpp unrar/strlist.cpp unrar/strfn.cpp unrar/pathfn.cpp unrar/savepos.cpp unrar/smallfn.cpp unrar/global.cpp unrar/file.cpp unrar/filefn.cpp unrar/filcreat.cpp unrar/archive.cpp unrar/arcread.cpp unrar/unicode.cpp unrar/system.cpp unrar/isnt.cpp unrar/crypt.cpp unrar/crc.cpp unrar/rawread.cpp unrar/encname.cpp unrar/resource.cpp unrar/match.cpp unrar/timefn.cpp unrar/rdwrfn.cpp unrar/consio.cpp unrar/options.cpp unrar/ulinks.cpp unrar/errhnd.cpp unrar/rarvm.cpp unrar/secpassword.cpp unrar/rijndael.cpp unrar/getbits.cpp unrar/sha1.cpp unrar/extinfo.cpp unrar/extract.cpp unrar/volume.cpp unrar/list.cpp unrar/find.cpp unrar/unpack.cpp unrar/cmddata.cpp unrar/filestr.cpp unrar/scantree.cpp calibre/utils/unrar.cpp",
--        "inc_dirs": "unrar",
--        "defines": "SILENT RARDLL UNRAR _FILE_OFFSET_BITS=64 _LARGEFILE_SOURCE",
--        "windows_defines": "SILENT RARDLL UNRAR",
--        "haiku_defines": "LITTLE_ENDIAN SILENT RARDLL UNRAR _FILE_OFFSET_BITS=64 _LARGEFILE_SOURCE _BSD_SOURCE",
--        "haiku_libraries": "bsd",
--        "optimize_level": 2,
--        "windows_libraries": "User32 Advapi32 kernel32 Shell32"
-     }
- ]
-diff --git a/src/calibre/ebooks/metadata/archive.py b/src/calibre/ebooks/metadata/archive.py
-index f5c0b7bed3..32257dcdae 100644
---- a/src/calibre/ebooks/metadata/archive.py
-+++ b/src/calibre/ebooks/metadata/archive.py
-@@ -44,7 +44,7 @@ class ArchiveExtract(FileTypePlugin):
-     description = _('Extract common e-book formats from archive files '
-         '(ZIP/RAR). Also try to autodetect if they are actually '
-         'CBZ/CBR files.')
--    file_types = set(['zip', 'rar'])
-+    file_types = set(['zip'])
-     supported_platforms = ['windows', 'osx', 'linux']
-     on_import = True
- 
--- 
-2.12.2
-
-- 
2.14.1




Information forwarded to guix-patches <at> gnu.org:
bug#28235; Package guix-patches. (Sat, 02 Sep 2017 11:31:01 GMT) Full text and rfc822 format available.

Message #17 received at 28235 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Roel Janssen <roel <at> gnu.org>, 28235 <at> debbugs.gnu.org
Subject: Re: [bug#28235] [PATCH 2/3] gnu: Add python-html5-parser,
 python2-html5-parser
Date: Sat, 02 Sep 2017 13:30:11 +0200
[Message part 1 (text/plain, inline)]
Roel Janssen <roel <at> gnu.org> writes:

> * gnu/packages/python.scm (python-html5-parser): New variable.
>   (python2-html5-parser: New variable.
> ---
>  gnu/packages/python.scm | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
>
> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index 9bf46fb6f..8629228db 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -5868,6 +5868,35 @@ and written in Python.")
>  (define-public python2-html5lib-0.9
>    (package-with-python2 python-html5lib-0.9))
>  
> +(define-public python-html5-parser
> +  (package
> +    (name "python-html5-parser")
> +    (version "0.4.4")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (pypi-uri "html5-parser" version))
> +              (sha256
> +               (base32
> +                "1d8sxhl41ffh7qlk7wlsy17xw6slzx5v1yna9s72wx5qrpaa3wxr"))))
> +    (build-system python-build-system)
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("libxml2" ,libxml2)))
> +    (propagated-inputs
> +     `(("python-lxml" ,python-lxml)
> +       ("python-beautifulsoup4" ,python-beautifulsoup4)))
> +    (home-page "https://html5-parser.readthedocs.io")
> +    (synopsis "Fast C-based HTML5 parsing for Python")
> +    (description "This package provides a fast implementation of the HTML5
> +parsing spec for Python.  Parsing is done in C using a variant of the gumbo
> +parser.  The gumbo parse tree is then transformed into an lxml tree, also in
> +C, yielding parse times that can be a thirtieth of the html5lib parse times.")
> +    (license license:asl2.0)))

The files 'src/as-libxml.[ch]' are GPL3.  Everything else in this series LGTM!
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#28235; Package guix-patches. (Wed, 06 Sep 2017 10:34:01 GMT) Full text and rfc822 format available.

Message #20 received at 28235 <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: Marius Bakke <mbakke <at> fastmail.com>
Cc: 28235 <at> debbugs.gnu.org
Subject: Re: [bug#28235] [PATCH 2/3] gnu: Add python-html5-parser,
 python2-html5-parser
Date: Wed, 06 Sep 2017 12:32:51 +0200
Marius Bakke writes:

> Roel Janssen <roel <at> gnu.org> writes:
>
>> * gnu/packages/python.scm (python-html5-parser): New variable.
>>   (python2-html5-parser: New variable.
>> ---
>>  gnu/packages/python.scm | 29 +++++++++++++++++++++++++++++
>>  1 file changed, 29 insertions(+)
>>
>> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
>> index 9bf46fb6f..8629228db 100644
>> --- a/gnu/packages/python.scm
>> +++ b/gnu/packages/python.scm
>> @@ -5868,6 +5868,35 @@ and written in Python.")
>>  (define-public python2-html5lib-0.9
>>    (package-with-python2 python-html5lib-0.9))
>>  
>> +(define-public python-html5-parser
>> +  (package
>> +    (name "python-html5-parser")
>> +    (version "0.4.4")
>> +    (source (origin
>> +              (method url-fetch)
>> +              (uri (pypi-uri "html5-parser" version))
>> +              (sha256
>> +               (base32
>> +                "1d8sxhl41ffh7qlk7wlsy17xw6slzx5v1yna9s72wx5qrpaa3wxr"))))
>> +    (build-system python-build-system)
>> +    (native-inputs
>> +     `(("pkg-config" ,pkg-config)))
>> +    (inputs
>> +     `(("libxml2" ,libxml2)))
>> +    (propagated-inputs
>> +     `(("python-lxml" ,python-lxml)
>> +       ("python-beautifulsoup4" ,python-beautifulsoup4)))
>> +    (home-page "https://html5-parser.readthedocs.io")
>> +    (synopsis "Fast C-based HTML5 parsing for Python")
>> +    (description "This package provides a fast implementation of the HTML5
>> +parsing spec for Python.  Parsing is done in C using a variant of the gumbo
>> +parser.  The gumbo parse tree is then transformed into an lxml tree, also in
>> +C, yielding parse times that can be a thirtieth of the html5lib parse times.")
>> +    (license license:asl2.0)))
>
> The files 'src/as-libxml.[ch]' are GPL3.  Everything else in this series LGTM!

Oh, it seems this is the case for as-python-tree.[ch], not as-libxml.[ch].

Good catch!  I'll update the license list and push this patch series.

Thanks for your time.

Kind regards,
Roel Janssen




Reply sent to Roel Janssen <roel <at> gnu.org>:
You have taken responsibility. (Wed, 06 Sep 2017 10:47:03 GMT) Full text and rfc822 format available.

Notification sent to Roel Janssen <roel <at> gnu.org>:
bug acknowledged by developer. (Wed, 06 Sep 2017 10:47:03 GMT) Full text and rfc822 format available.

Message #25 received at 28235-done <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: Marius Bakke <mbakke <at> fastmail.com>
Cc: 28235-done <at> debbugs.gnu.org
Subject: Re: [bug#28235] [PATCH 2/3] gnu: Add python-html5-parser,
 python2-html5-parser
Date: Wed, 06 Sep 2017 12:45:59 +0200
Roel Janssen writes:

> Marius Bakke writes:
>
>> Roel Janssen <roel <at> gnu.org> writes:
>>
>>> * gnu/packages/python.scm (python-html5-parser): New variable.
>>>   (python2-html5-parser: New variable.
>>> ---
>>>  gnu/packages/python.scm | 29 +++++++++++++++++++++++++++++
>>>  1 file changed, 29 insertions(+)
>>>
>>> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
>>> index 9bf46fb6f..8629228db 100644
>>> --- a/gnu/packages/python.scm
>>> +++ b/gnu/packages/python.scm
>>> @@ -5868,6 +5868,35 @@ and written in Python.")
>>>  (define-public python2-html5lib-0.9
>>>    (package-with-python2 python-html5lib-0.9))
>>>  
>>> +(define-public python-html5-parser
>>> +  (package
>>> +    (name "python-html5-parser")
>>> +    (version "0.4.4")
>>> +    (source (origin
>>> +              (method url-fetch)
>>> +              (uri (pypi-uri "html5-parser" version))
>>> +              (sha256
>>> +               (base32
>>> +                "1d8sxhl41ffh7qlk7wlsy17xw6slzx5v1yna9s72wx5qrpaa3wxr"))))
>>> +    (build-system python-build-system)
>>> +    (native-inputs
>>> +     `(("pkg-config" ,pkg-config)))
>>> +    (inputs
>>> +     `(("libxml2" ,libxml2)))
>>> +    (propagated-inputs
>>> +     `(("python-lxml" ,python-lxml)
>>> +       ("python-beautifulsoup4" ,python-beautifulsoup4)))
>>> +    (home-page "https://html5-parser.readthedocs.io")
>>> +    (synopsis "Fast C-based HTML5 parsing for Python")
>>> +    (description "This package provides a fast implementation of the HTML5
>>> +parsing spec for Python.  Parsing is done in C using a variant of the gumbo
>>> +parser.  The gumbo parse tree is then transformed into an lxml tree, also in
>>> +C, yielding parse times that can be a thirtieth of the html5lib parse times.")
>>> +    (license license:asl2.0)))
>>
>> The files 'src/as-libxml.[ch]' are GPL3.  Everything else in this series LGTM!
>
> Oh, it seems this is the case for as-python-tree.[ch], not as-libxml.[ch].
>
> Good catch!  I'll update the license list and push this patch series.
>
> Thanks for your time.
>
> Kind regards,
> Roel Janssen

I pushed the three patches with the above license indication changes.

Thanks!

Kind regards,
Roel Janssen




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Wed, 04 Oct 2017 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 7 years and 338 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.