Bug 704706 – media-gfx/iscan-3.61.0 USE="imagemagick -graphicsmagick" with media-gfx/imagemagick-? - doc-locate.cpp:265:25: error: ‘class Magick::Geometry’ has no member named ‘xNegative’

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Chris Smith 2020-01-03 15:08:51 UTC]

media-gfx/iscan-3.61.0 fails to compile, maybe an imagemagick upgrade caused an ABI mismatch

Reproducible: Always

Actual Results:  
g++ -DHAVE_CONFIG_H   -I.. -I/usr/include -fopenmp -DMAGICKCORE_HDRI_ENABLE=0 -DMAGICKCORE_QUANTUM_DEPTH=16 -fopenmp -DMAGICKCORE_HDRI_ENABLE=0 -DMAGICKCORE_QUANTUM_DEPTH=16 -fopenmp -DMAGICKCORE_HDRI_ENABLE=0 -DMAGICKCORE_QUANTUM_DEPTH=16 -I/usr/include/ImageMagick-7 -I/usr/include -Wall -Werror  -O2 -march=native -fomit-frame-pointer -pipe -c -o doc_locate-doc-locate.o `test -f 'doc-locate.cpp' || echo './'`doc-locate.cpp
doc-locate.cpp: In member function ‘Magick::Geometry {anonymous}::locator::cropdoc() const’:
doc-locate.cpp:265:25: error: ‘class Magick::Geometry’ has no member named ‘xNegative’
  265 |       if (clone.page ().xNegative ())
      |                         ^~~~~~~~~
doc-locate.cpp:269:25: error: ‘class Magick::Geometry’ has no member named ‘yNegative’
  269 |       if (clone.page ().yNegative ())
      |                         ^~~~~~~~~
make[3]: *** [Makefile:939: doc_locate-doc-locate.o] Error 1


emerge --info '=media-gfx/iscan-3.61.0::gentoo'
Portage 2.3.84 (python 3.6.10-final-0, default/linux/amd64/17.1/desktop/plasma/systemd, gcc-9.2.0, glibc-2.30-r3, 5.4.7-gentoo x86_64)
=================================================================
                         System Settings
=================================================================
System uname: Linux-5.4.7-gentoo-x86_64-Intel-R-_Core-TM-_i7-3770K_CPU_@_3.50GHz-with-gentoo-2.6
KiB Mem:    32837472 total,  23284920 free
KiB Swap:    6291452 total,   6291452 free
Timestamp of repository gentoo: Fri, 03 Jan 2020 14:25:45 +0000
Head commit of repository gentoo: 243ebe242f63303cc7df20d765c3498fde713744

Head commit of repository audio-overlay: 3330e7083807346cae900deb5d09694840cdf6e9

sh bash 5.0_p11
ld GNU ld (Gentoo 2.33.1 p1) 2.33.1
app-shells/bash:          5.0_p11::gentoo
dev-java/java-config:     2.2.0-r4::gentoo
dev-lang/perl:            5.30.1::gentoo
dev-lang/python:          2.7.17-r1::gentoo, 3.6.10::gentoo, 3.7.6::gentoo, 3.8.1::gentoo
dev-util/cmake:           3.16.2-r1::gentoo
dev-util/pkgconfig:       0.29.2::gentoo
sys-apps/baselayout:      2.6-r1::gentoo
sys-apps/sandbox:         2.18::gentoo
sys-devel/autoconf:       2.13-r1::gentoo, 2.69-r5::gentoo
sys-devel/automake:       1.16.1-r2::gentoo
sys-devel/binutils:       2.33.1::gentoo
sys-devel/gcc:            9.2.0-r3::gentoo
sys-devel/gcc-config:     2.1::gentoo
sys-devel/libtool:        2.4.6-r6::gentoo
sys-devel/make:           4.2.1-r4::gentoo
sys-kernel/linux-headers: 5.4::gentoo (virtual/os-headers)
sys-libs/glibc:           2.30-r3::gentoo
Repositories:

gentoo
    location: /usr/portage
    sync-type: git
    sync-uri: https://anongit.gentoo.org/git/repo/sync/gentoo.git
    priority: -1000

audio-overlay
    location: /usr/local/portage/audio-overlay
    sync-type: git
    sync-uri: https://github.com/gentoo-audio/audio-overlay.git
    masters: gentoo

tcgOverlay
    location: /usr/local/portage
    masters: gentoo

