<div dir="ltr"><div dir="ltr">On Tue, Jul 21, 2020 at 6:07 PM Marvin Scholz <<a href="mailto:epirat07@gmail.com">epirat07@gmail.com</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"><br>
<br>
On 21 Jul 2020, at 14:29, Kartik Ohri wrote:<br>
<br>
> 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>
><br>
>> Aren't a lot of the the usual triplet we already use ?<br>
>><br>
>> I need to check for Android and iOS but for Windows and Linux those are<br>
> different. For macOS, the docs (<a href="https://wiki.videolan.org/MacOSCompile/" rel="noreferrer" target="_blank">https://wiki.videolan.org/MacOSCompile/</a>)<br>
> say that triple is like x86_64-apple-darwin15 so if some user specifies a<br>
> version suffixed triple for macOS the Rust target will be incorrect again.<br>
><br>
>> 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>
<br>
Due to recent events you can not assume<br>
macOS == x86_64<br></blockquote><div>Yes, right. And Rust does not have an ARM mac toolchain yet. So, I'll check for x86_64 while setting this. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>>> +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><br>
<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>
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>