<blockquote>
<p>Finally, I would claim that it is not a fix of an undefined symbol but a fix of the make system (only for ESP32 at the moment).</p>
</blockquote>
<p>This is why I would prefer to merge them before this one :)<br>
They fix something, when this one only changes the detection.</p>
<p>My changes only rely on "it does not provide compilation error" or maybe checking some build size.</p>
<p>Your fixes are more about reviewing the defined symbols so easier for me to review on their own as I am not sure about the consequences of my changes :D</p>
<p>However, putting the -ffunctions-sections did not solve the <code>esp8266</code> issues so there may be something more.<br>
I noticed that also the other architectures provide the option in both the CFLAGS and the LINKFLAGS but I do not understand why it should be that way…</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/RIOT-OS/RIOT/pull/11246#issuecomment-476858120">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AEn7YHMKWySDljMLMSMx9DTpHcilKlaxks5vao_EgaJpZM4cFDGe">mute the thread</a>.<img src="https://github.com/notifications/beacon/AEn7YCGMuVJqTk7aLNfAmyvi22WIbhpbks5vao_EgaJpZM4cFDGe.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/RIOT-OS/RIOT","title":"RIOT-OS/RIOT","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/RIOT-OS/RIOT"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cladmi in #11246: \u003e Finally, I would claim that it is not a fix of an undefined symbol but a fix of the make system (only for ESP32 at the moment).\r\n\r\nThis is why I would prefer to merge them before this one :)\r\nThey fix something, when this one only changes the detection.\r\n\r\nMy changes only rely on \"it does not provide compilation error\" or maybe checking some build size.\r\n\r\nYour fixes are more about reviewing the defined symbols so easier for me to review on their own as I am not sure about the consequences of my changes :D\r\n\r\nHowever, putting the -ffunctions-sections did not solve the `esp8266` issues so there may be something more.\r\nI noticed that also the other architectures provide the option in both the CFLAGS and the LINKFLAGS but I do not understand why it should be that way…"}],"action":{"name":"View Pull Request","url":"https://github.com/RIOT-OS/RIOT/pull/11246#issuecomment-476858120"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/RIOT-OS/RIOT/pull/11246#issuecomment-476858120",
"url": "https://github.com/RIOT-OS/RIOT/pull/11246#issuecomment-476858120",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>