<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">​Hi all,​</div><br><div class="gmail_quote">2016-08-26 4:18 GMT-03:00 Raphael Hertzog <span dir="ltr"><<a href="mailto:hertzog@debian.org" target="_blank">hertzog@debian.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Please respect the git conventions to format git commit logs.<br>
One line summary. One empty line. Then a long description.</div></blockquote></div><br><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">​Sorry, will follow this on the next commits, thanks for warning me.​</div><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">​Then I believe this commit to be wrong, or at least your description is<br>
wrong.<br>
<br>
"-pie" disables -fPIE while "+all" enables it... so it's not redundant,<br>
that was the correct syntax to say "enable everything except -fPIE".<br>
<br>
But if it works with -fPIE then there's no reason to disable it for sure.<br>
I did not check that.​</div></blockquote><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">​<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">Yeah,
 the commit description is wrong, -fPIE would work because the CFLAGS 
were being reset on <a href="http://configure.ac" target="_blank">configure.ac</a> ("CFLAGS=") instead of associating with
 previously passed flags ("CFLAGS+=").<br><br>​2016-08-26 5:54 GMT-03:00 Marcos Fouces <span dir="ltr"><<a href="mailto:mfouces@yahoo.es" target="_blank">mfouces@yahoo.es</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>When
 i first packaged t50, i needed to disable -pie flag because it 
triggered a compilation error and i wasn't able to build the sources 
with all the hardening flags (with just hardening=+all). As Raphaël 
points out, this is the opposite of you believe.<br>
<br>
Anyway, you did a good move adapting my patch not to include the -g flag
 but to get rid of other CPU specific artifacts. I just build the 
package and it seems to be fine.<br>
<br>
I updated the changelog to reflect your work in a new revision and added a missing comma in control file.<br>
<br>
IMHO, it worths a new upload in order to check if it builds on more architectures.<br>
<br>
Thanks for your help!<br>
</div></blockquote><br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">I believe the above explains the -pie flag problem, i agreed we need a new t50 upload, Raphael, can you do that for us?<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">I see that i missed a comma on the control file and you fixed for me, thanks a lot!<br></div><div class="gmail_extra">​<br clear="all"><div><div data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="font-family:verdana,sans-serif"><span style="color:rgb(102,102,102)">Samuel Henrique O. P. [samueloph]</span></span></div></div></div></div></div>
</div></div></div>