[PATCH] gnu: vips: Update to 8.14.5.

  • Open
  • quality assurance status badge
Details
3 participants
  • Hendursaga
  • Mathieu Othacehe
  • Sharlatan Hellseher
Owner
unassigned
Submitted by
Hendursaga
Severity
normal
H
H
Hendursaga wrote on 18 Sep 2023 22:04
(address . guix-patches@gnu.org)
1f13ccf386121689ba5be643290293a5769b21e5.1695066872.git.hendursaga@aol.com
* gnu/packages/image-processing.scm (vips): Update to 8.14.5.
[source]: Switch to git-fetch.
[build-system]: Switch to meson-build-system.
[arguments]: Build all available docs, fix finding niftilib headers.
[native-inputs]: Add doxygen, font-liberation, gettext-minimal,
glib:bin, graphviz, gtk-doc, and pandoc, for building docs.
[inputs]: Add additional dependencies cfitsio, fontconfig, libheif,
libimagequant, libjxl, and openjpeg.
[home-page]: Update URL.
---
gnu/packages/image-processing.scm | 103 ++++++++++++++++++++----------
1 file changed, 68 insertions(+), 35 deletions(-)

Toggle diff (147 lines)
diff --git a/gnu/packages/image-processing.scm b/gnu/packages/image-processing.scm
index 8fc93a8842..ee73260fb7 100644
--- a/gnu/packages/image-processing.scm
+++ b/gnu/packages/image-processing.scm
@@ -49,9 +49,11 @@ (define-module (gnu packages image-processing)
#:use-module (guix build-system qt)
#:use-module (guix build-system cmake)
#:use-module (guix build-system gnu)
+ #:use-module (guix build-system meson)
#:use-module (guix build-system python)
#:use-module (gnu packages)
#:use-module (gnu packages algebra)
+ #:use-module (gnu packages astronomy)
#:use-module (gnu packages bison)
#:use-module (gnu packages boost)
#:use-module (gnu packages check)
@@ -61,11 +63,13 @@ (define-module (gnu packages image-processing)
#:use-module (gnu packages docbook)
#:use-module (gnu packages documentation)
#:use-module (gnu packages flex)
+ #:use-module (gnu packages fonts)
#:use-module (gnu packages fontutils)
#:use-module (gnu packages game-development)
#:use-module (gnu packages gcc)
#:use-module (gnu packages gd)
#:use-module (gnu packages geo)
+ #:use-module (gnu packages gettext)
#:use-module (gnu packages ghostscript)
#:use-module (gnu packages gimp)
#:use-module (gnu packages gl)
@@ -75,6 +79,7 @@ (define-module (gnu packages image-processing)
#:use-module (gnu packages graphviz)
#:use-module (gnu packages gstreamer)
#:use-module (gnu packages gtk)
+ #:use-module (gnu packages haskell-xyz)
#:use-module (gnu packages icu4c)
#:use-module (gnu packages image)
#:use-module (gnu packages imagemagick)
@@ -748,41 +753,69 @@ (define-public opencv
(define-public vips
(package
(name "vips")
- (version "8.13.1")
- (source
- (origin
- (method url-fetch)
- (uri (string-append
- "https://github.com/libvips/libvips/releases/download/v"
- version "/vips-" version ".tar.gz"))
- (sha256
- (base32 "00kp3439jcqv9l2gcjg88xzvlq8clv54z1m3x66i3chvarz7ndxd"))))
- (build-system gnu-build-system)
- (native-inputs
- (list gobject-introspection pkg-config))
- (inputs
- (list expat
- fftw
- giflib
- glib
- hdf5
- imagemagick
- lcms
- libexif
- libgsf
- libjpeg-turbo
- libpng
- (librsvg-for-system)
- libtiff
- libxml2
- libwebp
- matio
- niftilib
- openexr-2
- orc
- pango
- poppler))
- (home-page "https://libvips.github.io/libvips/")
+ (version "8.14.5")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/libvips/libvips")
+ (commit (string-append "v" version))))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32
+ "020hksb5c3xmx30xg65csph0vyxjvxf3ijspdxc9njnkmih9ffsg"))))
+ (build-system meson-build-system)
+ (arguments
+ (list #:configure-flags #~(list "-Ddoxygen=true" "-Dgtk_doc=true"
+ (string-append "-Dnifti-prefix-dir="
+ #$(this-package-input
+ "niftilib")))
+ #:phases #~(modify-phases %standard-phases
+ (add-after 'unpack 'set-home-directory
+ (lambda _
+ ;; Prevent fontconfig (via Graphviz) complaining
+ ;; about no writable cache directories.
+ (setenv "HOME" "/tmp")))
+ (add-after 'unpack 'fix-nifti-include
+ (lambda _
+ (substitute* "meson.build"
+ ((" / 'nifti'")
+ "")))))))
+ (native-inputs (list doxygen
+ font-liberation ;a URW Type 1 font, for Graphviz
+ gettext-minimal
+ gobject-introspection
+ (list glib "bin") ;glib-mkenums
+ graphviz ;full install needed due to pango
+ gtk-doc
+ pandoc ;convert markdown to DocBook for gtk-doc
+ pkg-config))
+ (inputs (list cfitsio
+ expat
+ fftw
+ fontconfig
+ giflib
+ hdf5
+ imagemagick
+ lcms
+ libexif
+ libgsf
+ libheif
+ libimagequant
+ libjpeg-turbo
+ libjxl
+ libpng
+ (librsvg-for-system)
+ libtiff
+ libxml2
+ libwebp
+ matio
+ niftilib
+ openexr-2
+ openjpeg
+ orc
+ pango
+ poppler))
+ (home-page "https://www.libvips.org/")
(synopsis "Multithreaded image processing system with low memory needs")
(description
"VIPS is a demand-driven, horizontally threaded image processing library.

base-commit: 3d9ebc7b2ed24312fd6a0916c203f7b86d57753d
--
2.41.0
M
M
Mathieu Othacehe wrote on 14 Oct 2023 10:26
(name . Hendursaga)(address . hendursaga@aol.com)(address . 66094@debbugs.gnu.org)
87wmvpehk5.fsf_-_@gnu.org
Hello,

https://qa.guix.gnu.org/issue/66094reports a new linter warning:

Toggle snippet (3 lines)
label 'glib' does not match package name 'glib:bin'

as well as new newly broken package: nip2 with this change.

Could you please have a look :)?

Thanks,

Mathieu
S
S
Sharlatan Hellseher wrote on 19 Nov 2023 20:46
(address . 66094@debbugs.gnu.org)
CAO+9K5qoG7aCSU9KKTiE3sA+bnBvZDNbYwmwbM3ZbSX7PZnL2Q@mail.gmail.com
Hi,

Thank you for the patch.

It contains quite a long list of modifications and makes the review harder
to follow.
May you split the patch into logical sub patches, please. It may help to
rework QA warnings more explicit.

Some recommendations of grouping:
- update only version and hash, add missing input to complete the build
- any cosmetic changes, like update synopsis, description, homepage
- change in build system and fixing related issues
- extend build in arguments, which adds new outputs
- enabling/disabling tests

Thanks,
Oleg
Attachment: file
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 66094@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 66094
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch