Arch Linux User Repository

Hi, this is harmonization with wiki cmake template https://wiki.archlinux.org/title/CMake_package_guidelines#Template and fix for systems where the default cmake build system is something else than GNU make (e.g.: ninja)

diff --git a/PKGBUILD b/PKGBUILD
index 9c1d80d..e0a0b06 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -1,7 +1,8 @@
 # Maintainer: lain <aur@hacktheinter.net>
 pkgname=ffts-git
-pkgver=r799.fe86885
-pkgrel=3
+_pkgname=ffts
+pkgver=r800.b22d839b61c
+pkgrel=1
 pkgdesc="The Fastest Fourier Transform in the South"
 provides=("${pkgname%-git}")
 conflicts=("${pkgname%-git}")
@@ -20,12 +21,12 @@ md5sums=(
 )

 pkgver() {
-  cd ffts
+  cd "$srcdir/$_pkgname"
   printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
 }

 prepare() {
-  cd ffts
+  cd "$srcdir/$_pkgname"
   # Patch to fix compilation on non-SSE2 machines
   # Source: https://github.com/anthonix/ffts/pull/78
   # (has no effect on SSE2 machines, so we can just always apply the patch)
@@ -33,15 +34,15 @@ prepare() {
 }

 build() {
-  mkdir -p ffts/build
-  cd ffts/build
-  cmake -DENABLE_SHARED=ON -DCMAKE_INSTALL_PREFIX=/usr ..
-  make
+  cmake -B build -S "$srcdir/$_pkgname" \
+    -DENABLE_SHARED=ON \
+    -DCMAKE_INSTALL_PREFIX=/usr \
+    -Wno-dev
+  cmake --build build
 }

 package() {
-  cd ffts
+  DESTDIR="$pkgdir" cmake --install build
+  cd "$srcdir/$_pkgname"
   install -Dm644 COPYRIGHT "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
-  cd build
-  make DESTDIR="$pkgdir" install
 }

Building this version (r799.fe86885-3) doesn't update the pkgver/pkgrel but installs it as the old version (r799.fe86885ecaf-1), resulting in the package reappearing again and again as available update.

A quick fix to this is to delete the entire pkgver () function in the PKGBUILD (line 22 - 25).

@xyproto Thanks, both are now fixed :)

This patch was needed in order to build ffts-git on Arch Linux ARM (without SSE2):

https://github.com/anthonix/ffts/pull/78

The dependency on cmake seems to be missing.

@kbeckmann done!

It would be nice if this package built a shared library as well. Could you please add -DENABLE_SHARED=ON to the cmake arguments?