Bug 61087 – Please update media-plugins/coriander ebuild - KEYWORDS and src_install()

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Daniel Westermann-Clark 2004-08-20 16:54:38 UTC]

I'm working on getting my iSight working on an x86 system (pentium4), based on information at:

http://www.ywesee.com/index.cgi?Isight

There is currently an ebuild for Coriander (media-plugins/coriander), which contains only ppc in KEYWORDS.  I copied the ebuild to my overlay and added ~x86 to the KEYWORDS.  It compiled fine, but died when trying to install pixmaps with a sandbox violation:

[...]
make[2]: Nothing to be done for `install-exec-am'.
if test -d ./pixmaps; then \
  /bin/sh ./mkinstalldirs /usr/share/pixmaps/coriander; \
  for pixmap in ./pixmaps/*; do \
    if test -f $pixmap; then \
      /bin/install -c -m 644 $pixmap /usr/share/pixmaps/coriander; \
    fi \
  done \
fi
mkdir /usr/share/pixmaps/coriander
ACCESS DENIED  mkdir:     /usr/share/pixmaps/coriander
mkdir: cannot create directory `/usr/share/pixmaps/coriander': Permission denied
ACCESS DENIED  open_wr:   /usr/share/pixmaps/coriander
/bin/install: cannot create regular file `/usr/share/pixmaps/coriander': Permission denied
ACCESS DENIED  open_wr:   /usr/share/pixmaps/coriander
/bin/install: cannot create regular file `/usr/share/pixmaps/coriander': Permission denied
make[2]: *** [install-data-local] Error 1
make[2]: Leaving directory `/var/tmp/portage/coriander-1.0.0_pre3/work/coriander-1.0.0-pre3'
make[1]: *** [install-am] Error 2
make[1]: Leaving directory `/var/tmp/portage/coriander-1.0.0_pre3/work/coriander-1.0.0-pre3'
make: *** [install-recursive] Error 1

!!! ERROR: media-plugins/coriander-1.0.0_pre3 failed.
!!! Function einstall, Line 385, Exitcode 2
!!! einstall failed

--------------------------- ACCESS VIOLATION SUMMARY ---------------------------
LOG FILE = "/tmp/sandbox-media-plugins_-_coriander-1.0.0_pre3-10080.log"

mkdir:     /usr/share/pixmaps/coriander
open_wr:   /usr/share/pixmaps/coriander
open_wr:   /usr/share/pixmaps/coriander
--------------------------------------------------------------------------------

Looking at the Makefile, this seems to be a problem of DESTDIR not being set correctly.  According to the ebuild howto, "make DESTDIR=${D} install" is preferred over "einstall".  Changing the appropriate line in src_install() fixed the sandbox violation and correctly installed Coriander for me.

Once installed, Coriander correctly detected my iSight and was able to display the video.  Would it be possible to update the KEYWORDS and make the src_install() fix?

Reproducible: Always
Steps to Reproduce:




Portage 2.0.50-r9 (default-x86-2004.2, gcc-3.3.3, glibc-2.3.3.20040420-r1, 2.6.8.1)
=================================================================
System uname: 2.6.8.1 i686 Intel(R) Xeon(TM) CPU 2.40GHz
Gentoo Base System version 1.4.16
distcc 2.13 i686-pc-linux-gnu (protocols 1 and 2) (default port 3632) [enabled]
Autoconf: sys-devel/autoconf-2.59-r4
Automake: sys-devel/automake-1.8.3
ACCEPT_KEYWORDS="x86"
AUTOCLEAN="yes"
CFLAGS="-mcpu=pentium4 -O2 -fomit-frame-pointer -pipe"
CHOST="i686-pc-linux-gnu"
COMPILER="gcc3"
CONFIG_PROTECT="/etc /usr/X11R6/lib/X11/xkb /usr/kde/2/share/config
/usr/kde/3.2/share/config /usr/kde/3/share/config /usr/lib/mozilla/defaults/pref
/usr/share/config /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-mcpu=pentium4 -O2 -fomit-frame-pointer -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoaddcvs ccache distcc sandbox userpriv"
GENTOO_MIRRORS="http://open-systems.ufl.edu/mirrors/gentoo
http://gentoo.oregonstate.edu http://www.ibiblio.org/pub/Linux/distributions/gentoo"
MAKEOPTS="-j5"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.namerica.gentoo.org/gentoo-portage"
USE="X aalib acpi alsa apm avi berkdb cdr crypt cups divx4linux dvd encode
foomaticdb gdbm gif gnome gpm gtk gtk2 gtkhtml imlib java jpeg kde libg++ libwww
mad maildir mikmod mmx motif mozilla mpeg ncurses nls oggvorbis opengl oss pam
pdflib perl png ppds python qt quicktime readline sdl slang spell ssl svga tcpd
tiff truetype x86 xml xml2 xmms xv xvid zlib"

[Comment 1 Seemant Kulleen (RETIRED) 2004-08-20 19:57:16 UTC]

pieter, might be better to switch from einstall to make DESTDIR=${D} install instead

[Comment 2 Benoit C 2004-09-18 16:07:17 UTC]

Created attachment 39878 [details, diff]
diff to current ebuild

A diff to currentr ebuild that allow to build coriander on x86 platform.

[Comment 3 Pieter Van den Abeele (RETIRED) 2004-10-22 06:56:20 UTC]

fixed and version bumped.  Thnxs