Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 452170 - gnome-base/gconf installs /etc/env.d/50gconf with unexpanded ${EPREFIX}
Summary: gnome-base/gconf installs /etc/env.d/50gconf with unexpanded ${EPREFIX}
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-15 03:43 UTC by Arfrever Frehtes Taifersar Arahesis
Modified: 2013-01-17 20:39 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Arfrever Frehtes Taifersar Arahesis 2013-01-15 03:43:21 UTC
gnome-base/gconf installs /etc/env.d/50gconf with unexpanded ${EPREFIX}:

# cat /etc/env.d/50gconf
CONFIG_PROTECT_MASK="${EPREFIX}/etc/gconf"
GSETTINGS_BACKEND="gconf"
# env-update
>>> Regenerating /etc/ld.so.cache...
# . /etc/profile
# grep CONFIG_PROTECT_MASK /etc/profile.env
export CONFIG_PROTECT_MASK='/etc/gentoo-release /etc/sandbox.d /etc/fonts/fonts.conf ${EPREFIX}/etc/gconf /etc/terminfo /etc/ca-certificates.conf /etc/revdep-rebuild /etc/splash'
# echo "${CONFIG_PROTECT_MASK}"
/etc/gentoo-release /etc/sandbox.d /etc/fonts/fonts.conf ${EPREFIX}/etc/gconf /etc/terminfo /etc/ca-certificates.conf /etc/revdep-rebuild /etc/splash
Comment 1 Pacho Ramos gentoo-dev 2013-01-15 21:47:19 UTC
Is prefix overlay version also affected by this?
Comment 2 Arfrever Frehtes Taifersar Arahesis 2013-01-15 22:24:39 UTC
(In reply to comment #1)
> Is prefix overlay version also affected by this?

Their ebuild does not use ${EPREFIX}:
http://overlays.gentoo.org/proj/alt/browser/trunk/prefix-overlay/gnome-base/gconf/gconf-3.2.5.ebuild?rev=60859#L72

Quoting needs to be fixed in gnome-base/gconf ebuilds in gentoo-x86:

- echo 'CONFIG_PROTECT_MASK="${EPREFIX}/etc/gconf"' > 50gconf
+ echo "CONFIG_PROTECT_MASK=\"${EPREFIX}/etc/gconf\"" > 50gconf

${EPREFIX} was introduced in ebuilds by Justin Lecher in 2012-05-22, but this bug was not noticed until recent deletion of "doc" USE flag triggered reinstallation.
Comment 3 Pacho Ramos gentoo-dev 2013-01-17 15:29:18 UTC
+  17 Jan 2013; Pacho Ramos <pacho@gentoo.org> gconf-2.32.4.ebuild,
+  gconf-3.2.5.ebuild:
+  Stop setting unexpanded , bug #452170 by Arfrever Frehtes Taifersar Arahesis
+

Should we stabilize a newer revision to get this installed in most systems? :/
Comment 4 Arfrever Frehtes Taifersar Arahesis 2013-01-17 20:24:45 UTC
(In reply to comment #3)

Your commit is wrong:
- echo 'CONFIG_PROTECT_MASK="${EPREFIX}/etc/gconf"' > 50gconf
+ echo 'CONFIG_PROTECT_MASK='${EPREFIX}/etc/gconf'' > 50gconf

If ${EPREFIX} contains >=2 subsequent spaces, then incorrect value will be stored in 50gconf. Please use what I suggested in comment #2.
Comment 5 Fabian Groffen gentoo-dev 2013-01-17 20:27:42 UTC
(In reply to comment #4)
> (In reply to comment #3)
> 
> Your commit is wrong:
> - echo 'CONFIG_PROTECT_MASK="${EPREFIX}/etc/gconf"' > 50gconf
> + echo 'CONFIG_PROTECT_MASK='${EPREFIX}/etc/gconf'' > 50gconf
> 
> If ${EPREFIX} contains >=2 subsequent spaces, then incorrect value will be
> stored in 50gconf. Please use what I suggested in comment #2.

or
echo 'CONFIG_PROTECT_MASK="'"${EPREFIX}/etc/gconf"'"' > 50gconf
Comment 6 Pacho Ramos gentoo-dev 2013-01-17 20:39:25 UTC
(In reply to comment #4)
> (In reply to comment #3)
> 
> Your commit is wrong:
> - echo 'CONFIG_PROTECT_MASK="${EPREFIX}/etc/gconf"' > 50gconf
> + echo 'CONFIG_PROTECT_MASK='${EPREFIX}/etc/gconf'' > 50gconf
> 
> If ${EPREFIX} contains >=2 subsequent spaces, then incorrect value will be
> stored in 50gconf. Please use what I suggested in comment #2.

I tested it before committing and it worked fine:
$ cat /etc/env.d/50gconf 
CONFIG_PROTECT_MASK=/etc/gconf
GSETTINGS_BACKEND="gconf"

But didn't think in your case:
+  17 Jan 2013; Pacho Ramos <pacho@gentoo.org> gconf-2.32.4.ebuild,
+  gconf-3.2.5.ebuild:
+  Previous form still fails when EPREFIX contains >= spaced (#452170#c4 by
+  Arfrever Frehtes Taifersar Arahesis)
+