<div dir="ltr"><div dir="ltr">On Wed, Jul 22, 2020 at 8:03 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</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">Le tiistaina 21. heinäkuuta 2020, 21.40.21 EEST Martin Storsjö a écrit :<br>
> On Tue, 21 Jul 2020, 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>
> <br>
> This is incorrectly assuming that windows implies x86. HAVE_WIN* only<br>
> implies the bitness, but it can still be armv7 and aarch64 in addition to<br>
> i686 and x86_64.<br>
<br>
Not even that. HAVE_WIN32 implies that the Win32 API is available, which <br>
requires pointers of 32 bits *or more*.<br>
<br>
And HAVE_WIN64 is not even defined in VLC contribs.<br>
<br></blockquote><div>I have fixed the bitness issue in the newer patch I sent for this. In that patch, I have assumed that HAVE_WIN32 implies the target is Windows (as Cmake and Meson do the same) and dropped the use of HAVE_WIN64.</div><div><br></div><div>But it seems that HAVE_WIN64 is defined in contribs around line 41 in contrib/src/main.mak.</div><div><pre style="background-color:rgb(43,43,43);color:rgb(169,183,198);font-family:"JetBrains Mono",monospace;font-size:9.8pt">ifeq ($(ARCH)-$(HAVE_WIN32),x86_64-1)<br>HAVE_WIN64 := 1<br>endif<br>ifeq ($(ARCH)-$(HAVE_WIN32),aarch64-1)<br>HAVE_WIN64 := 1<br>endif</pre></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-- <br>
雷米‧德尼-库尔蒙<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
<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></blockquote></div></div>