<div dir="auto">I think it's the sip_include test which is causing issues. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mar. 1 oct. 2019 à 12:38, Sandro Santilli <<a href="mailto:strk@kbt.io">strk@kbt.io</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ok sorry for the noise, the OptName=s syntax is to specify that<br>
the option takes an argument, doesn't give an indication of a<br>
corresponding short-version option. I'm still trying to understand<br>
why the server_auto.sip file is modified by travis...<br>
<br>
--strk;<br>
<br>
On Tue, Oct 01, 2019 at 12:31:13PM +0200, Sandro Santilli wrote:<br>
> Denis, you seem to have introduced both the new switch (with the<br>
> typo) and the corresponding call with commit<br>
> 6301f9b6f3ba1c9b39ffe55698365ab98aa059a1 on May 2018.<br>
> <br>
> I'm not sure how this could have gone unnoticed till now,<br>
> maybe the <a href="http://sipify.pl" rel="noreferrer noreferrer" target="_blank">sipify.pl</a> sript should noisily fail when passed<br>
> unknown switches ?<br>
> <br>
> --strk;<br>
> <br>
> On Tue, Oct 01, 2019 at 12:28:39PM +0200, Sandro Santilli wrote:<br>
> > Looking at Travis output, the qgis_sip_uptodate test passes:<br>
> > <br>
> >   <a href="https://travis-ci.org/qgis/QGIS/jobs/591915632#L958" rel="noreferrer noreferrer" target="_blank">https://travis-ci.org/qgis/QGIS/jobs/591915632#L958</a><br>
> > <br>
> > BUT a test I've added to check if repository files changed<br>
> > shows that the `python/server/server_auto.sip` file in the<br>
> > git repository was changed during tests:<br>
> > <br>
> >   <a href="https://travis-ci.org/qgis/QGIS/jobs/591915632#L941-L944" rel="noreferrer noreferrer" target="_blank">https://travis-ci.org/qgis/QGIS/jobs/591915632#L941-L944</a><br>
> > <br>
> > Are the two facts contraddictory ?<br>
> > <br>
> > Reading scripts/<a href="http://sipify.pl" rel="noreferrer noreferrer" target="_blank">sipify.pl</a> it looks like the `-p` switch is<br>
> > not supported, while the `-s` switch is used for two different<br>
> > switches (a typo?):<br>
> > <br>
> >     GetOptions ("debug" => \$debug, "sip_output=s" => \$sip_output,<br>
> >                                                       "python_output=s" => \$python_output) && @ARGV == 1;<br>
> > <br>
> > I mention this because tests/code_layout/test_sipfiles_uptodate.sh<br>
> > uses the -p switch instead:<br>
> > <br>
> >             ./scripts/<a href="http://sipify.pl" rel="noreferrer noreferrer" target="_blank">sipify.pl</a> -p python/${module}/auto_additions/${pyfile}.temp $header |<br>
> >                             diff python/$<a href="http://sipfile.in" rel="noreferrer noreferrer" target="_blank">sipfile.in</a> -<br>
> > <br>
> > So, I suspect <a href="http://sipify.pl" rel="noreferrer noreferrer" target="_blank">sipify.pl</a> is _overriding_ the source file and NEVER producing<br>
> > a .temp file, resulting in:<br>
> > <br>
> >   1) A false OK message from Travis (no difference!)<br>
> >     2) A modification of source tree during test<br>
> > <br>
> > --strk;<br>
</blockquote></div>