Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 61087 - Please update media-plugins/coriander ebuild - KEYWORDS and src_install()
Summary: Please update media-plugins/coriander ebuild - KEYWORDS and src_install()
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High minor (vote)
Assignee: Pieter Van den Abeele (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-20 16:54 UTC by Daniel Westermann-Clark
Modified: 2004-10-22 06:56 UTC (History)
0 users

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


Attachments
diff to current ebuild (coriander-1.0.0_pre3.ebuild.diff,555 bytes, patch)
2004-09-18 16:07 UTC, Benoit C
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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) gentoo-dev 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) gentoo-dev 2004-10-22 06:56:20 UTC
fixed and version bumped.  Thnxs