[pkgsrc/trunk]: pkgsrc/math/fftw math/fftw: fix pkglint issues

[thor <thor%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/fc0c28e17985
branches:  trunk
changeset: 452240:fc0c28e17985
user:      thor <thor%pkgsrc.org@localhost>
date:      Fri May 07 12:31:22 2021 +0000

description:
math/fftw: fix pkglint issues

diffstat:

 math/fftw/Makefile   |  27 ++++++++++++---------------
 math/fftw/PLIST      |  30 +++++++++++++++---------------
 math/fftw/options.mk |  18 +++++++++---------
 3 files changed, 36 insertions(+), 39 deletions(-)

diffs (171 lines):

diff -r 4cbba84892e6 -r fc0c28e17985 math/fftw/Makefile
--- a/math/fftw/Makefile        Fri May 07 12:31:04 2021 +0000
+++ b/math/fftw/Makefile        Fri May 07 12:31:22 2021 +0000
@@ -1,19 +1,17 @@
-# $NetBSD: Makefile,v 1.67 2021/05/07 11:59:41 thor Exp $
-
-PKGREVISION=   5
-
-COMMENT=       Collection of fast C routines to compute DFTs
+# $NetBSD: Makefile,v 1.68 2021/05/07 12:31:22 thor Exp $
 
 DISTNAME=      fftw-3.3.9
+PKGREVISION=   5
 CATEGORIES=    math
 MASTER_SITES=  ftp://ftp.fftw.org/pub/fftw/
 MASTER_SITES+= http://www.fftw.org/
 
 MAINTAINER=    pkgsrc-users%NetBSD.org@localhost
 HOMEPAGE=      http://www.fftw.org/
+COMMENT=       Collection of fast C routines to compute DFTs
 LICENSE=       gnu-gpl-v2
 
-SUPERSEDES=    fftwf
+SUPERSEDES=    fftwf-[0-9]*
 CONFLICTS+=    fftwf-[0-9]*
 
 USE_LANGUAGES=         c
@@ -64,10 +62,10 @@
 FFTW_FLOAT_OPTS+=      --enable-sse --enable-sse2 --enable-avx
 FFTW_DOUBLE_OPTS+=     --enable-sse2 --enable-avx
 # From gcc-4.9 on, these should work trouble-free.
-.if empty(CC_VERSION:Mgcc-4.8.*)
+.  if empty(CC_VERSION:Mgcc-4.8.*)
 FFTW_FLOAT_OPTS+=      --enable-avx2 --enable-avx512 --enable-avx-128-fma
-FFTW_DOUBLE_OPTS+=     --enable-avx2 --enable-avx512 --enable-avx-128-fma 
-.endif
+FFTW_DOUBLE_OPTS+=     --enable-avx2 --enable-avx512 --enable-avx-128-fma
+.  endif
 .endif
 
 # TODO: Add the above for other architectures.
@@ -82,21 +80,20 @@
 pre-configure:
 .for d in ${CONFIGURE_DIRS}
        mkdir -p ${WRKSRC:Q}/${d}
-.if   ${d} == "build/float"
+.  if   ${d} == "build/float"
        printf '#!/bin/sh\nexec ../../configure "$$@" --enable-${d:S,build/,,} ${FFTW_FLOAT_OPTS}' \
        > ${WRKSRC:Q}/${d}/configure
-.elif ${d} == "build/double"
+.  elif ${d} == "build/double"
        printf '#!/bin/sh\nexec ../../configure "$$@" ${FFTW_DOUBLE_OPTS}' \
        > ${WRKSRC:Q}/${d}/configure
-.elif ${d} == "build/quad-precision"
+.  elif ${d} == "build/quad-precision"
        printf '#!/bin/sh\nexec ../../configure "$$@" --enable-${d:S,build/,,} --disable-mpi\n' \
        > ${WRKSRC:Q}/${d}/configure
-.else
+.  else
        printf '#!/bin/sh\nexec ../../configure "$$@" --enable-${d:S,build/,,}\n' \
        > ${WRKSRC:Q}/${d}/configure
-.endif
+.  endif
        chmod +x ${WRKSRC:Q}/${d}/configure
 .endfor
 
 .include "../../mk/bsd.pkg.mk"
-
diff -r 4cbba84892e6 -r fc0c28e17985 math/fftw/PLIST
--- a/math/fftw/PLIST   Fri May 07 12:31:04 2021 +0000
+++ b/math/fftw/PLIST   Fri May 07 12:31:22 2021 +0000
@@ -1,13 +1,15 @@
-@comment $NetBSD: PLIST,v 1.14 2021/04/29 23:30:15 thor Exp $
+@comment $NetBSD: PLIST,v 1.15 2021/05/07 12:31:22 thor Exp $
+bin/fftw-wisdom
 bin/fftw-wisdom-to-conf
-bin/fftw-wisdom
 bin/fftwf-wisdom
 ${PLIST.long}bin/fftwl-wisdom
 ${PLIST.quad}bin/fftwq-wisdom
+${PLIST.mpi}include/fftw3-mpi.f03
+${PLIST.mpi}include/fftw3-mpi.h
 include/fftw3.f
 include/fftw3.f03
 include/fftw3.h
-${PLIST.mpi}include/fftw3-mpi.h
+${PLIST.mpi}include/fftw3l-mpi.f03
 include/fftw3l.f03
 include/fftw3q.f03
 info/fftw3.info
@@ -20,22 +22,20 @@
 ${PLIST.quad}lib/cmake/fftw3/FFTW3qConfig.cmake
 ${PLIST.quad}lib/cmake/fftw3/FFTW3qConfigVersion.cmake
 lib/libfftw3.la
-lib/libfftw3f.la
-${PLIST.long}lib/libfftw3l.la
-${PLIST.quad}lib/libfftw3q.la
-${PLIST.pthreads}lib/libfftw3_threads.la
-${PLIST.pthreads}lib/libfftw3f_threads.la
-${PLIST.pthreads}${PLIST.long}lib/libfftw3l_threads.la
-${PLIST.pthreads}${PLIST.quad}lib/libfftw3q_threads.la
+${PLIST.mpi}lib/libfftw3_mpi.la
 ${PLIST.omp}lib/libfftw3_omp.la
+${PLIST.pthreads}lib/libfftw3_threads.la
+lib/libfftw3f.la
+${PLIST.mpi}lib/libfftw3f_mpi.la
 ${PLIST.omp}lib/libfftw3f_omp.la
+${PLIST.pthreads}lib/libfftw3f_threads.la
+${PLIST.long}lib/libfftw3l.la
+${PLIST.mpi}${PLIST.long}lib/libfftw3l_mpi.la
 ${PLIST.omp}${PLIST.long}lib/libfftw3l_omp.la
+${PLIST.pthreads}${PLIST.long}lib/libfftw3l_threads.la
+${PLIST.quad}lib/libfftw3q.la
 ${PLIST.omp}${PLIST.quad}lib/libfftw3q_omp.la
-${PLIST.mpi}lib/libfftw3_mpi.la
-${PLIST.mpi}lib/libfftw3f_mpi.la
-${PLIST.mpi}${PLIST.long}lib/libfftw3l_mpi.la
-${PLIST.mpi}include/fftw3l-mpi.f03
-${PLIST.mpi}include/fftw3-mpi.f03
+${PLIST.pthreads}${PLIST.quad}lib/libfftw3q_threads.la
 lib/pkgconfig/fftw3.pc
 lib/pkgconfig/fftw3f.pc
 ${PLIST.long}lib/pkgconfig/fftw3l.pc
diff -r 4cbba84892e6 -r fc0c28e17985 math/fftw/options.mk
--- a/math/fftw/options.mk      Fri May 07 12:31:04 2021 +0000
+++ b/math/fftw/options.mk      Fri May 07 12:31:22 2021 +0000
@@ -1,14 +1,14 @@
-# $NetBSD: options.mk,v 1.4 2021/04/29 23:30:15 thor Exp $
+# $NetBSD: options.mk,v 1.5 2021/05/07 12:31:22 thor Exp $
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.fftw
 # fftw (double) and fftwf (single) are always built, you can add
 # fftwl (long) and fftwq (quad).
 PKG_SUPPORTED_OPTIONS= fftw-fortran openmp mpi fftw-long fftw-quad
 # Enable fortran support by default on platforms supported by lang/g95.
-.if (!empty(MACHINE_ARCH:Mi386) || !empty(MACHINE_ARCH:Mx86_64) || \
-       !empty(MACHINE_ARCH:Mia64) || !empty(MACHINE_ARCH:Mpowerpc*) || \
-       !empty(MACHINE_ARCH:Mhppa) || !empty(MACHINE_ARCH:Msparc*) || \
-       !empty(MACHINE_ARCH:Malpha) || !empty(MACHINE_ARCH:Mmips*))
+.if (${MACHINE_ARCH} == i386 || ${MACHINE_ARCH} == x86_64 || \
+       ${MACHINE_ARCH} == ia64 || !empty(MACHINE_ARCH:Mpowerpc*) || \
+       ${MACHINE_ARCH} == hppa || !empty(MACHINE_ARCH:Msparc*) || \
+       ${MACHINE_ARCH} == alpha || !empty(MACHINE_ARCH:Mmips*))
 # ...but disable it until lang/g95 issue is resolved.
 #PKG_SUGGESTED_OPTIONS=        fftw-fortran
 .endif
@@ -21,9 +21,9 @@
 CONFIGURE_ARGS+=       --disable-fortran
 .endif
 
-PLIST_VARS+=   omp
+PLIST_VARS+=           omp
 .if !empty(PKG_OPTIONS:Mopenmp)
-PLIST.omp=     yes
+PLIST.omp=             yes
 CONFIGURE_ARGS+=       --enable-openmp
 .endif
 
@@ -38,10 +38,10 @@
 
 .if !empty(PKG_OPTIONS:Mfftw-long)
 FFTW_PRECISION+=       long-double
-PLIST.long=    yes
+PLIST.long=            yes
 .endif
 
 .if !empty(PKG_OPTIONS:Mfftw-quad)
 FFTW_PRECISION+=       quad-precision
-PLIST.quad=    yes
+PLIST.quad=            yes
 .endif