<font size="2"><font face="tahoma,sans-serif">Hi;<br></font></font><br><div class="gmail_quote">On Mon, Nov 8, 2010 at 12:16 PM, Frits van Bommel <span dir="ltr"><<a href="mailto:fvbommel@gmail.com">fvbommel@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Sun, Nov 7, 2010 at 9:14 PM, Rafael Espindola<br>
<<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> +  if (IsUbuntu(Distro))<br>
> +    ExtraOpts.push_back("-z relro");<br>
<br>
</div>Regular binutils ld doesn't mind this, but gold gives an error: "ld:<br>
relro: unknown -z option" (note that there are _two_ spaces in front<br>
of "relro" there)<br>
I think the problem is that you're pushing this as a single argument<br>
with a space in it, and apparently gold doesn't strip the space when<br>
it processes the -z option.<br>
You should probably either push "-zrelro" or push "-z" and "relro"<br>
separately. (neither give an error from the command line, though I<br>
don't know what it's supposed to do so I can't check whether the first<br>
does what's expected...)<br>
The attached patch implements the second option.<br></blockquote><div><br></div><div>As far as I know "-z,relro" should work too. </div><div><br></div><div>Regards,</div><div>ismail</div><div><br></div></div>