Skip to content

Commit 0fff9f8

Browse files
authored
Prepare some more for the split mingw-w64-git package (#663)
As part of my effort to upstream the `mingw-w64-git` package definition to MSYS2, I had to change the default to _omit_ building `mingw-w64-git-pdb`. Before marking [the upstreaming PR](msys2/MINGW-packages#26470) as ready for review, I want to ensure that Git for Windows already has the identical changes (see git-for-windows/MINGW-packages#174). This requires two changes in `build-extra` to accommodate for the now-inverted logic.
2 parents 97c7768 + 9986fc6 commit 0fff9f8

3 files changed

Lines changed: 13 additions & 7 deletions

File tree

git-extra/PKGBUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ pkgbase="mingw-w64-${_realname}"
55
pkgname=($_realname
66
"${MINGW_PACKAGE_PREFIX}-${_realname}")
77
_ver_base=1.1
8-
pkgver=1.1.683.0c63e840c
8+
pkgver=1.1.685.a08718cdd
99
pkgrel=1
1010
pkgdesc="Git for Windows extra files"
1111
arch=('any')
@@ -82,7 +82,7 @@ sha256sums=('8ed76d1cb069ac8568f21c431f5e23caebea502d932ab4cdff71396f4f0d5b72'
8282
'683ab066be19cb4defec470ebd53f165ca5dbf761fd40c13aee8abe31ba42803'
8383
'1cf2c13fb97c51375a76ed479362c3cbcdb51ab4d3a745e8d2c3a780badd8d46'
8484
'32223da4e1264d0c9663dd3d4b347131a634e96c2676d673ca58b2a6c58d2aea'
85-
'2cd753e134a9050037a5cbada81d91af6ec9c85e9da696ae908a3633f2c5340a'
85+
'1fae4ff3ce1e75ac90fe6f6130db4d041bc34f59deea679501f57d2c41d37560'
8686
'22f41610dea842890955032af30efdb60e80f310e95a04e57ab36b10e0376923'
8787
'8120b41b869693e07acc4707a0ab45505e7c5e4156a3a44b19152f1f425df85f'
8888
'15b40ab72dea884f659cfbe441e9a40b2d8d63e490a3c14824a55607368e476d'

git-extra/git-sdk.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,7 @@ sdk () {
154154
# for building
155155
makepkg|makepkg-mingw)
156156
cmd=$1; shift
157+
WITH_PDBS="$(! grep -q WITH_PDBS PKGBUILD || ! sdk find_mspdb_dll || echo true)" \
157158
WITHOUT_PDBS="$(! grep -q WITHOUT_PDBS PKGBUILD || sdk find_mspdb_dll || echo true)" \
158159
MAKEFLAGS=${MAKEFLAGS:--j$(nproc)} PKGEXT='.pkg.tar.xz' $cmd --syncdeps --noconfirm --skipchecksums --skippgpcheck "$@"
159160
;;

please.sh

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3551,13 +3551,18 @@ build_mingw_w64_git () { # [--only-i686] [--only-x86_64] [--only-aarch64] [--ski
35513551
# Work around bug where the incorrect xmlcatalog.exe wrote /etc/xml/catalog
35523552
sed -i -e 's|C:/git-sdk-64-ci||g' /etc/xml/catalog
35533553
3554-
test true = "$GITHUB_ACTIONS" || # GitHub Actions' agents have the mspdb.dll, and cv2pdb finds it
3555-
test -n "$SYSTEM_COLLECTIONURI$SYSTEM_TASKDEFINITIONSURI" || # Same for Azure Pipelines
3556-
test "$MINGW_ARCH" = "clangarm64" || # We don't need cv2pdb when compiling using Clang/LLVM
3557-
find_mspdb_dll >/dev/null || {
3554+
test true != "$GITHUB_ACTIONS" && # GitHub Actions' agents have the mspdb.dll, and cv2pdb finds it
3555+
test -z "$SYSTEM_COLLECTIONURI$SYSTEM_TASKDEFINITIONSURI" && # Same for Azure Pipelines
3556+
test "$MINGW_ARCH" != "clangarm64" && # We don't need cv2pdb when compiling using Clang/LLVM
3557+
! find_mspdb_dll >/dev/null && {
3558+
WITH_PDBS=
35583559
WITHOUT_PDBS=1
3559-
export WITHOUT_PDBS
3560+
} || {
3561+
WITH_PDBS=1
3562+
WITHOUT_PDBS=
35603563
}
3564+
export WITH_PDBS
3565+
export WITHOUT_PDBS
35613566
35623567
(if test -n "$(git config alias.signtool)"
35633568
then

0 commit comments

Comments
 (0)