search

nlfiedler / magick-rust

Rust bindings for ImageMagick
https://crates.io/crates/magick_rust
Apache License 2.0
246 stars 63 forks source link

Update imagemmagick checks to support 7.1 #82

Closed brownjohnf closed 3 years ago

brownjohnf commented 3 years ago

On Arch Linux, imagemagick is currently at 7.1.0. I tried increasing the max version and the version used in the Dockerfile to 7.1 and it appears to compile and tests pass. Version used for local testing:

Linux 5.12.12-arch1-1 #1 SMP PREEMPT Fri, 18 Jun 2021 21:59:22 +0000 x86_64 GNU/Linux

Version: ImageMagick 7.1.0-1 Q16 x86_64 2021-06-21 https://imagemagick.org
Copyright: (C) 1999-2021 ImageMagick Studio LLC
License: https://imagemagick.org/script/license.php
Features: Cipher DPC HDRI Modules OpenMP(4.5)
Delegates (built-in): bzlib cairo djvu fontconfig freetype heic jbig jng jp2 jpeg lcms lqr ltdl lzma openexr pangocairo png raqm raw rsvg tiff webp wmf x xml zip zlib
brownjohnf commented 3 years ago

I'm not sure how to determine whether or not this is safe to merge... As I mention in the commit, it compiles and tests pass. I'm also able to use it in my own projects successfully.

nlfiedler commented 3 years ago

Indeed, it works for me as well. Thank you.

brownjohnf commented 2 years ago

Any idea when this will be released?

nlfiedler commented 2 years ago

It went out 3 weeks ago, is it not working for you? I did forget to "draft" the release on GitHub, which I just fixed, but the tag was pushed to GitHub and the crate was updated on crates.io on August 10th.

brownjohnf commented 2 years ago

Oh interesting. It's probably on my end then, but I didn't see it when I went to upgrade. Just ignore me!