<p><b>@b4n</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2067#discussion_r281847855">.travis.yml</a>:</p>
<pre style='color:#555'>> @@ -37,7 +112,10 @@ script:
       mkdir _build                        &&
       cd _build                           &&
       { ../configure $CONFIGURE_FLAGS || { cat config.log; exit 1; } ; } &&
-      make -j2                            &&
-      make -j2 check                      &&
-      make -j2 distcheck DISTCHECK_CONFIGURE_FLAGS="$CONFIGURE_FLAGS";
+      if [ -z "$DISTCHECK_ONLY" ]; then
+        make -j2 VERBOSE=1 -n                           &&
+        make -j2 check;
+      else
+        make -j2 distcheck DISTCHECK_CONFIGURE_FLAGS="$CONFIGURE_FLAGS";
+      fi;
</pre>
<p>I don't like that, because what you do is build one more time, and only that one do the distcheck.  And given how distcheck works (make the tarball and try to build it), it could possibly be affected by the configuration options selecting buildig or not some code (even though it should not if we didn't mess up, but CI is to catch case where we did, isn't it <g-emoji class="g-emoji" alias="smile" fallback-src="https://github.githubassets.com/images/icons/emoji/unicode/1f604.png">😄</g-emoji> ).  So it is a good thing it's checked for all variations of the environment.</p>
<p>Only the same environment with different compilers is not useful, because building with another compiler shouldn't affect <em>what</em> we're building.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2067#pullrequestreview-234785015">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ3TGNE3WOW2VC4L6QDPUH4Q7ANCNFSM4GSODDDQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ6AZTJSO3MOTZBDNXLPUH4Q7A5CNFSM4GSODDD2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBX7IR5Y.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2067#pullrequestreview-234785015",
"url": "https://github.com/geany/geany/pull/2067#pullrequestreview-234785015",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>