Arch Linux User Repository

@HurricanePootis thanks for the feedback.

Given that it reportedly works for you and @kinker31, I looked back into it, and it turns out there was an issue on my end. I had an older /etc/makepkg.conf file. After merging it with the /etc/makepkg.conf.pacnew file, vkquake now builds without errors, no patch needed.

Diffing the two files, the default CFLAGS do look quite different. I can only assume that the error was caused by one of them.

I can still build vkquake with gcc 13 without experiencing those builds

VKQuake builds, installs, and plays without any major issue on my end, though for some reason the program doesn't save the option whever to enable smooth animations/movements or not, and defaults to "Smooth", which I personally find undesirable.

@HurricanePootis the build currently fails with GCC13.

The patch that @spillner provided below was apparently taken from upstream https://github.com/Novum/vkQuake/commit/19d76588037977f59506c18aa3d307f8f5124d0f

Related upstream issue can be found here: https://github.com/Novum/vkQuake/issues/688

@zed123 @spillner, I was able to build vkquake without the patch in a clean chroot. Please provide more information on this patch and where it comes from before expecting me to take it in.

Patch to build with GCC 13:

diff -Naur orig/PKGBUILD new/PKGBUILD
--- orig/PKGBUILD       2023-09-04 08:10:25.101533679 +1000
+++ new/PKGBUILD        2023-09-04 08:03:48.411677421 +1000
@@ -17,15 +17,21 @@
 makedepends=('vulkan-validation-layers' 'meson' 'vulkan-headers' 'glslang' 'spirv-tools')
 install=$pkgname.install
 source=("https://github.com/Novum/vkQuake/archive/${pkgver}.tar.gz"
+       'vkquake-gcc13.patch'
        'vkquake.desktop'
        'vkquake.png'
        'vkquake.svg')
 sha512sums=('e29a469e6c923e899e142860e27e343eafd54f02dfb5842f081aadb077f97187d5aa092dcb98352b76f13dbc18d88a3afabf2e21799a305fe536c35fb6d68bbd'
+           '23d12f3695d286528e8d84c1473721b9c2e382975db87cf4137b8cc7c77d34b54ca4df9156f5c120812b43ce2a512a59c8709c4b41b33d626a783a15b8c30e30'
         '75881c2fea0628dcecff496a5e3035bbb4e408b81527737339ebfd66ad04ca04621d4e13e3e0537f091f8b1127dc231583f746d11cc6c11658cdb976db77e145'
         '9ac17fb2f549656869a02a9a1896786aa4a4f753c6b1174d14b999c94693164ec71a73f9ab6f4a96591ef3af407ffce4f29b0ed87e878f14bb177e396898f29f'
         '5a712a6c0447acea7a6c690cb7e63a2111e28646ef0935dfbd40cc4164c9ed5fc6fdba6eeafe44b618b17e856b3b0f60cc3ddbdedd8e536feef97de969d407b0')


+prepare () {
+  patch -p1 --directory="$_quake-$pkgver" --input="${srcdir}/vkquake-gcc13.patch"
+}
+
 build() {

   cd "$srcdir/$_quake-$pkgver"
diff -Naur orig/vkquake-gcc13.patch new/vkquake-gcc13.patch
--- orig/vkquake-gcc13.patch    1970-01-01 10:00:00.000000000 +1000
+++ new/vkquake-gcc13.patch     2023-09-04 07:55:05.374019208 +1000
@@ -0,0 +1,25 @@
+diff --git a/Quake/gl_heap.c b/Quake/gl_heap.c
+index d2a0ccfa..db55ad0b 100644
+--- a/Quake/gl_heap.c
++++ b/Quake/gl_heap.c
+@@ -645,7 +645,7 @@ glheapallocation_t *GL_HeapAllocate (glheap_t *heap, VkDeviceSize size, VkDevice
+ 
+       if (size < heap->segment_size)
+       {
+-              allocinfo_t                alloc_info;
++              ZEROED_STRUCT (allocinfo_t, alloc_info);
+               const VkDeviceSize size_alignment_max = q_max (size, alignment);
+               alloc_info.is_small_alloc = size_alignment_max <= heap->page_size / 2;
+               if (alloc_info.is_small_alloc)
+diff --git a/Quake/sv_phys.c b/Quake/sv_phys.c
+index 3f57f53e..f214aea2 100644
+--- a/Quake/sv_phys.c
++++ b/Quake/sv_phys.c
+@@ -276,6 +276,7 @@ int SV_FlyMove (edict_t *ent, float time, trace_t *steptrace)
+       blocked = 0;
+       VectorCopy (ent->v.velocity, original_velocity);
+       VectorCopy (ent->v.velocity, primal_velocity);
++      VectorCopy (ent->v.velocity, new_velocity);
+       numplanes = 0;
+ 
+       time_left = time;

This version does not allow mod switching within the UI. Only the appimage supports this ATM.

Vkquake.pak is no longer needed. vkquake.pak is embedded in the executable. https://github.com/Novum/vkQuake/releases/tag/1.22.0

You can remove the creation and install of vkquake.pak

I think zopfli can then be removed as a makedepends.

I compiled it without vkquake.pak and zopfli and it ran fine.

@hatred, I can compile vkquake just fine the way it is with the current link