-
Notifications
You must be signed in to change notification settings - Fork 626
Start using new git-extra prefix #461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Start using new git-extra prefix #461
Conversation
Test results of I tested this locally by basically mimicking sync.yml:
The resulting commit looks good to me: $ git show 64498be630d1dc0003beefa306cdb28302f47d28
commit 64498be630d1dc0003beefa306cdb28302f47d28 (HEAD -> main)
Author: Dennis Ameling <dennis@dennisameling.com>
Date: Tue Dec 27 21:54:32 2022 +0100
Update 1 package
git-extra (1.1.616.3f26b1b0c-1 -> 1.1.616.3f26b1b0c-1)
Signed-off-by: Dennis Ameling <dennis@dennisameling.com>
diff --git a/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/mtree b/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/mtree
deleted file mode 100644
index 9fb27af6af..0000000000
Binary files a/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/mtree and /dev/null differ
diff --git a/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/desc b/var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/desc
similarity index 93%
rename from var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/desc
rename to var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/desc
index 9379cd8aea..f84650410c 100644
--- a/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/desc
+++ b/var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/desc
@@ -1,5 +1,5 @@
%NAME%
-git-extra
+mingw-w64-x86_64-git-extra
%VERSION%
1.1.616.3f26b1b0c-1
@@ -14,13 +14,13 @@ Git for Windows extra files
https://github.com/git-for-windows/build-extra
%ARCH%
-x86_64
+any
%BUILDDATE%
1671657907
%INSTALLDATE%
-1672110153
+1672174026
%PACKAGER%
Johannes Schindelin <johannes.schindelin@gmx.de>
@@ -46,7 +46,10 @@ vim
filesystem
%CONFLICTS%
+git-extra
mingw-w64-clang-aarch64-git-extra
-mingw-w64-x86_64-git-extra
mingw-w64-i686-git-extra
+%PROVIDES%
+git-extra
+
diff --git a/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/files b/var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/files
similarity index 100%
rename from var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/files
rename to var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/files
diff --git a/var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/install b/var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/install
similarity index 100%
rename from var/lib/pacman/local/git-extra-1.1.616.3f26b1b0c-1/install
rename to var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/install
diff --git a/var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/mtree b/var/lib/pacman/local/mingw-w64-x86_64-git-extra-1.1.616.3f26b1b0c-1/mtree
new file mode 100644
index 0000000000..c7d0d7279f |
Test results of When the package was built, the appropriate part of
Here's the artifact in case you're interested: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will also have to take care of git-sdk-arm64
's sync
job, but that can only happen once we register the aarch64 Pacman repository.
git-extra/PKGBUILD
Outdated
mkdir -p ${TMPDIR} | ||
sed -e "s|@@PACKAGE_NAME@@|${MINGW_PACKAGE_PREFIX}-${_realname}|g" \ | ||
<update-via-pacman.bat >"$TMPDIR"/update-via-pacman.bat || | ||
(echo "Could not generate update-via-pacman script" && exit 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The canonical way to do this is to have a update-via-pacman.bat.in
script and generate the update-via-pacman.bat
version from it. Also, could we use GIT_EXTRA_PACKAGE_NAME
instead of the less-descriptive PACKAGE_NAME
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense - in 6746dc5, I went with an approach to copy update-via-pacman.bat
into the build folder instead of the startdir
, as its contents will be different for every architecture. So we'll only have an update-via-pacman.bat.in
in the startdir
moving forward. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So we'll only have an
update-via-pacman.bat.in
in thestartdir
moving forward. WDYT?
@dennisameling sounds good! TBH I hope that we can soon replace update-via-pacman.bat
with the .ps1
script from git-for-windows/git-sdk-arm64#4 (meaning: we can simply delete it from git-extra
, as the script will live directly in the respective git-sdk-*
repository).
a92fa63
to
76f0782
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonderful!
In a recent commit, we started publishing the mingw-w64-git-extra package in addition to git-extra. We intend to completely remove the old git-extra package over time, but let's start using the new mingw-w64-git-extra prefix now. More background on this change can be found in the attached commit. Ref: 3f26b1b (git-extra: publish both with and without MINGW prefix, 2022-12-21) Signed-off-by: Dennis Ameling <dennis@dennisameling.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
In a recent commit, we started publishing the mingw-w64-git-extra package in addition to git-extra. We intend to completely remove the old git-extra package over time, but let's start using the new mingw-w64-git-extra prefix now. More background on this change can be found in the attached commit. Ref: 3f26b1b (git-extra: publish both with and without MINGW prefix, 2022-12-21) Signed-off-by: Dennis Ameling <dennis@dennisameling.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
In a recent commit, we started publishing the mingw-w64-git-extra package in addition to git-extra. We intend to completely remove the old git-extra package over time, but let's start using the new mingw-w64-git-extra prefix now. More background on this change can be found in the attached commit. Ref: 3f26b1b (git-extra: publish both with and without MINGW prefix, 2022-12-21) Signed-off-by: Dennis Ameling <dennis@dennisameling.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
76f0782
to
24bdb1a
Compare
Signed-off-by: Dennis Ameling <dennis@dennisameling.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
24bdb1a
to
bdc6d7d
Compare
/deploy git-extra The i686/x86_64 and the arm64 workflow runs were started. |
This depends on #457 being merged first, as it removes a lot of code that we'd otherwise have to update.
I used the list of files that need updating from this comment (thanks @dscho!)