Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 661430 - sci-libs/mpir-3.0.0: missing dependency to yasm
Summary: sci-libs/mpir-3.0.0: missing dependency to yasm
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Science Related Packages
URL: https://github.com/wbhart/mpir/issues...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-17 19:03 UTC by Rolf Eike Beer
Modified: 2019-11-28 17:20 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (sci-libs_mpir-3.0.0_use_ZlPGX,6.59 KB, text/plain)
2018-07-17 19:03 UTC, Rolf Eike Beer
Details
config.log (config.log,33.48 KB, text/plain)
2018-07-17 19:04 UTC, Rolf Eike Beer
Details
patch to remove yasm check on non-x86 platforms (fix-mpir-compilation-on-non-x86.patch,1.10 KB, patch)
2019-04-28 16:02 UTC, Ross Charles C.
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rolf Eike Beer archtester 2018-07-17 19:03:18 UTC
Created attachment 539906 [details]
build.log

Looking for a system-wide yasm...
which: no yasm in (/usr/lib/portage/python3.5/ebuild-helpers/xattr:/usr/lib/portage/python3.5/ebuild-helpers:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/sparc-unknown-linux-gnu/gcc-bin/7.3.0:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/bin)
configure: error: no system-wide yasm found

Portage 2.3.40 (python 3.5.5-final-0, default/linux/sparc/13.0, gcc-7.3.0, glibc-2.26-r7, 4.16.8 sparc64)
=================================================================
System uname: Linux-4.16.8-sparc64-sun4v-with-gentoo-2.4.1
KiB Mem:    33133328 total,  10922144 free
KiB Swap:          0 total,         0 free
Timestamp of repository gentoo: Tue, 17 Jul 2018 00:04:36 +0000
Head commit of repository gentoo: 9210574bf22412a24f3b43f25db792007f90bdd4
sh bash 4.4_p12
ld GNU ld (Gentoo 2.30 p2) 2.30.0
app-shells/bash:          4.4_p12::gentoo
dev-lang/perl:            5.24.3-r1::gentoo
dev-lang/python:          2.7.14-r1::gentoo, 3.5.5::gentoo, 3.6.5::gentoo
dev-util/cmake:           3.9.6::gentoo
dev-util/pkgconfig:       0.29.2::gentoo
sys-apps/baselayout:      2.4.1-r2::gentoo
sys-apps/openrc:          0.34.11::gentoo
sys-apps/sandbox:         2.13::gentoo
sys-devel/autoconf:       2.13::gentoo, 2.69-r4::gentoo
sys-devel/automake:       1.11.6-r3::gentoo, 1.15.1-r2::gentoo
sys-devel/binutils:       2.30-r2::gentoo
sys-devel/gcc:            7.3.0-r3::gentoo
sys-devel/gcc-config:     1.8-r1::gentoo
sys-devel/libtool:        2.4.6-r3::gentoo
sys-devel/make:           4.2.1::gentoo
sys-kernel/linux-headers: 4.13::gentoo (virtual/os-headers)
sys-libs/glibc:           2.26-r7::gentoo
Repositories:

gentoo
    location: /usr/portage
    sync-type: rsync
    sync-uri: rsync://rsync.gentoo.org/gentoo-portage
    priority: -1000
    sync-rsync-extra-opts: 
    sync-rsync-verify-jobs: 1
    sync-rsync-verify-metamanifest: yes
    sync-rsync-verify-max-age: 24

