Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 367911 - sci-geosciences/googleearth-6.0.2.2074 does not build without tweak to ebuild
Summary: sci-geosciences/googleearth-6.0.2.2074 does not build without tweak to ebuild
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Prefix Support (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Prefix
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-18 12:12 UTC by Asfand Yar Qazi
Modified: 2013-02-10 15:27 UTC (History)
0 users

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


Attachments
ebuild patch (googleearth-6.0.2.2074.ebuild.patch,397 bytes, patch)
2011-05-18 12:12 UTC, Asfand Yar Qazi
Details | Diff
ebuild patch (googleearth-6.2.2.6613.ebuild.prefix.patch,457 bytes, patch)
2012-07-02 15:15 UTC, Asfand Yar Qazi
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Asfand Yar Qazi 2011-05-18 12:12:30 UTC
Created attachment 273823 [details, diff]
ebuild patch

sci-geosciences/googleearth-6.0.2.2074 does not work on Gentoo Prefix without tweak to ebuild

${D} needs to become ${ED}

Patch attached

I still get some warnings when chown-ing about operation not permitted, but really who cares about that if it works?

emerge --info:
Portage 2.2.01.18252-prefix (prefix/linux/x86, gcc-4.5.2, unavailable, 2.6.32-30-generic-pae i686)
=================================================================
System uname: Linux-2.6.32-30-generic-pae-i686-Intel-R-_Core-TM-_i5_CPU_650_@_3.20GHz-with-debian-squeeze-sid
Timestamp of tree: Mon, 16 May 2011 10:10:19 +0000
ccache version 3.1.4 [disabled]
app-shells/bash:     4.2_p8
dev-java/java-config: 2.1.11-r2
dev-lang/python:     2.7.1-r1
dev-util/ccache:     3.1.4
dev-util/cmake:      2.8.4-r1
sys-devel/autoconf:  2.68
sys-devel/automake:  1.11.1
sys-devel/binutils:  2.21.51.0.6
sys-devel/gcc:       4.2.4-r01.2, 4.5.2-r00.1
sys-devel/gcc-config: 1.4.1-r00.2
sys-devel/libtool:   2.4-r1
sys-devel/make:      3.82
sys-kernel/linux-headers: 2.6.38
virtual/os-headers:  0
Repositories: gentoo_prefix my_local_overlay
Installed sets: 
ACCEPT_KEYWORDS="~x86 ~x86-linux"
ACCEPT_LICENSE="*"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-march=i686 -Os -pipe"
CHOST="i686-pc-linux-gnu"
CONFIG_PROTECT="/etc"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/env.d /etc/env.d/java/ /etc/fonts/fonts.conf /etc/gconf /etc/portage /etc/revdep-rebuild /etc/terminfo"
CXXFLAGS="-march=i686 -Os -pipe"
DISTDIR="/var/tmp/gentoo/usr/portage/distfiles"
FEATURES="assume-digests binpkg-logs collision-protect distlocks fixlafiles fixpackages news parallel-fetch preserve-libs protect-owned sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch"
FFLAGS=""
GENTOO_MIRRORS="http://distfiles.gentoo.org"
LANG="en_GB.utf8"
LDFLAGS="-Wl,-O1"
LINGUAS="en_GB en_US en C"
MAKEOPTS="-j2"
PKGDIR="/var/tmp/gentoo/usr/portage/packages"
PORTAGE_CONFIGROOT="/var/tmp/gentoo/"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress --force --whole-file --delete --stats --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages"
PORTAGE_TMPDIR="/var/tmp/gentoo/var/tmp"
PORTDIR="/var/tmp/gentoo/usr/portage"
PORTDIR_OVERLAY="/var/tmp/gentoo/usr/local/portage"
SYNC="rsync://localhost:8873/gentoo-portage-prefix"
USE="X alsa bash-completion berkdb bzip2 cli cracklib crypt cxx dbus dri fortran gdbm gtk iconv ipv6 libnotify mmx modules mudflap ncurses nls nptl nptlonly opengl openmp pcre perl pppd prefix python readline session sse sse2 ssl startup-notification sysfs tcpd threads truetype unicode x86 xinerama xorg zlib" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem bt87x ca0106 cmipci emu10k1 emu10k1x ens1370 ens1371 es1938 es1968 fm801 hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem ymfpci" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug file hooks iec958 ioplug ladspa lfloat linear meter mmap_emul mulaw multi null plug rate route share shm softvol" APACHE2_MODULES="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="braindump flow karbon kexi kpresenter krita tables words" CAMERAS="ptp2" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf superstar2 timing tsip tripmate tnt ubx" INPUT_DEVICES="keyboard mouse evdev" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LINGUAS="en_GB en_US en C" PHP_TARGETS="php5-3" RUBY_TARGETS="ruby18" USERLAND="GNU" 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, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS
Comment 1 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2011-05-18 13:18:40 UTC
I think you want to add your KEYWORD too.

(In reply to comment #0)
> I still get some warnings when chown-ing about operation not permitted, but
> really who cares about that if it works?

This can be avoided by "use prefix || chown <blah>"
Comment 2 Vlastimil Babka (Caster) (RETIRED) gentoo-dev 2011-05-24 16:23:20 UTC
There's no direct chown, but this:

        cp -pPR "${WORKDIR}"/data/* "${D}"/opt/${PN} || die
        fowners -R root:root /opt/${PN}
        fperms -R a-x,a+X /opt/googleearth/resources

What is the correct prefix way then?
Comment 3 Asfand Yar Qazi 2011-05-24 16:43:26 UTC
(In reply to comment #2)
> There's no direct chown, but this:
> 
>         cp -pPR "${WORKDIR}"/data/* "${D}"/opt/${PN} || die
>         fowners -R root:root /opt/${PN}
>         fperms -R a-x,a+X /opt/googleearth/resources
> 
> What is the correct prefix way then?

The correct prefix way is to leave the ebuild untouched, and let the chown operation not permitted errors to be output as they currently are.  It works fine that way, and allows for less customization of ebuilds and hence easier maintainability.
Comment 4 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2011-05-24 17:05:35 UTC
(In reply to comment #3)
> The correct prefix way is to leave the ebuild untouched, 

Umm, no.

(In reply to comment #2)
> What is the correct prefix way then?

Same concept as comment #1: "use prefix || fowners ..." - There are already examples of this in gentoo-x86, you can use 'qgrep -H "use prefix || fowners"' to identify the examples.
Comment 5 Fabian Groffen gentoo-dev 2011-05-29 18:54:27 UTC
FYI: sys-apps/portage-2.2.01.18706 should automatically ignore fowners in Prefix
Comment 6 Vlastimil Babka (Caster) (RETIRED) gentoo-dev 2012-07-02 12:31:06 UTC
(In reply to comment #5)
> FYI: sys-apps/portage-2.2.01.18706 should automatically ignore fowners in
> Prefix

Does it mean the ebuild needs no changes now? If yes, please close.
Comment 7 Asfand Yar Qazi 2012-07-02 15:15:32 UTC
Created attachment 316981 [details, diff]
ebuild patch

Although the fperms issue may have gone away, the original issue still persists: a ${D} needs to be changed to ${ED}.  Updated patch for googleearth-6.2.2 attached.  The tool to do the conversion obviously doesn't do this automatically because it is programmed to not touch the last ${D}, which is usually the correct thing to do, but not in this case.
Comment 8 Julian Ospald 2013-02-10 15:27:53 UTC
+  10 Feb 2013; Julian Ospald <hasufell@gentoo.org>
+  googleearth-7.0.2.8415-r1.ebuild:
+  fix prefix installation