2020-02-11 13:28:16 +00:00
|
|
|
|
#!/bin/bash
|
|
|
|
|
|
2020-02-25 17:13:39 +00:00
|
|
|
|
set -e
|
2020-02-11 13:28:16 +00:00
|
|
|
|
|
|
|
|
|
# We need to add a new remote for the upstream master, since this script could
|
|
|
|
|
# be running in a personal fork of the repository which has out of date branches.
|
2020-06-28 13:01:19 +00:00
|
|
|
|
if [ "${CI_PROJECT_NAMESPACE}" != "GNOME" ]; then
|
|
|
|
|
echo "Retrieving the current upstream repository from ${CI_PROJECT_NAMESPACE}/${CI_PROJECT_NAME}..."
|
|
|
|
|
git remote add upstream https://gitlab.gnome.org/GNOME/gtk.git
|
|
|
|
|
git fetch upstream
|
|
|
|
|
ORIGIN="upstream"
|
|
|
|
|
else
|
|
|
|
|
echo "Reusing the existing repository on ${CI_PROJECT_NAMESPACE}/${CI_PROJECT_NAME}"
|
|
|
|
|
ORIGIN="origin"
|
|
|
|
|
fi
|
2020-02-11 13:28:16 +00:00
|
|
|
|
|
|
|
|
|
# Work out the newest common ancestor between the detached HEAD that this CI job
|
|
|
|
|
# has checked out, and the upstream target branch (which will typically be
|
|
|
|
|
# `upstream/master` or `upstream/gtk-3-24`).
|
|
|
|
|
#
|
|
|
|
|
# `${CI_MERGE_REQUEST_TARGET_BRANCH_NAME}` is only defined if we’re running in
|
|
|
|
|
# a merge request pipeline; fall back to `${CI_DEFAULT_BRANCH}` otherwise.
|
2020-06-28 13:01:19 +00:00
|
|
|
|
newest_common_ancestor_sha=$(diff --old-line-format='' --new-line-format='' <(git rev-list --first-parent "${ORIGIN}/${CI_MERGE_REQUEST_TARGET_BRANCH_NAME:-${CI_DEFAULT_BRANCH}}") <(git rev-list --first-parent HEAD) | head -1)
|
2020-02-11 13:28:16 +00:00
|
|
|
|
git diff -U0 --no-color "${newest_common_ancestor_sha}" | .gitlab-ci/clang-format-diff.py -binary "clang-format" -p1
|
2020-02-25 17:13:39 +00:00
|
|
|
|
exit_status=$?
|
2020-02-11 13:28:16 +00:00
|
|
|
|
|
|
|
|
|
# The style check is not infallible. The clang-format configuration cannot
|
|
|
|
|
# perfectly describe GTK’s coding style: in particular, it cannot align
|
|
|
|
|
# function arguments. The documented coding style for GTK takes priority over
|
|
|
|
|
# clang-format suggestions. Hopefully we can eventually improve clang-format to
|
|
|
|
|
# be configurable enough for our coding style. That’s why this CI check is OK
|
|
|
|
|
# to fail: the idea is that people can look through the output and ignore it if
|
|
|
|
|
# it’s wrong. (That situation can also happen if someone touches pre-existing
|
|
|
|
|
# badly formatted code and it doesn’t make sense to tidy up the wider coding
|
|
|
|
|
# style with the changes they’re making.)
|
|
|
|
|
echo ""
|
|
|
|
|
echo "Note that clang-format output is advisory and cannot always match the"
|
|
|
|
|
echo "GTK coding style, documented at:"
|
|
|
|
|
echo " https://gitlab.gnome.org/GNOME/gtk/blob/master/docs/CODING-STYLE"
|
|
|
|
|
echo "Warnings from this tool can be ignored in favour of the documented "
|
|
|
|
|
echo "coding style, or in favour of matching the style of existing"
|
|
|
|
|
echo "surrounding code."
|
2020-02-25 17:13:39 +00:00
|
|
|
|
|
|
|
|
|
exit ${exit_status}
|