diff options
author | Matthias Clasen <mclasen@redhat.com> | 2022-09-09 19:59:02 -0400 |
---|---|---|
committer | Matthias Clasen <mclasen@redhat.com> | 2022-09-09 20:23:44 -0400 |
commit | 7526f77e9ad4f72397bb6b0ed9f98ae90cc53f5a (patch) | |
tree | 3af0e771c0a310b3c5f32918aff48fc920d04bd3 /.gitlab-ci | |
parent | 24572c93c2102d6a8bd6eacbed8c682586b7fb92 (diff) | |
download | glib-7526f77e9ad4f72397bb6b0ed9f98ae90cc53f5a.tar.gz |
ci: Improve style check script
Sync with changes from GTK:
- Set a git config to work around security
improvements in recent git.
- Only fetch the two branches we want to compare
for the style check.
- Use the existing git merge-base command to find
their common ancestor.
Diffstat (limited to '.gitlab-ci')
-rwxr-xr-x | .gitlab-ci/search-common-ancestor.sh | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/.gitlab-ci/search-common-ancestor.sh b/.gitlab-ci/search-common-ancestor.sh index 96f45fe69..914e2e997 100755 --- a/.gitlab-ci/search-common-ancestor.sh +++ b/.gitlab-ci/search-common-ancestor.sh @@ -4,6 +4,9 @@ set -e ancestor_horizon=28 # days (4 weeks) +# Recently, git is picky about directory ownership. Tell it not to worry. +git config --global --add safe.directory "$PWD" + # We need to add a new remote for the upstream target branch, since this script # could be running in a personal fork of the repository which has out of date # branches. @@ -14,7 +17,6 @@ ancestor_horizon=28 # days (4 weeks) if ! git ls-remote --exit-code upstream >/dev/null 2>&1 ; then git remote add upstream https://gitlab.gnome.org/GNOME/glib.git fi -git fetch --shallow-since="$(date --date="${ancestor_horizon} days ago" +%Y-%m-%d)" upstream # 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 @@ -25,9 +27,13 @@ git fetch --shallow-since="$(date --date="${ancestor_horizon} days ago" +%Y-%m-% # otherwise. source_branch="${CI_MERGE_REQUEST_SOURCE_BRANCH_NAME:-${CI_COMMIT_BRANCH}}" +target_branch="${CI_MERGE_REQUEST_TARGET_BRANCH_NAME:-${CI_DEFAULT_BRANCH}}" + git fetch --shallow-since="$(date --date="${ancestor_horizon} days ago" +%Y-%m-%d)" origin "${source_branch}" +git fetch --shallow-since="$(date --date="${ancestor_horizon} days ago" +%Y-%m-%d)" upstream "${target_branch}" + +newest_common_ancestor_sha=$(git merge-base "upstream/${target_branch}" "origin/${source_branch}") -newest_common_ancestor_sha=$(diff --old-line-format='' --new-line-format='' <(git rev-list --first-parent "upstream/${CI_MERGE_REQUEST_TARGET_BRANCH_NAME:-${CI_DEFAULT_BRANCH}}") <(git rev-list --first-parent "origin/${source_branch}") | head -1) if [ -z "${newest_common_ancestor_sha}" ]; then echo "Couldn’t find common ancestor with upstream main branch. This typically" echo "happens if you branched from main a long time ago. Please update" |