Bug 160328 – x11-wm/wmii fails due to sed expression error in ebuild

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Romain Iehl 2007-01-05 14:20:56 UTC]

in x11-wm/wmii/wmii-3.5.1.ebuild is the following sed expression :

	sed -i \
		-e "/^PREFIX/s/=.*/= \/usr/" \
		-e "/^CONFPREFIX/s/=.*/= \/etc/" \
		-e "/^X11INC/s/=.*/= \/usr\/include/" \
		-e "/^X11LIB/s/=.*/= \/usr\/lib/" \
		-e "/^CFLAGS/s/= -Os/+=/" \
		-e "/^LDFLAGS/s/=/+=/" \
		-e "/^CC/s/=.*/= $(tc-getCC)/" \
		config.mk || die "sed failed"

On my system $(tc-getCC) is evaluated to /usr/share/cache/bin/gcc

The '/' characters make the sed expression fail, a dirty workaround is to replace :
-e "/^CC/s/=.*/= $(tc-getCC)/" \
by :
-e "/^CC/s\\=.*\\= $(tc-getCC)\\" \

[Comment 1 Jakub Moc (RETIRED) 2007-01-05 14:22:42 UTC]

Works just fine here...

[Comment 2 Ciaran McCreesh 2007-01-16 14:21:14 UTC]

Jakub, did you even read the bug? There's clearly broken code there and it's obvious under what conditions it will fail. Closing it off as WORKSFORME is silly -- we know it works for you, and anyone else who happens to not have a / in certain variables, but it doesn't work for those who do.

[Comment 3 Stephen Bennett (RETIRED) 2007-01-16 14:22:48 UTC]

That it works for one person does not mean that it is not a bug.

[Comment 4 David Shakaryan (RETIRED) 2007-01-17 00:24:07 UTC]

Should be fixed. Thank you for the bug report. :)