diff mbox

package/tyrian: fixes compilation with static libs.

Message ID 1491222343-29223-1-git-send-email-julien.boibessot@free.fr
State Accepted
Headers show

Commit Message

Julien Boibessot April 3, 2017, 12:25 p.m. UTC 
From: Julien BOIBESSOT <julien.boibessot@armadeus.com>

Has been tested with: "./support/scripts/test-pkg -c tyrian.cfg -p opentyrian"
Fixes: http://autobuild.buildroot.net/results/0e2345db82b33f591958fc0f72ad914adafe0522
and some similar previous build failure.

Thanks Thomas for the tip ;-).

Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>
---
 package/opentyrian/opentyrian.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni April 3, 2017, 12:33 p.m. UTC | #1 
Hello,

On Mon,  3 Apr 2017 14:25:43 +0200, julien.boibessot@free.fr wrote:
> From: Julien BOIBESSOT <julien.boibessot@armadeus.com>
> 
> Has been tested with: "./support/scripts/test-pkg -c tyrian.cfg -p opentyrian"
> Fixes: http://autobuild.buildroot.net/results/0e2345db82b33f591958fc0f72ad914adafe0522
> and some similar previous build failure.
> 
> Thanks Thomas for the tip ;-).
> 
> Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>
> ---
>  package/opentyrian/opentyrian.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks.

Thomas
Peter Korsgaard April 4, 2017, 9:15 p.m. UTC | #2 
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > Hello,
 > On Mon,  3 Apr 2017 14:25:43 +0200, julien.boibessot@free.fr wrote:
 >> From: Julien BOIBESSOT <julien.boibessot@armadeus.com>
 >> 
 >> Has been tested with: "./support/scripts/test-pkg -c tyrian.cfg -p opentyrian"
 >> Fixes: http://autobuild.buildroot.net/results/0e2345db82b33f591958fc0f72ad914adafe0522
 >> and some similar previous build failure.
 >> 
 >> Thanks Thomas for the tip ;-).
 >> 
 >> Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>
 >> ---
 >> package/opentyrian/opentyrian.mk | 2 +-

I know it is too late now, but subject should have said
package/opentyrian, not package/tyrian.
Peter Korsgaard April 4, 2017, 9:45 p.m. UTC | #3 
>>>>> "julien" == julien boibessot <julien.boibessot@free.fr> writes:

 > From: Julien BOIBESSOT <julien.boibessot@armadeus.com>
 > Has been tested with: "./support/scripts/test-pkg -c tyrian.cfg -p opentyrian"
 > Fixes: http://autobuild.buildroot.net/results/0e2345db82b33f591958fc0f72ad914adafe0522
 > and some similar previous build failure.

 > Thanks Thomas for the tip ;-).

 > Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>

Committed to 2017.02.x, thanks.
diff mbox

Patch

diff --git a/package/opentyrian/opentyrian.mk b/package/opentyrian/opentyrian.mk
index 4417512..12e41ad 100644
--- a/package/opentyrian/opentyrian.mk
+++ b/package/opentyrian/opentyrian.mk
@@ -24,7 +24,7 @@  define OPENTYRIAN_BUILD_CMDS
 		CC="$(TARGET_CC)" \
 		STRIP="/bin/true" \
 		SDL_CONFIG="$(STAGING_DIR)/usr/bin/sdl-config" \
-		LDFLAGS="-lm" \
+		LDFLAGS="$(TARGET_LDFLAGS) -lm" \
 		WITH_NETWORK="$(OPENTYRIAN_NETWORK)" \
 		-C $(@D) release
 endef