[glib: 2/3] ci: Move style-check-diff commands out to a separate sh file



commit 805dc2ae937e3116d20e00d1a257565472eee5ef
Author: Philip Withnall <withnall endlessm com>
Date:   Thu Nov 28 11:44:24 2019 +0000

    ci: Move style-check-diff commands out to a separate sh file
    
    This doesn’t change how they run, but does split the code out a bit and
    mean we can interleave it with comments. Should make it a little less
    vile.
    
    Suggested by Emmanuele Bassi; see !1252.
    
    Signed-off-by: Philip Withnall <withnall endlessm com>

 .gitlab-ci.yml                     |  4 +---
 .gitlab-ci/run-style-check-diff.sh | 16 ++++++++++++++++
 2 files changed, 17 insertions(+), 3 deletions(-)
---
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 56a12e92b..ba75de770 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -26,9 +26,7 @@ style-check-diff:
   stage: style-check
   allow_failure: true
   script:
-    - git remote add upstream https://gitlab.gnome.org/GNOME/glib.git
-    - git fetch upstream
-    - git diff -U0 --no-color $(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 HEAD) | 
head -1) | ./clang-format-diff.py -binary "clang-format-7" -p1
+    - .gitlab-ci/run-style-check-diff.sh
 
 fedora-x86_64:
   extends: .only-default
diff --git a/.gitlab-ci/run-style-check-diff.sh b/.gitlab-ci/run-style-check-diff.sh
new file mode 100755
index 000000000..4ed6284b6
--- /dev/null
+++ b/.gitlab-ci/run-style-check-diff.sh
@@ -0,0 +1,16 @@
+#!/bin/bash
+
+set +e
+
+# 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.
+git remote add upstream https://gitlab.gnome.org/GNOME/glib.git
+git fetch 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
+# `upstream/master` or `upstream/glib-2-62`).
+# `${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.
+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 HEAD) | 
head -1)
+git diff -U0 --no-color "${newest_common_ancestor_sha}" | ./clang-format-diff.py -binary "clang-format-7" -p1


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]