<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  <meta http-equiv="Content-Style-Type" content="text/css" />
  <meta name="generator" content="pandoc" />
  <title></title>
  <style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi Rémi,</p>
<p>On 2017-03-15 14:51, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> On March 15, 2017 11:10:27 AM GMT+02:00, "Filip Roséen" <filip@atch.se> wrote:</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code>There are cases where it is annoying to build VLC with this
restriction as you would like to use symbols that are simply not
present (in terms of definition) when things are being built.

SUMMARY:

 - "-Wl,-z,defs" is now opt-out + automatic detection
 - "-no-undefined" is now opt-out

Both options are governed by a new configure flag,
"--{enable,disable}-no-undefined-symbols" as they are tightly coupled.</code></pre>
</blockquote>
</blockquote>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> I added this because I am fed up with devs not checking their
 dependencies correctly, breaking my and Debian builds frequently.</code></pre>
</blockquote>
<p>I certainly understand the frustration of waking up early to work on vlc, only to realize that things are broken after a fresh <code>git pull</code>.</p>
<p>However, I do not see that as a compelling argument for not enabling the functionality that this patch introduce, especially given that the check for undefined-reference’s are opt-<strong>out</strong> and not opt-<strong>in</strong>.</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> So obviously, I am very much against this change. This removes a
 very useful build-time diagnostic and WILL cause more build breaks.</code></pre>
</blockquote>
<p>The check to verify that no undefined symbols are present in <em>vlc-objects</em> are very nice to have per default, and I salute you for the addition, but I do not see how this patch contradicts that statement nor functionality.</p>
<p>Best Regards,<br />
Filip</p>
</body>
</html>