ACCEPT_KEYWORDS="sparc"
ACCEPT_LICENSE="* -@EULA"
CBUILD="sparc-unknown-linux-gnu"
CFLAGS="-O2 -mcpu=niagara2 -pipe"
CHOST="sparc-unknown-linux-gnu"
CONFIG_PROTECT="/etc /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/php/apache2-php7.0/ext-active/ /etc/php/apache2-php7.1/ext-active/ /etc/php/cgi-php7.0/ext-active/ /etc/php/cgi-php7.1/ext-active/ /etc/php/cli-php7.0/ext-active/ /etc/php/cli-php7.1/ext-active/ /etc/revdep-rebuild /etc/sandbox.d /etc/terminfo /etc/texmf/language.dat.d /etc/texmf/language.def.d /etc/texmf/updmap.d /etc/texmf/web2c"
CXXFLAGS="-O2 -mcpu=niagara2 -pipe"
DISTDIR="/usr/portage/distfiles"
ENV_UNSET="DBUS_SESSION_BUS_ADDRESS DISPLAY 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=""
FEATURES="assume-digests binpkg-logs config-protect-if-modified distlocks ebuild-locks fixlafiles merge-sync news parallel-fetch preserve-libs protect-owned sandbox sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch userpriv usersandbox usersync xattr"
FFLAGS=""
GENTOO_MIRRORS="ftp://ftp.wh2.tu-dresden.de/pub/mirrors/gentoo ftp://ftp.join.uni-muenster.de/pub/linux/distributions/gentoo"
LANG="en_US.UTF-8"
LDFLAGS="-Wl,-O1 -Wl,--as-needed"
MAKEOPTS="-j20"
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="acl berkdb big-endian bzip2 cli crypt cxx dri fortran gdbm iconv ipv6 libtirpc modules ncurses nls nptl openmp pam pcre readline sparc ssl tcpd unicode xattr zlib" 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="karbon plan sheets stage words" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock isync itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf skytraq superstar2 timing tsip tripmate tnt ublox ubx" INPUT_DEVICES="libinput keyboard mouse" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LIBREOFFICE_EXTENSIONS="presenter-console presenter-minimizer" OFFICE_IMPLEMENTATION="libreoffice" PHP_TARGETS="php5-6 php7-0" POSTGRES_TARGETS="postgres9_5 postgres10" PYTHON_SINGLE_TARGET="python3_5" PYTHON_TARGETS="python2_7 python3_5" RUBY_TARGETS="ruby23" USERLAND="GNU" VIDEO_CARDS="fbdev glint mga r128 radeon dummy v4l" 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:  CC, CPPFLAGS, CTARGET, CXX, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, LINGUAS, PORTAGE_BINHOST, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS
Comment 1 Rolf Eike Beer archtester 2018-07-17 19:04:31 UTC
Created attachment 539908 [details]
config.log
Comment 2 Rolf Eike Beer archtester 2018-09-28 18:14:47 UTC
Also affects 2.7.2. The problem is that there is an explicit dependency to yasm on x86 and amd64, but --system-yasm is passed to configure on all platforms.
Comment 3 François Bissey 2018-09-28 20:27:13 UTC
One could argue that upstream configure script should be able to cope with that but it certainly something we can fix on our side.
Comment 4 Andreas Sturmlechner gentoo-dev 2019-03-03 16:20:29 UTC
yasm dependency should be conditional to cpudetection (aka --enable-fat) then?
Comment 5 Ross Charles C. 2019-04-28 16:02:05 UTC
Created attachment 574536 [details, diff]
patch to remove yasm check on non-x86 platforms

I wrote a patch to fix configure.ac to remove mandatory dependency on yasm if it is not compiled on x86 platform. This seems to compile without problem on rpi3 but I'm not sure about other platforms.
Comment 6 Rolf Eike Beer archtester 2019-04-29 17:55:12 UTC
It builds for me on sparc with the patch, too.

However I personally dislike it: it duplicates things between the ebuild and the configure script. IMHO one should pass a flag on those platforms that expect yasm to be there and the configure script should simply ignore yasm scanning if the flag is not passed. Or something like that. But again, that's personal taste about maintainability, more important would be to get it working at all.
Comment 7 Ross Charles C. 2019-05-01 10:05:26 UTC
I looked into the whole build script, the whole thing is written in a very nasty way, and there is no such options to "turn off" the usage of yasm. For platforms that don't use yasm, it simply wouldn't execute any yasm command. The mandatory dependency for external yasm has been around since the bundled one was removed from the source tree, so I reported the issue to upstream: https://github.com/wbhart/mpir/issues/270
Comment 8 Rolf Eike Beer archtester 2019-05-01 12:22:14 UTC
A short term fix for Gentoo could be to pass /bin/true as yasm binary on those platforms that don't require it.
Comment 9 Mike Gilbert gentoo-dev 2019-11-11 21:41:59 UTC
(In reply to Rolf Eike Beer from comment #8)
> A short term fix for Gentoo could be to pass /bin/true as yasm binary on
> those platforms that don't require it.

I think I would rather pass /bin/false so that the build system dies if it gets executed accidentally.

Would you mind testing that?
Comment 10 Larry the Git Cow gentoo-dev 2019-11-11 21:48:47 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=298d3c36aed5cdb79508f1527944cf2e64bf8686

commit 298d3c36aed5cdb79508f1527944cf2e64bf8686
Author:     Mike Gilbert <floppym@gentoo.org>
AuthorDate: 2019-11-11 21:48:10 +0000
Commit:     Mike Gilbert <floppym@gentoo.org>
CommitDate: 2019-11-11 21:48:42 +0000

    sci-libs/mpir: pass --with-yasm=/bin/false on non-x86 archs
    
    Closes: https://bugs.gentoo.org/661430
    Package-Manager: Portage-2.3.78_p18, Repoman-2.3.17_p114
    Signed-off-by: Mike Gilbert <floppym@gentoo.org>

 sci-libs/mpir/mpir-3.0.0.ebuild | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)