<div dir="ltr"><div dir="ltr">On Tue, Jul 21, 2020 at 4:38 PM Steve Lhomme <<a href="mailto:robux4@ycbcr.xyz" target="_blank">robux4@ycbcr.xyz</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Aren't a lot of the the usual triplet we already use ?<br>
<br></blockquote><div>I need to check for Android and iOS but for Windows and Linux those are different. For macOS, the docs (<a href="https://wiki.videolan.org/MacOSCompile/">https://wiki.videolan.org/MacOSCompile/</a>) say that triple is like <span style="background-color:rgb(248,249,250);color:rgb(0,0,0);font-family:monospace,monospace;font-size:14px;white-space:pre-wrap">x86_64-apple-darwin15 </span>so if some user specifies a version suffixed triple for macOS the Rust target will be incorrect again. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
In that case it would be better to use the triplet by default and only <br>
set the values that differ from it.<br>
<br>
Alternatively you could disable building RUST contribs if RUST_TARGET is <br>
not set.<br>
<br>
On 2020-07-21 11:34, rustyc wrote:<br>
> ---<br>
>   contrib/src/main.mak | 30 ++++++++++++++++++++++++++++++<br>
>   1 file changed, 30 insertions(+)<br>
> <br>
> diff --git a/contrib/src/main.mak b/contrib/src/main.mak<br>
> index 3119a39875..c8d7df4202 100644<br>
> --- a/contrib/src/main.mak<br>
> +++ b/contrib/src/main.mak<br>
> @@ -440,6 +440,36 @@ else<br>
>   MESON = meson $(MESONFLAGS)<br>
>   endif<br>
>   <br>
> +ifdef HAVE_WIN32<br>
> +RUST_TARGET = i686-pc-windows-gnu<br>
> +else ifdef HAVE_WIN64<br>
> +RUST_TARGET = x86_64-pc-windows-gnu<br>
> +else ifdef HAVE_BSD<br>
> +RUST_TARGET = x86_64-unknown-freebsd<br>
> +else ifdef HAVE_ANDROID<br>
> +ifeq ($(PLATFORM_SHORT_ARCH), arm64)<br>
> +RUST_TARGET = aarch64-linux-android<br>
> +else ifeq ($(PLATFORM_SHORT_ARCH), arm)<br>
> +RUST_TARGET = arm-linux-androideabi<br>
> +else ifeq ($(PLATFORM_SHORT_ARCH), x86)<br>
> +RUST_TARGET = i686-linux-android<br>
> +else ifeq ($(PLATFORM_SHORT_ARCH), x86_64)<br>
> +RUST_TARGET = x86_64-linux-android<br>
> +endif<br>
> +else ifdef HAVE_IOS<br>
> +ifeq ($(PLATFORM_SHORT_ARCH), arm64)<br>
> +RUST_TARGET = aarch64-apple-ios<br>
> +else ifeq ($(PLATFORM_SHORT_ARCH), x86_64)<br>
> +RUST_TARGET = x86_64-apple-ios<br>
> +endif<br>
> +else ifdef HAVE_MACOSX<br>
> +RUST_TARGET = x86_64-apple-darwin<br>
> +else ifdef HAVE_SOLARIS<br>
> +RUST_TARGET = x86_64-sun-solaris<br>
> +else ifdef HAVE_LINUX<br>
> +RUST_TARGET = x86_64-unknown-linux-gnu<br>
> +endif<br>
> +<br>
>   ifdef GPL<br>
>   REQUIRE_GPL =<br>
>   else<br>
> -- <br>
> 2.25.1<br>
> <br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
> <br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div></div>