From: Jason Ish Date: Wed, 28 Jun 2023 18:32:35 +0000 (-0600) Subject: github-ci: allow pull request URL in SV_BRANCH X-Git-Tag: suricata-6.0.14~80 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F9129%2Fhead;p=thirdparty%2Fsuricata.git github-ci: allow pull request URL in SV_BRANCH Allow the SV_BRANCH variable to contain the full link to an OISF/suricata-verify pull request. This will cause GitHub to create a cross-link for us. (cherry picked from commit f870dcd4cc1c72cdf5faf790a1582347ed87ab9f) --- diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index c6a815c56d..447f9121e0 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -13,7 +13,11 @@ Describe changes: ### Provide values to any of the below to override the defaults. -To use a pull request use a branch name like `pr/N` where `N` is the pull request number. +To use a pull request use a branch name like `pr/N` where `N` is the +pull request number. + +Alternatively, `SV_BRANCH` may also be a link to an +OISF/suricata-verify pull-request. ``` SV_REPO= diff --git a/.github/workflows/builds.yml b/.github/workflows/builds.yml index 3b35a3ca86..8e1dc09db5 100644 --- a/.github/workflows/builds.yml +++ b/.github/workflows/builds.yml @@ -90,7 +90,13 @@ jobs: - name: Fetching suricata-verify run: | - pr=$(echo "${SV_BRANCH}" | sed -n 's/^pr\/\([[:digit:]]\+\)$/\1/p') + # Looking for a pull request number. in the SV_BRANCH + # value. This could be "pr/NNN", "pull/NNN" or a link to an + # OISF/suricata-verify pull request. + pr=$(echo "${SV_BRANCH}" | sed -n \ + -e 's/^https:\/\/github.com\/OISF\/suricata-verify\/pull\/\([0-9]*\)$/\1/p' \ + -e 's/^pull\/\([0-9]*\)$/\1/p' \ + -e 's/^pr\/\([0-9]*\)$/\1/p') if [ "${pr}" ]; then SV_BRANCH="refs/pull/${pr}/head" echo "Using suricata-verify pull-request ${SV_BRANCH}"