Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 486470 - x11-apps/xedit : does not respect AR
Summary: x11-apps/xedit : does not respect AR
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal QA (vote)
Assignee: Matt Turner
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: tc-directly
  Show dependency tree
 
Reported: 2013-09-29 10:46 UTC by Agostino Sarubbo
Modified: 2015-02-21 21:53 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 Agostino Sarubbo gentoo-dev 2013-09-29 10:46:37 UTC
You receive this bug because this package does not respect my system's AR (x86_64-pc-linux-gnu-ar) and calls directly /usr/bin/ar
The possible solutions to fix this issue are:
1)Fix the buildsystem, if you can;
2)inherit toolchain-funcs and use tc-export AR
3)inherit toolchain-funcs and use emake AR="$(tc-getAR)"


From the build log:

ar cru liblisp.a liblisp_a-bytecode.o liblisp_a-core.o liblisp_a-debugger.o liblisp_a-format.o liblisp_a-hash.o liblisp_a-helper.o liblisp_a-io.o liblisp_a-lisp.o 
liblisp_a-math.o liblisp_a-package.o liblisp_a-pathname.o liblisp_a-read.o liblisp_a-regex.o liblisp_a-require.o liblisp_a-stream.o liblisp_a-string.o liblisp_a-struct.o 
liblisp_a-time.o liblisp_a-write.o liblisp_a-xedit.o
Comment 1 Ted Tanberry 2014-03-30 12:37:29 UTC
--- xedit-1.2.1.ebuild
+++ xedit-1.2.1.ebuild
@@ -4,7 +4,7 @@

 EAPI=5

-inherit xorg-2
+inherit xorg-2 eutils

 DESCRIPTION="simple text editor for X"
 KEYWORDS="amd64 hppa ~mips ppc ppc64 sparc x86"
@@ -15,3 +15,9 @@
 	x11-libs/libXmu
 	x11-libs/libX11"
 DEPEND="${RDEPEND}"
+
+src_prepare() {
+	# Fixes hard-coded AR #486470
+	sed -i '/AM_PROG_CC_C_O/aAM_PROG_AR' configure.ac
+	eautoreconf
+}
Comment 2 Ted Tanberry 2014-03-30 12:38:22 UTC
[...]
x86_64-pc-linux-gnu-ar cru liblisp.a liblisp_a-bytecode.o liblisp_a-core.o liblisp_a-debugger.o liblisp_a-format.o liblisp_a-hash.o liblisp_a-helper.o liblisp_a-io.o liblisp_a-lisp.o liblisp_a-math.o liblisp_a-package.o liblisp_a-pathname.o liblisp_a-read.o liblisp_a-regex.o liblisp_a-require.o liblisp_a-stream.o liblisp_a-string.o liblisp_a-struct.o liblisp_a-time.o liblisp_a-write.o liblisp_a-xedit.o
[...]
Comment 3 Matt Turner gentoo-dev 2014-03-30 20:27:05 UTC
(In reply to Ted Tanberry from comment #1)
> --- xedit-1.2.1.ebuild
> +++ xedit-1.2.1.ebuild
> @@ -4,7 +4,7 @@
> 
>  EAPI=5
> 
> -inherit xorg-2
> +inherit xorg-2 eutils
> 
>  DESCRIPTION="simple text editor for X"
>  KEYWORDS="amd64 hppa ~mips ppc ppc64 sparc x86"
> @@ -15,3 +15,9 @@
>  	x11-libs/libXmu
>  	x11-libs/libX11"
>  DEPEND="${RDEPEND}"
> +
> +src_prepare() {
> +	# Fixes hard-coded AR #486470
> +	sed -i '/AM_PROG_CC_C_O/aAM_PROG_AR' configure.ac
> +	eautoreconf
> +}

Instead of us hacking it, please just send a git-formatted patch upstream to xorg-devel@lists.x.org, and Cc me and Gaetan Nadon <memsize@videotron.ca>.
Comment 5 Matt Turner gentoo-dev 2015-02-21 21:53:15 UTC
I've cut a release of xedit with your patch (version 1.2.2) and pushed it into the tree.

Thanks!