<div dir="ltr">Hi!<div>Regarding the rustfmt in CI, what do we want as the desired behaviour. Should the CI check and fail if the Rust style conventions are not followed ? It is very easy to obey the style conventions. We just need to run cargo fmt in src/ and modules/ so should not be much of an issue for the developers. We can distribute a simple pre-commit hook as well to automate it as well. What are your thoughts?</div><div>Thanks.</div><div>Regards,</div><div>Kartik</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 17, 2020 at 1:49 AM Alexandre Janniaux <<a href="mailto:ajanni@videolabs.io">ajanni@videolabs.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
On Thu, Sep 17, 2020 at 01:24:41AM +0530, Kartik Ohri wrote:<br>
> > I noticed a problem during testing: src/vlccore-rs and src/vlccore-sys<br>
> > are not rebuilt when sources are modified. Cargo should probably be<br>
> > called everytime to check if it must rebuild things.<br>
> ><br>
>  Yes, this can be done. I can add a Rust Sources variable in the <a href="http://common.am" rel="noreferrer" target="_blank">common.am</a><br>
>  makefile. It might also be fixed using a cargo workspace. I have been<br>
>  exploring how to run rustfmt on all rust crates and execute all rust tests.<br>
>  The best solution seems to be a cargo workspace. I'll try it out and report<br>
>  my findings.<br>
<br>
A cargo workspace seems like the right tool for that indeed.<br>
You probably want two of them so as to integrate within the<br>
buildsystem nicely though.<br>
<br>
Regards,<br>
--<br>
Alexandre Janniaux<br>
Videolabs<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>