]> granicus.if.org Git - graphviz/commit
CI: remove obsolete exporting of SHELLOPTS = "igncr" in windows-cygwin-cmake-build
authorMagnus Jacobsson <Magnus.Jacobsson@berotec.se>
Wed, 15 Sep 2021 17:10:52 +0000 (19:10 +0200)
committerMagnus Jacobsson <Magnus.Jacobsson@berotec.se>
Sat, 18 Sep 2021 06:16:53 +0000 (08:16 +0200)
commit8d23aa7237e5fe1f2c6bc78910e591fb1326e48c
tree6ee3367e41d4facd19a5704fac071c0f2a465950
parent9aa632b892a28d42c5aaa7abc55fe3dcc6e7de1a
CI: remove obsolete exporting of SHELLOPTS = "igncr" in windows-cygwin-cmake-build

This is no longer necessary since crlf is now replaced with lf in all
text files in the repository before Cygwin starts.

Exporting SHELLOPTS has the negative side effect that other flags such
as -e, -u and -x are inherited by child processes which could cause
invoked bash scripts to fail (because of -u) and abort (because of -e)
unintentionally. While this does not happen in the current Cygwin
CMake build, this has been observed in Cygwin autotools builds that
will be introduced in an upcoming MR. This commit ensures that this
can not occur in the future for CMake builds either.

The -x option is more or less harmless, but causes excessive logging
in invoked scripts. This can be seen in
e.g. https://gitlab.com/graphviz/graphviz/-/jobs/1593005342.
.gitlab-ci.yml