[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1)
From: |
Andy Tai |
Subject: |
[bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1) |
Date: |
Mon, 8 Feb 2021 15:14:22 -0800 |
updated patch attached
On Mon, Feb 8, 2021 at 1:36 PM Jelle Licht <jlicht@posteo.net> wrote:
>
>
> > * gnu/packages/ocr.scm (tesseract-ocr): update to 4.1.1
> ^ This is usually capitalised.
capitalized
> > [snip]
> > (define-public tesseract-ocr
> > + ;; some useful commits beyond last official stable release in release
> > branch
> > + (let ((commit "97079fa353557af6df86fd20b5d2e0dff5d8d5df"))
> > (package
> > (name "tesseract-ocr")
> > - (version "3.04.01")
> > + (version "4.1.1")
> ^ Since we are not _actually_ using version 4.1.1, but a
> later commit, you could use
> `(git-version "4.1.1" revision commit)' here instead
changed as suggested
>
> > (source
> > (origin
> > (method git-fetch)
> > (uri (git-reference
> > (url "https://github.com/tesseract-ocr/tesseract")
> > - (commit version)))
> > - (file-name (git-file-name name version))
> > + (commit commit)
> > + ;; Fetch git submodules
> > + (recursive? #t)))
>
> Instead of stating what the code does, would you consider adding a
> comment why this is needed?
>
commented as suggested
> > + (file-name (string-append name "-" version "-" commit "-checkout"))
> ^ You can use `(git-file-name name version)' here.
>
changed as suggested
> > (sha256
> > - (base32 "0h1x4z1h86n2gwknd0wck6gykkp99bmm02lg4a47a698g4az6ybv"))))
> > + (base32 "0axwla82fpzp86lc553wp3hk0fz5dylw4as0jbf4hkqcyajlbzp4"))))
> > (build-system gnu-build-system)
> > (inputs
> > - `(("leptonica" ,leptonica)))
> > + `( ("cairo" ,cairo)
> > + ("icu" ,icu4c)
> > + ("leptonica" ,leptonica)
> > + ("pango" ,pango)))
> > + (native-inputs
> > + `(("autoconf" ,autoconf)
> > + ("autoconf-archive" ,autoconf-archive)
> > + ("automake" ,automake)
> > + ("googletest" ,googletest)
> > + ("libtool" ,libtool)
> > + ("pkg-config" ,pkg-config)))
> > (arguments
> > '(#:configure-flags
> > (let ((leptonica (assoc-ref %build-inputs "leptonica")))
> > - (list (string-append "LIBLEPT_HEADERSDIR=" leptonica
> > "/include")))))
> > + (list (string-append "LIBLEPT_HEADERSDIR=" leptonica "/include")))
> > + ;; some test, applybox_test fails to build
> > + #:tests? #f))
> 2 nits: Is it possible to patch or disable only the failing tests?
tests failing to build probably due to some issue with parallel
builds; did not dig into it as probably will take much time; will be
TODO if time allows)
> Is there a reason googletest is added if testing is subsequently disabled?
>
removed
> Thanks,
> - Jelle
--
Andy Tai, atai@atai.org, Skype: licheng.tai, Line: andy_tai, WeChat: andytai1010
Year 2021 民國110年
自動的精神力是信仰與覺悟
自動的行為力是勞動與技能
0001-gnu-tesseract-ocr-Update-to-4.1.1.patch
Description: Text Data
- [bug#46377] [PATCH] gnu: tesseract-ocr: update to 4.1.1, Andy Tai, 2021/02/08
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1, Andy Tai, 2021/02/08
- Message not available
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/08
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Jelle Licht, 2021/02/08
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1),
Andy Tai <=
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Jelle Licht, 2021/02/09
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/09
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Jelle Licht, 2021/02/09
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/09
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/10
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/10
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Jelle Licht, 2021/02/11
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/11
- bug#46376: [PATCH] gnu: tesseract-ocr: update to 4.1.1), Jelle Licht, 2021/02/13
- [bug#46376] [PATCH] gnu: tesseract-ocr: update to 4.1.1), Andy Tai, 2021/02/13