ACCEPT_KEYWORDS="amd64 ~amd64"
ACCEPT_LICENSE="* -@EULA"
CBUILD="x86_64-pc-linux-gnu"
CC="gcc"
CFLAGS="-O2 -march=native -fomit-frame-pointer -pipe"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/share/config /usr/share/gnupg/qualified.txt"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/dconf /etc/env.d /etc/fonts/fonts.conf /etc/gconf /etc/gentoo-release /etc/revdep-rebuild /etc/sandbox.d /etc/terminfo"
CXX="g++"
CXXFLAGS="-O2 -march=native -fomit-frame-pointer -pipe"
DISTDIR="/usr/portage/distfiles"
EMERGE_DEFAULT_OPTS="--with-bdeps y --autounmask-keep-masks"
ENV_UNSET="DBUS_SESSION_BUS_ADDRESS DISPLAY GOBIN PERL5LIB PERL5OPT PERLPREFIX PERL_CORE PERL_MB_OPT PERL_MM_OPT XAUTHORITY XDG_CACHE_HOME XDG_CONFIG_HOME XDG_DATA_HOME XDG_RUNTIME_DIR"
FCFLAGS="-O2 -pipe"
FEATURES="assume-digests binpkg-docompress binpkg-dostrip binpkg-logs clean-logs config-protect-if-modified distlocks ebuild-locks fixlafiles ipc-sandbox merge-sync multilib-strict network-sandbox news parallel-fetch pid-sandbox preserve-libs protect-owned sandbox sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch userpriv usersandbox usersync xattr"
FFLAGS="-O2 -pipe"
GENTOO_MIRRORS="http://gentoo.osuosl.org/"
LANG="en_US.utf8"
LDFLAGS="-Wl,-O2 -Wl,--as-needed"
LINGUAS="en_US en"
MAKEOPTS="V=1 -j1"
PKGDIR="/usr/portage/packages"
PORTAGE_CONFIGROOT="/"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --omit-dir-times --compress --force --whole-file --delete --stats --human-readable --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages --exclude=/.git"
PORTAGE_TMPDIR="/var/tmp"
USE="X a52 aac aalib acl acpi activities aes alsa amd64 amr apache2 audiofile avx bash-completion berkdb bluray branding btrfs bzip2 cairo caps cdaudio cdda cddb cdparanoia cdr cdrdao chm chromecast cleartype cli corefonts crypt cups curl cxx dbus declarative djvu dri dssi dts dv dvb dvd dvdr ebook egl emboss encode exif fam fdk ffmpeg flac fortran gdbm gif gles2 gmp google gphoto2 gpm gstreamer gtk gtk3 iconv icu ieee1394 imagemagick imap imlib introspection ipv6 jack java jpeg jpeg2k kde kipi kpathsea kwallet ladspa lame lash lcms libffi libinput libnotify libtirpc linguas_en_US lv2 lzma mad marble midi mmap mmx mmxext mng mozilla mp3 mp4 mpeg mtp musicbrainz ncurses network nls nodrm nptl nsplugin nvenc nvidia ofx ogg opencl openexr opengl openmp opus pam pango pcre pdf phonon plasma png policykit popcnt ppds pulseaudio qml qt5 readline rtmp sasl scanner sdl seccomp semantic-desktop slp smp sndfile speex spell split-usr sse sse2 sse3 sse4_1 sse4_2 ssl ssse3 startup-notification subversion svg systemd theora threads tiff tk transparency truetype type1 udev udisks unicode unwind upower usb v4l vaapi vcd vdpau vim-syntax vorbis vpx vst wav wavpack wayland webp widgets wmf wxwidgets x264 x265 xattr xcb xcomposite xine xkb xml xscreensaver xv xvid xvmc zlib" ABI_X86="64" ADA_TARGET="gnat_2018" ALSA_CARDS="hda-intel hdsp hpet hrtimer rtctimer seqdummy virmidi" APACHE2_MODULES="authn_core authz_core socache_shmcb unixd actions alias auth_basic authn_alias authn_anon authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user autoindex cache cgi cgid dav dav_fs dav_lock deflate dir disk_cache env expires ext_filter file_cache filter headers include info log_config logio mem_cache mime mime_magic negotiation rewrite setenvif speling status unique_id userdir usertrack vhost_alias" CALLIGRA_FEATURES="author braindump flow karbon kexi krita sheets stage words" CAMERAS="*" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" CPU_FLAGS_X86="aes avx mmx mmxext popcnt sse sse2 sse3 sse4_1 sse4_2 ssse3" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock greis isync itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf skytraq superstar2 timing tsip tripmate tnt ublox ubx" GRUB_PLATFORMS="efi-64 pc" INPUT_DEVICES="libinput" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LIBREOFFICE_EXTENSIONS="presenter-console presenter-minimizer" NETBEANS_MODULES="apisupport cnd groovy gsf harness ide identity j2ee java mobility nb php profiler soa visualweb webcommon websvccommon xml" NGINX_MODULES_HTTP="autoindex fancyindex fastcgi gzip" OFFICE_IMPLEMENTATION="libreoffice" PHP_TARGETS="php7-2" POSTGRES_TARGETS="postgres10 postgres11" PYTHON_SINGLE_TARGET="python3_6" PYTHON_TARGETS="python2_7 python3_6" QEMU_SOFTMMU_TARGETS="i386 x86_64" QEMU_USER_TARGETS="i386 x86_64" RUBY_TARGETS="(-ruby23) (-ruby24) (-ruby25) ruby26" SANE_BACKENDS="epson epson2" USERLAND="GNU" VIDEO_CARDS="nvidia nouveau" XTABLES_ADDONS="quota2 psd pknock lscan length2 ipv4options ipset ipp2p iface geoip fuzzy condition tee tarpit sysrq steal rawnat logmark ipmark dhcpmac delude chaos account"
Unset:  CPPFLAGS, CTARGET, INSTALL_MASK, LC_ALL, PORTAGE_BINHOST, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS

