<!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-16 15:52, 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 jeudi 16 mars 2017 12:20:35 EET Marvin Scholz wrote:</code></pre>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code>  test_modules_packetizer_hxxx_LDFLAGS = -no-install -static # WTF

 +endif

  test_modules_keystore_SOURCES = modules/keystore/test.c

  test_modules_keystore_LDADD = $(LIBVLCCORE) $(LIBVLC)

  test_modules_tls_SOURCES = modules/misc/tls.c</code></pre>

</blockquote>

<pre><code> Removing -no-install makes no sense: Those targets are not meant to be 

 installed. In fact, -no-install should be set for all noinst_- and 

 check_PROGRAMS targets (and obviously MUST NOT be used for any other PROGRAMS 

 target).

 I am not sure if -static still carries any meaning, since libvlc and 

 libvlccore are not built statically in any case. IIRC, in some versions of 

 libtool, it is or was necessary to make -no-install actually work. But it 

 might also be cargo cult. But maybe ask whomever wrote "# WTF" there??</code></pre>

</blockquote>

<p>You did, in <a href="http://git.videolan.org/?p=vlc.git;a=commitdiff;h=2b5b2845f34;hp=062c2c3ea191fc7228bd8fe7e4771c1b43ff196f"><code>2b5b2845f34</code></a>.</p>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code> So in any case, this does not look right.</code></pre>

</blockquote>

</body>

</html>