Package Details: aptdec-git r212.4d4a0c9-1

Git Clone URL: https://aur.archlinux.org/aptdec-git.git (read-only, click to copy)
Package Base: aptdec-git
Description: NOAA APT satellite imagery decoder
Upstream URL: https://github.com/Xerbo/aptdec
Keywords: apt noaa radio satellite
Licenses: GPL2
Conflicts: aptdec
Provides: aptdec
Submitter: shuyuan-liu
Maintainer: xiretza
Last Packager: xiretza
Votes: 3
Popularity: 0.000107
First Submitted: 2021-04-07 12:56 (UTC)
Last Updated: 2022-11-16 20:45 (UTC)

Dependencies (4)

Required by (2)

Sources (2)

Latest Comments

« First ‹ Previous 1 2

joshuarubin commented on 2021-06-28 17:43 (UTC)

I'm getting a linker error with this now:

[ 94%] Linking C executable aptdec
[ 94%] Built target aptdec
In function ‘FactPiv’,
    inlined from ‘polyreg’ at /home/jrubin/.cache/yay/aptdec-git/src/aptdec/src/libs/reg.c:68:2:
/home/jrubin/.cache/yay/aptdec-git/src/aptdec/src/libs/reg.c:115:34: error: ‘A’ may be used uninitialized [-Werror=maybe-uninitialized]
  115 |         DET = DET * A[Row[N - 1]][N - 1];
      |                     ~~~~~~~~~~~~~^~~~~~~
/home/jrubin/.cache/yay/aptdec-git/src/aptdec/src/libs/reg.c: In function ‘polyreg’:
/home/jrubin/.cache/yay/aptdec-git/src/aptdec/src/libs/reg.c:23:16: note: ‘A’ declared here
   23 |         double A[DMAX][DMAX];   /* A */
      |                ^
cc1: all warnings being treated as errors
make[2]: *** [CMakeFiles/apt.dir/build.make:132: CMakeFiles/apt.dir/src/libs/reg.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:113: CMakeFiles/apt.dir/all] Error 2
make: *** [Makefile:156: all] Error 2
==> ERROR: A failure occurred in build().
    Aborting...
error making: aptdec-git

shuyuan-liu commented on 2021-05-09 10:36 (UTC)

@gromain Thanks for letting me know, fixed. As you said, the CMakeLists.txt in upstream now uses standard CMake variables to define install paths so a patch is no longer needed.

gromain commented on 2021-05-09 08:59 (UTC) (edited on 2021-05-09 09:02 (UTC) by gromain)

Hi, This doesn't build against the latest version in the github repository. There is a problem with the patch for palettes, since this pull request changed the way things are handled: https://github.com/Xerbo/aptdec/pull/14/files

I think the patch needs to be updated to reflect those changes (or removed completely perhaps? I'm not sure).

Cheers

shuyuan-liu commented on 2021-04-24 18:16 (UTC)

Thanks @xiretza for the suggestion. Now both the executable and palettes are installed to proper locations.

xiretza commented on 2021-04-24 12:15 (UTC)

This needs a -DCMAKE_INSTALL_PREFIX='/usr'.