<blockquote>
<p>Thanks. Your patch seems to completely remove the "-nopie" option for all versions of gcc. I'm pretty sure this is going to break the versions that required the workaround in the first place.</p>
</blockquote>

<p>I did not find a version of <code>gcc</code> that requires (or even understands) <code>-nopie</code>.<br>
Do you have any information what systems do require this?</p>

<blockquote>
<p>Does your version of gcc actually require the workaround? It's not required on mine (also 4.9.2).</p>
</blockquote>

<p>By default <code>gcc</code> does not enable PIE. This changes when you enable hardening toolchains and/or enable some options in your environment, though.</p>

<p>But that is what my change is about: It uses the preprocessor to check whether or not <code>gcc</code> generates PIE code. The workaround is added in first case only.</p>

<p>Thinking about some more changes...</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you commented.<br />Reply to this email directly or <a href="https://github.com/ipxe/ipxe/pull/49#issuecomment-197251481">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AArTVEgdvrGJ7bm0P5siH59e8VMYz8QOks5pt9tMgaJpZM4HxBRW.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/ipxe/ipxe/pull/49#issuecomment-197251481"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>