[Comment 1 Chris Smith 2020-01-03 15:10:15 UTC]

emerge -pqv '=media-gfx/iscan-3.61.0::gentoo'
[ebuild     U ] media-gfx/iscan-3.61.0 [2.30.4.2] USE="imagemagick%* -graphicsmagick% -gui% (-X%*) (-doc%) (-gimp%) (-nls%*)" L10N="(-ja%)"

[Comment 2 Chris Smith 2020-01-03 15:16:00 UTC]

Created attachment 602410 [details]
build log

[Comment 3 Jeroen Roovers (RETIRED) 2020-01-03 15:17:39 UTC]

Which version of media-gfx/imagemagick do you have?

[Comment 4 Chris Smith 2020-01-03 15:19:16 UTC]

(In reply to Jeroen Roovers from comment #3)
> Which version of media-gfx/imagemagick do you have?

media-gfx/imagemagick-7.0.9.8

[Comment 5 Jens Reinemuth 2020-01-07 10:31:51 UTC]

Bug has been addressed here: 
https://github.com/sirjaren/gentoo-overlay/commit/222aa3e123ee771d8997f6931997e3143186e0d6#diff-da5891e0da9de96256fdfe3125d95d9e

The file utsushi-9999-magick-pp.patch works to compile iscan

[Comment 6 Matthew Schultz 2020-01-07 15:09:57 UTC]

There is no such thing as media-gfx/iscan-3.61.0.  That version neither exists in the portage tree or upstream.  So there's nothing for me to do here.

[Comment 7 Matthew Schultz 2020-01-07 15:15:37 UTC]

I stand corrected.  I wasn't the one who bumped this package or found this new version of iscan.  Who wrote this new ebuild?

[Comment 8 Chris Smith 2020-01-07 15:18:43 UTC]

$ ls /usr/portage/media-gfx/iscan
Manifest  files  iscan-2.30.3.1.ebuild  iscan-2.30.4.2.ebuild  iscan-3.61.0.ebuild  metadata.xml

[Comment 9 Chris Smith 2020-01-07 15:20:09 UTC]

$ cat /usr/portage/media-gfx/iscan/iscan-3.61.0.ebuild
# Copyright 1999-2019 Gentoo Authors
# Distributed under the terms of the GNU General Public License v2

EAPI=7

DESCRIPTION="EPSON Image Scan v3 for Linux"
HOMEPAGE="http://support.epson.net/linux/en/imagescanv3.php"

SRC_URI="http://support.epson.net/linux/src/scanner/imagescanv3/common/imagescan_${PV}.orig.tar.gz"

LICENSE="GPL-3+"

SLOT="0"

IUSE="graphicsmagick gui imagemagick"

KEYWORDS="~amd64 ~x86"

DEPEND="
        dev-libs/boost:=
        media-gfx/sane-backends
        media-libs/tiff
        virtual/libusb:1
        virtual/jpeg
        gui? ( dev-cpp/gtkmm:= )
        imagemagick? (
                !graphicsmagick? ( media-gfx/imagemagick:= )
                graphicsmagick? ( media-gfx/graphicsmagick:= )
        )
"
RDEPEND=${DEPEND}

S="${WORKDIR}/utsushi-0.$(ver_cut 2-3)"

src_configure() {
        econf \
                $(use_with gui gtkmm) \
                --with-jpeg \
                $(use_with imagemagick magick) \
                $(use_with imagemagick magick-pp) \
                --with-tiff \
                --with-sane \
                --with-boost=yes
}

src_install() {
        default
        find "${ED}" -name '*.la' -delete || die

        elog "If you encounter problems with media-gfx/xsane when scanning (e.g., bad resolution),"
        elog "please try the built-in GUI and kde-misc/skanlite first before reporting bugs."
}

[Comment 10 Conrad Kostecki 2020-01-07 15:21:14 UTC]

(In reply to Matthew Schultz from comment #7)
> Who wrote this new ebuild?

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=407dd984406bb2881da27451a082758717d36010

[Comment 11 Matthew Schultz 2020-01-07 15:24:39 UTC]

I'm not against fixing bugs but since he's a gentoo developer, he wrote this new ebuild for this new version and he added himself as a maintainer for this package, I think it would be better if bugs related to this version were assigned to him.

[Comment 12 Chris Smith 2020-01-07 15:28:06 UTC]

(In reply to Matthew Schultz from comment #6)
> There is no such thing as media-gfx/iscan-3.61.0.  That version neither
> exists in the portage tree or upstream.  So there's nothing for me to do
> here.
(In reply to Matthew Schultz from comment #11)
> I'm not against fixing bugs but since he's a gentoo developer, he wrote this
> new ebuild for this new version and he added himself as a maintainer for
> this package, I think it would be better if bugs related to this version
> were assigned to him.

But clearly it exists and is in the portage tree.

[Comment 13 Matthew Schultz 2020-01-07 15:33:34 UTC]

(In reply to Chris Smith from comment #12)
> (In reply to Matthew Schultz from comment #6)
> > There is no such thing as media-gfx/iscan-3.61.0.  That version neither
> > exists in the portage tree or upstream.  So there's nothing for me to do
> > here.
> (In reply to Matthew Schultz from comment #11)
> > I'm not against fixing bugs but since he's a gentoo developer, he wrote this
> > new ebuild for this new version and he added himself as a maintainer for
> > this package, I think it would be better if bugs related to this version
> > were assigned to him.
> 
> But clearly it exists and is in the portage tree.

Yes my mistake.  I've been maintaining this package for a while.  I wasn't aware of a new version existing since apparently the upstream url changed and I did not bump it.

[Comment 14 Andreas Sturmlechner 2020-02-12 14:28:39 UTC]

*** Bug 706922 has been marked as a duplicate of this bug. ***

[Comment 15 Andreas Sturmlechner 2020-02-12 14:33:45 UTC]

(In reply to Matthew Schultz from comment #13)
> I've been maintaining this package for a while.
Does that mean you no longer wish to maintain it? Because then proxy-maint CC makes no sense here and we will drop you from metadata.xml.

[Comment 16 Matthew Schultz 2020-02-12 14:41:15 UTC]

(In reply to Andreas Sturmlechner from comment #15)
> (In reply to Matthew Schultz from comment #13)
> > I've been maintaining this package for a while.
> Does that mean you no longer wish to maintain it? Because then proxy-maint
> CC makes no sense here and we will drop you from metadata.xml.

Since Andreas bumped the package to version 3, he also listed himself as maintainer of this package.  As a result, I agree that proxy maintenance is no longer required for this package.  You can remove me as maintainer as well.

[Comment 17 Andreas Sturmlechner 2020-02-12 16:48:34 UTC]

We don't have a mechanism to split maintenance between versions of the same package; please note that dilfridge added himself as some kind of last resort maintainer if I got that right, so it is purely up to you if you want to give up maintenance.

Assuming you would have made the version bump as well at some point by request in bugzilla, I would look at it as work saved rather than breach of territory. The bug is after all not in the ebuild but in upstream sources to fix.

[Comment 18 Matthew Schultz 2020-02-12 16:57:00 UTC]

(In reply to Andreas Sturmlechner from comment #17)
> We don't have a mechanism to split maintenance between versions of the same
> package; please note that dilfridge added himself as some kind of last
> resort maintainer if I got that right, so it is purely up to you if you want
> to give up maintenance.
> 
> Assuming you would have made the version bump as well at some point by
> request in bugzilla, I would look at it as work saved rather than breach of
> territory. The bug is after all not in the ebuild but in upstream sources to
> fix.

What's a last resort maintainer?  If a gentoo dev is going to bump the package and add himself as maintainer, why would proxy maintenance still be necessary?

[Comment 19 Matthew Schultz 2020-02-12 17:02:37 UTC]

(In reply to Andreas Sturmlechner from comment #17)
> We don't have a mechanism to split maintenance between versions of the same
> package; please note that dilfridge added himself as some kind of last
> resort maintainer if I got that right, so it is purely up to you if you want
> to give up maintenance.
> 
> Assuming you would have made the version bump as well at some point by
> request in bugzilla, I would look at it as work saved rather than breach of
> territory. The bug is after all not in the ebuild but in upstream sources to
> fix.

I've seen several gentoo bugs appear as a result of this version bump.  They're probably all upstream problems but since I don't anything about this new version and Andreas was the one to bump it, I think he should be the one responsible for the problems associated to it.  Had I bumped this package, I would have tested it first to make sure it compiles properly because it doesn't even compile on my machine.  I would not have committed it if I didn't have a reasonable expectation that it would compile and work.

[Comment 20 Matthew Schultz 2020-02-12 17:09:48 UTC]

(In reply to Matthew Schultz from comment #19)
> (In reply to Andreas Sturmlechner from comment #17)
> > We don't have a mechanism to split maintenance between versions of the same
> > package; please note that dilfridge added himself as some kind of last
> > resort maintainer if I got that right, so it is purely up to you if you want
> > to give up maintenance.
> > 
> > Assuming you would have made the version bump as well at some point by
> > request in bugzilla, I would look at it as work saved rather than breach of
> > territory. The bug is after all not in the ebuild but in upstream sources to
> > fix.
> 
> I've seen several gentoo bugs appear as a result of this version bump. 
> They're probably all upstream problems but since I don't anything about this
> new version and Andreas was the one to bump it, I think he should be the one
> responsible for the problems associated to it.  Had I bumped this package, I
> would have tested it first to make sure it compiles properly because it
> doesn't even compile on my machine.  I would not have committed it if I
> didn't have a reasonable expectation that it would compile and work.

I suppose I could remain maintainer but it's not nice to bump packages without coordination or testing.  Andreas didn't notify proxy maintenance or me when he bumped this package and somehow now I'm responsible for all the bugs spawned by this.

[Comment 21 Andreas Sturmlechner 2020-02-12 20:09:12 UTC]

(In reply to Matthew Schultz from comment #19)
> I've seen several gentoo bugs appear as a result of this version bump.
That's life. ;) I see 3 bugs that aren't duplicates:
- 1 test failure from tinderbox - okay, that happens, not the end of the world
- 1 (fixed) build failure with libjpeg (non-turbo) - almost no one is using this*
- 1 failure with imagemagick-7 - which you won't at all get with 6 or graphicsmagick

* no maintainer can test all the things when they add packages in ~arch. I certainly don't test for libressl, to give one example, and I don't have a full chroot copy to test with libjeg (non-turbo).

So, you shouldn't assume this ebuild wasn't tested just because the particular package set on your machine is reproducing this one.

[Comment 22 Larry the Git Cow 2020-02-16 13:41:50 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=985c92af4730d864e86fa87746185b0246e9db93

commit 985c92af4730d864e86fa87746185b0246e9db93
Author:     Andreas Sturmlechner <[e-mail]>
AuthorDate: 2020-02-16 13:16:03 +0000
Commit:     Andreas Sturmlechner <[e-mail]>
CommitDate: 2020-02-16 13:41:33 +0000

    media-gfx/iscan: Fix build with >=imagemagick-7
    
    Non-maintainer commit.
    
    See also: https://github.com/utsushi/utsushi/issues/43
    
    Closes: https://bugs.gentoo.org/704706
    Package-Manager: Portage-2.3.89, Repoman-2.3.20
    Signed-off-by: Andreas Sturmlechner <[e-mail]>

 .../iscan/files/iscan-3.61.0-imagemagick-7.patch   | 54 ++++++++++++++++++++++
 media-gfx/iscan/iscan-3.61.0.ebuild                | 11 ++---
 2 files changed, 59 insertions(+), 6 deletions(-)