Skip to content

Commit 14d3302

Browse files
Merge pull request #442: Fix installer builds
I was working last week to update the `git-update-git-for-windows` script to repeat daily reminders to upgrade. I did not succeed in doing that, but I found some other issues that are resolved in this PR: 1. We can speed up the Windows installer builds by using partial clone of the `git-for-windows/build-extra` repo. 2. There is a `filename` variable that doesn't work in one place. Use the literal path instead. 3. The macOS builds need an explicit version update to compile with FS Monitor due to compatibility issues with older versions of macOS. This required some changes to `derrickstolee/git_osx_installer`, too. These changes were integrated in [a successful run of the `build-git-installers` workflow](https://github.com/microsoft/git/actions/runs/1321667808).
2 parents 7e17ee3 + bb4e5de commit 14d3302

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

.github/workflows/build-git-installers.yml

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ jobs:
7373
- name: Clone build-extra
7474
shell: bash
7575
run: |
76-
git clone --single-branch -b main https://github.com/git-for-windows/build-extra /usr/src/build-extra
76+
git clone --filter=blob:none --single-branch -b main https://github.com/git-for-windows/build-extra /usr/src/build-extra
7777
- name: Clone git
7878
shell: bash
7979
run: |
@@ -170,7 +170,7 @@ jobs:
170170
- name: Clone build-extra
171171
shell: bash
172172
run: |
173-
git clone --single-branch -b main https://github.com/git-for-windows/build-extra /usr/src/build-extra
173+
git clone --filter=blob:none --single-branch -b main https://github.com/git-for-windows/build-extra /usr/src/build-extra
174174
- name: Prepare home directory for code-signing
175175
env:
176176
CODESIGN_P12: ${{secrets.CODESIGN_P12}}
@@ -196,7 +196,7 @@ jobs:
196196
EOF
197197
198198
sed -i -e '/^#include "file-list.iss"/a\
199-
Source: {#SourcePath}\\..\\'$filename'; DestDir: {app}\\mingw64\\bin; Flags: replacesameversion; AfterInstall: DeleteFromVirtualStore' \
199+
Source: {#SourcePath}\\..\\git-update-git-for-windows.config; DestDir: {app}\\mingw64\\bin; Flags: replacesameversion; AfterInstall: DeleteFromVirtualStore' \
200200
-e '/^Type: dirifempty; Name: {app}\\{#MINGW_BITNESS}$/i\
201201
Type: files; Name: {app}\\{#MINGW_BITNESS}\\bin\\git-update-git-for-windows.config\
202202
Type: dirifempty; Name: {app}\\{#MINGW_BITNESS}\\bin' \
@@ -207,9 +207,13 @@ jobs:
207207
set -x
208208
209209
b=/usr/src/build-extra &&
210+
str="recently_seen=.*\$" &&
210211
211-
sed -i -e \
212-
's/recently_seen="\$(get_recently_seen)"/recently_seen="bogus-to-keep-pestering"/' \
212+
echo $b/git-extra/git-update-git-for-windows &&
213+
cat $b/git-extra/git-update-git-for-windows &&
214+
215+
sed -i -e '/^# when confirmation to do so is given./a\
216+
use_recently_seen=no' \
213217
$b/git-extra/git-update-git-for-windows
214218
- name: Set the installer Publisher to the Git Fundamentals team
215219
shell: bash
@@ -309,7 +313,7 @@ jobs:
309313
# Enable a bit stricter compile flags
310314
DEVELOPER: 1
311315
# For the osx-installer build
312-
OSX_VERSION: 10.6
316+
OSX_VERSION: 10.15
313317
V: 1
314318
steps:
315319
- name: Install git dependencies
@@ -366,13 +370,13 @@ jobs:
366370
367371
PATH=/usr/local/bin:/System/Library/Frameworks:$PATH \
368372
make -C git_osx_installer \
369-
OSX_VERSION=10.6 C_INCLUDE_PATH="$C_INCLUDE_PATH" V=1 \
370-
build/intel-universal-snow-leopard/git-$VERSION/osx-built-keychain ||
373+
OSX_VERSION=10.15 C_INCLUDE_PATH="$C_INCLUDE_PATH" V=1 \
374+
build/intel-x86_64-catalina/git-$VERSION/osx-built-keychain ||
371375
die "Build failed"
372376
373377
PATH=/usr/local/bin:$PATH \
374378
make -C git_osx_installer \
375-
OSX_VERSION=10.6 C_INCLUDE_PATH="$C_INCLUDE_PATH" V=1 image ||
379+
OSX_VERSION=10.15 C_INCLUDE_PATH="$C_INCLUDE_PATH" V=1 image ||
376380
die "Build failed"
377381
378382
mkdir -p artifacts

0 commit comments

Comments
 (0)