<div dir="ltr"><div dir="ltr">On Tue, Jul 21, 2020 at 4:38 PM Steve Lhomme <<a href="mailto:robux4@ycbcr.xyz">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">On 2020-07-21 11:34, rustyc wrote:<br>
> ---<br>
>   contrib/src/main.mak         |  6 +++---<br>
>   contrib/src/rav1e/SHA512SUMS |  1 +<br>
>   contrib/src/rav1e/rules.mak  | 23 +++++++++++++++++++++++<br>
>   3 files changed, 27 insertions(+), 3 deletions(-)<br>
>   create mode 100644 contrib/src/rav1e/SHA512SUMS<br>
>   create mode 100644 contrib/src/rav1e/rules.mak<br>
> <br>
> diff --git a/contrib/src/main.mak b/contrib/src/main.mak<br>
> index c8d7df4202..4d00cfda4d 100644<br>
> --- a/contrib/src/main.mak<br>
> +++ b/contrib/src/main.mak<br>
> @@ -440,10 +440,10 @@ 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>
> +ifdef HAVE_WIN64<br>
>   RUST_TARGET = x86_64-pc-windows-gnu<br>
> +else ifdef HAVE_WIN32<br>
> +RUST_TARGET = i686-pc-windows-gnu<br>
<br>
This should go in patch [2/4].<br>
<br>
>   else ifdef HAVE_BSD<br>
>   RUST_TARGET = x86_64-unknown-freebsd<br>
>   else ifdef HAVE_ANDROID<br>
> diff --git a/contrib/src/rav1e/SHA512SUMS b/contrib/src/rav1e/SHA512SUMS<br>
> new file mode 100644<br>
> index 0000000000..73d8e9feb8<br>
> --- /dev/null<br>
> +++ b/contrib/src/rav1e/SHA512SUMS<br>
> @@ -0,0 +1 @@<br>
> +140995820053e4ac8de5f31fcd430e991e47642c34b9433abea618ed6262ec593b777e9334e5f138aa34854e0bdd044d1d2dadc3c917394641193643b4e92e0c  rav1e-0.3.2.tar.gz<br>
> diff --git a/contrib/src/rav1e/rules.mak b/contrib/src/rav1e/rules.mak<br>
> new file mode 100644<br>
> index 0000000000..f4515adad8<br>
> --- /dev/null<br>
> +++ b/contrib/src/rav1e/rules.mak<br>
> @@ -0,0 +1,23 @@<br>
> +# rav1e<br>
> +<br>
> +RAV1E_VERSION := 0.3.2<br>
> +RAV1E_URL := <a href="https://github.com/xiph/rav1e/archive/v$(RAV1E_VERSION).tar.gz" rel="noreferrer" target="_blank">https://github.com/xiph/rav1e/archive/v$(RAV1E_VERSION).tar.gz</a><br>
> +<br>
> +ifeq ($(call need_pkg,"rav1e"),)<br>
> +# PKGS_FOUND += rav1e<br>
> +endif<br>
> +<br>
> +$(TARBALLS)/rav1e-$(RAV1E_VERSION).tar.gz:<br>
> +     $(call download_pkg,$(RAV1E_URL),rav1e)<br>
> +<br>
> +.sum-rav1e: rav1e-$(RAV1E_VERSION).tar.gz<br>
> +<br>
> +rav1e: rav1e-$(RAV1E_VERSION).tar.gz .sum-rav1e<br>
> +     $(UNPACK)<br>
> +     $(MOVE)<br>
> +<br>
> +.rav1e: rav1e<br>
> +     cd $< && RUSTFLAGS="-C lto -C panic=abort -C opt-level=z" cargo cinstall \<br>
> +      --no-default-features --features asm --prefix=$(PREFIX) \<br>
> +      --target=$(RUST_TARGET) --library-type staticlib --release<br>
<br>
Should the --release only be set for release builds ?<br></blockquote><div>The release is set for all builds as the rav1e devs suggested. rav1e 's has configured the release profile to allow debug symbols in it. So, the only difference is the release builds are optimized.</div><div><br></div><div>I have addressed the other issues in the new patch.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
You should create a CARGO variable in main.mak similar to CMAKE to put <br>
all the common stuff passed to cargo. So it can be reused by other modules.<br>
<br>
> +     touch $@<br>
> \ No newline at end of file<br>
<br>
You should add a new line to make git happy.<br>
<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>