474764 – media-libs/vamp-plugin-sdk : does not respect AR

Bug 474764 - media-libs/vamp-plugin-sdk : does not respect AR

Summary: media-libs/vamp-plugin-sdk : does not respect AR

Status:	RESOLVED FIXED

Alias:	None

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

Importance:	Normal QA (vote)
Assignee:	Professional Audio Applications Maintainers

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	tc-directly
	Show dependency tree

Reported:	2013-06-25 18:32 UTC by Agostino Sarubbo
Modified:	2013-08-12 16:19 UTC (History)
CC List:	1 user (show)

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

Attachments
vamp-plugin-sdk-2.5-ar.patch (vamp-plugin-sdk-2.5-ar.patch,702 bytes, patch) 2013-07-25 19:08 UTC, Michael Mair-Keimberger (iamnr3)	Details \| Diff
patch against latest ebuild (ebuild.patch,670 bytes, patch) 2013-07-25 19:10 UTC, Michael Mair-Keimberger (iamnr3)	Details \| Diff
View All 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 Agostino Sarubbo gentoo-dev

2013-06-25 18:32:32 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 r libvamp-sdk.a src/vamp-sdk/PluginAdapter.o src/vamp-sdk/RealTime.o src/vamp-sdk/FFT.o src/vamp-sdk/acsymbols.o

Comment 1 Michael Mair-Keimberger (iamnr3) 2013-07-25 19:08:36 UTC

Created attachment 354198 [details, diff]
vamp-plugin-sdk-2.5-ar.patch

This patch fixes the buildsystem so that respect user ar/ranlib, also fixes bug 474766. please review.

Comment 2 Michael Mair-Keimberger (iamnr3) 2013-07-25 19:10:04 UTC

Created attachment 354200 [details, diff]
patch against latest ebuild

Comment 3 Alexis Ballier gentoo-dev

2013-08-12 16:19:01 UTC

applied a simpler patch