<html><head></head><body>Hi,<br><br>Bootstrap was supposed to set manual parameters based on the command line flags, while every thing else is in the Makefiles. It was mostly consistent until people goofed around on the path of least resistance.<br><br><div class="gmail_quote">Le 5 juin 2019 14:47:48 GMT+03:00, "Martin Storsjö" <martin@martin.st> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">On Wed, 5 Jun 2019, Steve Lhomme wrote:<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">On 2019-06-05 13:10, Marvin Scholz wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><hr>  contrib/bootstrap | 7 ++-----<br>  1 file changed, 2 insertions(+), 5 deletions(-)<br><br>diff --git a/contrib/bootstrap b/contrib/bootstrap<br>index fc6fbcfb2b..02a958ca79 100755<br>--- a/contrib/bootstrap<br>+++ b/contrib/bootstrap<br>@@ -360,11 +360,10 @@ case "${OS}" in<br>                  add_make_enabled "HAVE_WIN32"<br>               case "${HOST}" in<br>                   *winphone*|*windowsphone*)<br>+                           add_make_enabled "HAVE_WINSTORE"<br>                            add_make_enabled "HAVE_WINDOWSPHONE"<br>                                ;;<br>-           esac<br>-         case "${HOST}" in<br>-                  *winphone*|*windowsphone*|*winrt*|*uwp*)<br>+                     *winrt*|*uwp*)<br>                                add_make_enabled "HAVE_WINSTORE"<br></blockquote> I think it's better to be able to associate directly what a define <br> corresponds to.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">                               ;;<br>            esac<br>@@ -372,8 +371,6 @@ case "${OS}" in<br>                   amd64*|x86_64*)<br>                               add_make_enabled "HAVE_WIN64"<br>                               ;;<br>-           esac<br>-         case "${HOST}" in<br></blockquote>armv7a and win64 are not in the same 'family' of defines. Another arm <br>variant could have win64 (and it should?) and another arm define. It's <br>better to keep them separate.<br></blockquote><br>Yes, HAVE_WIN64 is/should be set for windows/aarch64 as well. It is set by <br>main.mak though. I'm not sure of the division between setting it in <br>bootstrap and main.mak though, both should probably be done consistently <br>in either of them (or both?).<br><br>// Martin<hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>