<div dir="ltr"><div dir="ltr">On Mon, Sep 14, 2020 at 11:38 AM 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-09-11 21:33, Kartik Ohri wrote:<br>
> The API is written in two layers. vlccore-sys<br>
> crate contains one to one bindings of libvlc<br>
> functions. vlccore-rs crate wraps vlccore-sys and<br>
> exposes a Rust idiomatic API for writing modules<br>
> in rust. As of now, the API exposes only a subset<br>
> of the vlc_stream API. More functionality can be<br>
> added to Rust API as required in the future.<br>
> ---<br>
>   .gitignore                    |   2 +-<br>
>   <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>                  |  13 ++++<br>
>   src/vlccore-rs/Cargo.toml     |  10 +++<br>
>   src/vlccore-rs/src/<a href="http://lib.rs" rel="noreferrer" target="_blank">lib.rs</a>     |   1 +<br>
>   src/vlccore-rs/src/<a href="http://stream.rs" rel="noreferrer" target="_blank">stream.rs</a>  | 120 ++++++++++++++++++++++++++++++++++<br>
>   src/vlccore-sys/Cargo.toml    |  10 +++<br>
>   src/vlccore-sys/src/<a href="http://lib.rs" rel="noreferrer" target="_blank">lib.rs</a>    |   2 +<br>
>   src/vlccore-sys/src/<a href="http://stream.rs" rel="noreferrer" target="_blank">stream.rs</a> |  24 +++++++<br>
>   8 files changed, 181 insertions(+), 1 deletion(-)<br>
>   create mode 100644 src/vlccore-rs/Cargo.toml<br>
>   create mode 100644 src/vlccore-rs/src/<a href="http://lib.rs" rel="noreferrer" target="_blank">lib.rs</a><br>
>   create mode 100644 src/vlccore-rs/src/<a href="http://stream.rs" rel="noreferrer" target="_blank">stream.rs</a><br>
>   create mode 100644 src/vlccore-sys/Cargo.toml<br>
>   create mode 100644 src/vlccore-sys/src/<a href="http://lib.rs" rel="noreferrer" target="_blank">lib.rs</a><br>
>   create mode 100644 src/vlccore-sys/src/<a href="http://stream.rs" rel="noreferrer" target="_blank">stream.rs</a><br>
> <br>
> diff --git a/.gitignore b/.gitignore<br>
> index fc368212c8..6c049a80b1 100644<br>
> --- a/.gitignore<br>
> +++ b/.gitignore<br>
> @@ -46,7 +46,7 @@ wxvlc<br>
>   vlc_install_dir/*<br>
>   plugins.dat<br>
>   patches/*<br>
> -<br>
> +**/Cargo.lock<br>
>   include/vlc/libvlc_version.h<br>
>   <br>
>   # Ignore build dirs<br>
> diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> index 99e4669942..53d56a725c 100644<br>
> --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> @@ -1896,6 +1896,19 @@ AS_IF([test "${enable_sout}" != "no"], [<br>
>   ])<br>
>   AM_CONDITIONAL([ENABLE_SOUT], [test "${enable_sout}" != "no"])<br>
>   <br>
> +dnl Rust Modules<br>
> +AC_ARG_ENABLE([rust],<br>
> +    AS_HELP_STRING([--enable-rust], [enable building Rust modules (default disabled)]))<br>
> +AS_IF([test "${enable_rust}" = "yes"],<br>
> +      [AC_DEFINE(ENABLE_RUST, 1, [Define to 1 for building rust modules.])])<br>
> +AM_CONDITIONAL([BUILD_RUST], [test "${enable_rust}" = "yes"])<br>
<br>
You should only set to true this if cargo is available.<br>
<br>
> +if test "${enable_rust}" = "yes"<br>
> +then<br>
> +    AC_CHECK_PROG(CARGO, [cargo], [yes], [no])<br>
> +    AS_IF([test "x$CARGO" = "xno"],<br>
> +          AC_MSG_ERROR([cargo not found. cargo is required to build rust modules]))<br>
> +fi<br></blockquote><div><br></div><div>Ah! Yes. I have made the changes below and tested them locally. Does this address the issue ?<br>===================================================================<br>--- <a href="http://configure.ac">configure.ac</a>   (revision 329abd5c6c5a8ac45765b2fca9883dae7155207e)<br>+++ <a href="http://configure.ac">configure.ac</a>   (date 1600072942614)<br>@@ -1899,15 +1899,13 @@<br> dnl Rust Modules<br> AC_ARG_ENABLE([rust],<br>     AS_HELP_STRING([--enable-rust], [enable building Rust modules (default disabled)]))<br>-AS_IF([test "${enable_rust}" = "yes"],<br>-      [AC_DEFINE(ENABLE_RUST, 1, [Define to 1 for building rust modules.])])<br>-AM_CONDITIONAL([BUILD_RUST], [test "${enable_rust}" = "yes"])<br> if test "${enable_rust}" = "yes"<br> then<br>     AC_CHECK_PROG(CARGO, [cargo], [yes], [no])<br>     AS_IF([test "x$CARGO" = "xno"],<br>           AC_MSG_ERROR([cargo not found. cargo is required to build rust modules]))<br> fi<br>+AM_CONDITIONAL([BUILD_RUST], [test "x$CARGO" = "xyes"])<br> <br> dnl Lua modules<br> AC_ARG_ENABLE([lua],<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
____________________________________ </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">___________<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>