<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-20 16:08 GMT+01:00 Felipe Sateler <span dir="ltr"><<a href="mailto:fsateler@debian.org" target="_blank">fsateler@debian.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Mar 20, 2014 at 11:44 AM, Jaromír Mikeš <<a href="mailto:mira.mikes@gmail.com">mira.mikes@gmail.com</a>> wrote:<br>

><br>
><br>
><br>
> 2014-03-20 14:30 GMT+01:00 Felipe Sateler <<a href="mailto:fsateler@debian.org">fsateler@debian.org</a>>:<br>
><br>
>> On Thu, Mar 20, 2014 at 10:06 AM, Jaromír Mikeš <<a href="mailto:mira.mikes@gmail.com">mira.mikes@gmail.com</a>><br>
>> wrote:<br>
>> > 2014-03-20 13:23 GMT+01:00 <<a href="mailto:mira-guest@users.alioth.debian.org">mira-guest@users.alioth.debian.org</a>>:<br>
>> >><br>
>> >> The following commit has been merged in the master branch:<br>
>> >> commit c841640a6dc119dce9f85f8d65f09c5a2a753f02<br>
>> >> Author: Jaromír Mikeš <<a href="mailto:mira.mikes@seznam.cz">mira.mikes@seznam.cz</a>><br>
>> >> Date:   Thu Mar 20 13:23:01 2014 +0100<br>
>> >><br>
>> >>     Update patch.<br>
>> >><br>
>> >> diff --git a/debian/patches/01-gcc_flags.patch<br>
>> >> b/debian/patches/01-gcc_flags.patch<br>
>> >> index d9f88dc..72b02c8 100644<br>
>> >> --- a/debian/patches/01-gcc_flags.patch<br>
>> >> +++ b/debian/patches/01-gcc_flags.patch<br>
>> >> @@ -9,14 +9,14 @@ Forwarded: yes<br>
>> >><br>
>> >>  Index: yoshimi/src/CMakeLists.txt<br>
>> >>  ===================================================================<br>
>> >> ---- yoshimi.orig/src/CMakeLists.txt    2014-03-17 00:55:47.478391726<br>
>> >> +0100<br>
>> >> -+++ yoshimi/src/CMakeLists.txt 2014-03-17 00:55:47.474391845 +0100<br>
>> >> +--- yoshimi.orig/src/CMakeLists.txt    2014-03-20 12:26:13.702271262<br>
>> >> +0100<br>
>> >> ++++ yoshimi/src/CMakeLists.txt 2014-03-20 12:27:18.278644212 +0100<br>
>> >>  @@ -81,6 +81,9 @@<br>
>> >>   set (BuildOptionsDebug<br>
>> >>       "-O0 -g3 -ggdb -Wall -Wpointer-arith -ldl" CACHE STRING "Debug<br>
>> >> build<br>
>> >> flags")<br>
>> >><br>
>> >>  +set (BuildOptionsDebian<br>
>> >> -+    "-O3 -ffast-math -fomit-frame-pointer" CACHE STRING "Debian build<br>
>> >> flags")<br>
>> >> ++    "-O3 -ffast-math -fomit-frame-pointer -ldl" CACHE STRING "Debian<br>
>> >> build flags")<br>
>> >>  +<br>
>> >>   find_package (PkgConfig REQUIRED)<br>
>> >>   if (PKG_CONFIG_FOUND)<br>
>> ><br>
>> ><br>
>> > Hi,<br>
>> ><br>
>> > again having difficulty to pass flags ... Now LDFLAGS are problematic.<br>
>> > If I will patch CMakeLists.txt and add -z relro directly things working<br>
>> > but<br>
>> > I am not able pass flags via LDFLAGS variable. :(<br>
>> ><br>
>> > "-O3 -ffast-math -fomit-frame-pointer -ldl -z relro" CACHE STRING<br>
>> > "Debian<br>
>> > build flags")<br>
>> ><br>
>> > any idea?<br>
>><br>
>> The variables are CMAKE_EXE_LINKER_FLAGS, CMAKE_MODULE_LINKER_FLAGS<br>
>> and CMAKE_SHARED_LINKER_FLAGS.<br>
>><br>
>> You should probably do the same with the cflags CMAKE_C_FLAGS and<br>
>> CMAKE_CXX_FLAGS instead of patching the CMakeLists.txt.<br>
><br>
><br>
> Surprisingly just LDFLAGS don't works.<br>
><br>
> I added this to rules file but still no success.<br>
> CMAKE_EXE_LINKER_FLAGS+=$(LDFLAGS)<br>
> CMAKE_MODULE_LINKER_FLAGS+=$(LDFLAGS)<br>
> CMAKE_SHARED_LINKER_FLAGS+=$(LDFLAGS)<br>
><br>
> Can this line 360 from CMakeLists.txt be problematic? Because I see in build<br>
> log section "linking" to be -ldl passed twice.<br>
><br>
> set (CMAKE_EXE_LINKER_FLAGS "-ldl")<br>
><br>
> To change set to add or sth like this?<br>
<br>
</div></div>Yes, it is related, as the CMakeLists.txt is overriding the setting.<br>
You should probably remove that line and add dl to the<br>
target_link_libraries call just a couple of lines after.</blockquote><div><br></div><div>Works! Thank you!<br><br></div><div>mira <br></div></div></div></div>