From ff2962f1f161989a5e08a6913fbb2804b6aa0b5f Mon Sep 17 00:00:00 2001 From: Craig Ringer Date: Wed, 24 May 2023 12:21:50 +1200 Subject: [PATCH] docs: Comments on windows workflows Add comments on the Windows workflows to cross-reference them with each other and the new DEVELOPER_GUIDE.md docs section for Windows builds. Add a short note to the windows PR workflow on how to run it manually on a forked repo when submitting patches. Signed-off-by: Craig Ringer --- .github/workflows/call-build-windows.yaml | 10 ++++++++++ .github/workflows/pr-windows-build.yaml | 13 +++++++++++++ appveyor.yml | 10 ++++++++++ dockerfiles/Dockerfile.windows | 11 +++++++++++ 4 files changed, 44 insertions(+) diff --git a/.github/workflows/call-build-windows.yaml b/.github/workflows/call-build-windows.yaml index 53b62ec6282..1ced2e51bcd 100644 --- a/.github/workflows/call-build-windows.yaml +++ b/.github/workflows/call-build-windows.yaml @@ -1,6 +1,16 @@ --- name: Reusable workflow to build Windows packages optionally into S3 bucket +# +# If you change dependencies etc here, please also check and update +# the other Windows build resources: +# +# - DEVELOPER_GUIDE.md "Windows" section +# - appveyor.yml +# - .github/workflows/call-build-windows.yaml +# - dockerfiles/Dockerfile.windows +# + on: workflow_call: inputs: diff --git a/.github/workflows/pr-windows-build.yaml b/.github/workflows/pr-windows-build.yaml index 2831122fe33..7080471c159 100644 --- a/.github/workflows/pr-windows-build.yaml +++ b/.github/workflows/pr-windows-build.yaml @@ -1,5 +1,18 @@ name: PR - Windows checks + +# +# Test PRs on Windows +# +# This won't run automatically on PRs from untrusted repos, it must be approved +# manually. If PR authors want to run it themselves, they should enable running +# actions on their fork, then invoke it on their branch via their forked repo's +# Actions tab. +# + on: + # Enable invocation via Github repo Actions tab. Having this in the repo + # allows people with github forks to run this job on their own branches to + # build Windows branches conveniently. See DEVELOPER_GUIDE.md. workflow_dispatch: pull_request: diff --git a/appveyor.yml b/appveyor.yml index b13947a70dd..bdf29ece94a 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -1,5 +1,15 @@ version: v1-winbuild-{build} +# +# If you change dependencies etc here, please also check and update +# the other Windows build resources: +# +# - DEVELOPER_GUIDE.md "Windows" section +# - appveyor.yml +# - .github/workflows/call-build-windows.yaml +# - dockerfiles/Dockerfile.windows +# + image: Visual Studio 2019 platform: diff --git a/dockerfiles/Dockerfile.windows b/dockerfiles/Dockerfile.windows index eae4bda3075..1004c8bd27e 100644 --- a/dockerfiles/Dockerfile.windows +++ b/dockerfiles/Dockerfile.windows @@ -1,5 +1,16 @@ # escape=` +# This Dockerfile will only build on Docker for Windows. +# +# If you change dependencies etc here, please also check and update +# the other Windows build resources: +# +# - DEVELOPER_GUIDE.md "Windows" section +# - appveyor.yml +# - .github/workflows/call-build-windows.yaml +# - dockerfiles/Dockerfile.windows +# + ARG WINDOWS_VERSION=ltsc2019 # Builder Image - Windows Server Core