519286 – app-i18n/fbterm-1.7 - `myconfargs' should be `myeconfargs'

Bug 519286 - app-i18n/fbterm-1.7 - `myconfargs' should be `myeconfargs'

Summary: app-i18n/fbterm-1.7 - `myconfargs' should be `myeconfargs'

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	CJK Team

URL:
Whiteboard:
Keywords:	PATCH

Depends on:
Blocks:	myconfargs-typo
	Show dependency tree

Reported:	2014-08-07 05:42 UTC by Casper Ti. Vector
Modified:	2014-08-07 16:09 UTC (History)
CC List:	1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Casper Ti. Vector 2014-08-07 05:42:23 UTC

(Originally reported in bug #518106; now reported as separate bugs after consulting devs on #gentoo at freenode.)

As the title says.  Eclasses only provided the `myeconfargs' interface, not `myconfargs'.  Affected ebuild(s) do not seem to use `myconfargs' side by side with `myeconfargs' on purpose, so this is probably a typo.

Following is the patch:

--- /usr/portage/app-i18n/fbterm/fbterm-1.7.ebuild	2012-07-07 16:01:04.000000000 +0800
+++ -	2014-08-07 13:40:04.899806228 +0800
@@ -28,7 +28,7 @@
 DOCS=( AUTHORS NEWS README )
 
 src_configure() {
-	local myconfargs=(
+	local myeconfargs=(
 		$(use_enable gpm)
 		$(use_enable video_cards_vesa vesa)
 	)

Comment 1 Yixun Lan archtester

2014-08-07 14:28:41 UTC

+*fbterm-1.7-r1 (07 Aug 2014)
+
+  07 Aug 2014; Yixun Lan <dlan@gentoo.org> -fbterm-1.7.ebuild,
+  +fbterm-1.7-r1.ebuild:
+  fix typo, bug 519486, thanks Casper Ti. Vector

Comment 2 Casper Ti. Vector 2014-08-07 16:09:47 UTC

(In reply to Yixun Lan from comment #1)
Should be 519286 instead of 519486 (if that is of some importance) :D