commit:     3cb38722606339551a29bc92c19c3506d3571900
Author:     Thomas Deutschmann <whissi <AT> gentoo <DOT> org>
AuthorDate: Tue Dec 11 13:30:00 2018 +0000
Commit:     Thomas Deutschmann <whissi <AT> gentoo <DOT> org>
CommitDate: Tue Dec 11 13:44:38 2018 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3cb38722

media-gfx/imagemagick: use "identify" utility in test phase in IM 6

"magick" utility is IM 7+ only [Link 1].

Link 1: 
https://github.com/ImageMagick/ImageMagick/commit/fa1e43d6abadec9f10938d14052df38ed570f99f
Reported-by: Hans de Graaff <graaff <AT> gentoo.org>
Package-Manager: Portage-2.3.52, Repoman-2.3.12
Signed-off-by: Thomas Deutschmann <whissi <AT> gentoo.org>

 media-gfx/imagemagick/imagemagick-6.9.10.14.ebuild | 12 +++++++-----
 media-gfx/imagemagick/imagemagick-6.9.10.16.ebuild | 12 +++++++-----
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/media-gfx/imagemagick/imagemagick-6.9.10.14.ebuild 
b/media-gfx/imagemagick/imagemagick-6.9.10.14.ebuild
index ba0ba14a935..1cc2c6cf11a 100644
--- a/media-gfx/imagemagick/imagemagick-6.9.10.14.ebuild
+++ b/media-gfx/imagemagick/imagemagick-6.9.10.14.ebuild
@@ -5,7 +5,8 @@ EAPI="6"
 
 inherit eapi7-ver eutils flag-o-matic libtool perl-functions toolchain-funcs 
multilib
 
-MY_P=ImageMagick-$(ver_rs 3 '-')
+MY_PV="$(ver_rs 3 '-')"
+MY_P="ImageMagick-${MY_PV}"
 
 DESCRIPTION="A collection of tools and libraries for many image formats"
 HOMEPAGE="https://www.imagemagick.org/";
@@ -173,13 +174,14 @@ src_test() {
                die "Failed to install default blank policy.xml in 
'${_im_local_config_home}'"
 
        local im_command= IM_COMMANDS=()
-       IM_COMMANDS+=( "magick -version" ) # Verify that we are using version 
we just built
-       IM_COMMANDS+=( "magick -list policy" ) # Verify that policy.xml is used
+       IM_COMMANDS+=( "identify -version | grep -q -- \"${MY_PV}\"" ) # Verify 
that we are using version we just built
+       IM_COMMANDS+=( "identify -list policy" ) # Verify that policy.xml is 
used
        IM_COMMANDS+=( "emake check" ) # Run tests
 
        for im_command in "${IM_COMMANDS[@]}"; do
-               "${S}"/magick.sh \
-               ${im_command} || die
+               eval "${S}"/magick.sh \
+                       ${im_command} || \
+                       die "Failed to run \"${im_command}\""
        done
 }
 

diff --git a/media-gfx/imagemagick/imagemagick-6.9.10.16.ebuild 
b/media-gfx/imagemagick/imagemagick-6.9.10.16.ebuild
index 59b9a343932..51d1439ebc5 100644
--- a/media-gfx/imagemagick/imagemagick-6.9.10.16.ebuild
+++ b/media-gfx/imagemagick/imagemagick-6.9.10.16.ebuild
@@ -5,7 +5,8 @@ EAPI="6"
 
 inherit eapi7-ver eutils flag-o-matic libtool perl-functions toolchain-funcs 
multilib
 
-MY_P=ImageMagick-$(ver_rs 3 '-')
+MY_PV="$(ver_rs 3 '-')"
+MY_P="ImageMagick-${MY_PV}"
 
 DESCRIPTION="A collection of tools and libraries for many image formats"
 HOMEPAGE="https://www.imagemagick.org/";
@@ -173,13 +174,14 @@ src_test() {
                die "Failed to install default blank policy.xml in 
'${_im_local_config_home}'"
 
        local im_command= IM_COMMANDS=()
-       IM_COMMANDS+=( "magick -version" ) # Verify that we are using version 
we just built
-       IM_COMMANDS+=( "magick -list policy" ) # Verify that policy.xml is used
+       IM_COMMANDS+=( "identify -version | grep -q -- \"${MY_PV}\"" ) # Verify 
that we are using version we just built
+       IM_COMMANDS+=( "identify -list policy" ) # Verify that policy.xml is 
used
        IM_COMMANDS+=( "emake check" ) # Run tests
 
        for im_command in "${IM_COMMANDS[@]}"; do
-               "${S}"/magick.sh \
-               ${im_command} || die
+               eval "${S}"/magick.sh \
+                       ${im_command} || \
+                       die "Failed to run \"${im_command}\""
        done
 }
 

Reply via email to