<div dir="ltr"><div>Hi Sean,</div><div><br></div>Well, can't say for sure about the version numbering systems that are being used in different projects, but I think that at least nothing will break spuriously if the projects bump the major version and we'll continue using locally the possibly older version that's specified in the bootstrap file, and bump them manually sometimes. Major bump happens very rarely anyway.<div><br></div><div>All the best,</div><div>Pyry</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-10-21 23:08 GMT+03:00 Sean McGovern <span dir="ltr"><<a href="mailto:gseanmcg@gmail.com" target="_blank">gseanmcg@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Hi Pyry,</p><span class="">
<p dir="ltr">On Oct 21, 2016 15:32, "Pyry Kontio" <<a href="mailto:pyry.kontio@drasa.eu" target="_blank">pyry.kontio@drasa.eu</a>> wrote:<br>
><br>
> The bootstrap script that outputs which binary tools need to be installed, incorrectly<br>
> detects that Protocol Buffers 3.1 is a compatible version with 2.6. I changed the<br>
> check on the major version number to require it to be same, not just bigger,<br>
> since changes on the major version numbers might be breaking changes, as was the case<br>
> here.<br>
> ---<br>
>  extras/tools/bootstrap | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/extras/tools/bootstrap b/extras/tools/bootstrap<br>
> index 9666436..95870fa 100755<br>
> --- a/extras/tools/bootstrap<br>
> +++ b/extras/tools/bootstrap<br>
> @@ -60,7 +60,7 @@ else<br>
>      needminor=`echo $2|cut -d. -f2`<br>
>      needmicro=`echo $2|cut -d. -f3`<br>
>      [ -z "$needmicro" ] && needmicro=0<br>
> -    if [ "$needmajor" -gt "$gotmajor" \<br>
> +    if [ "$needmajor" -ne "$gotmajor" \<br>
>           -o "$needmajor" -eq "$gotmajor" -a "$needminor" -gt "$gotminor" \<br>
>           -o "$needmajor" -eq "$gotmajor" -a "$needminor" -eq "$gotminor" -a "$needmicro" -gt "$gotmicro" ]<br>
>      then<br>
> --</p>
</span><p dir="ltr">Oooh! Nice catch!</p>
<p dir="ltr">Can we be sure that all the other tools only bump major for a breaking change, though? If so, LGTM.</p>
<p dir="ltr">----<br>
Sean McGovern -  <a href="tel:%2B1%20613%20862%206507" value="+16138626507" target="_blank">+1 613 862 6507</a></p>
<br>______________________________<wbr>_________________<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/<wbr>listinfo/vlc-devel</a><br>
<br></blockquote></div><br></div>