Package Details: aptdec-git r184.794d69c-2

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: shuyuan-liu
Last Packager: shuyuan-liu
Votes: 1
Popularity: 0.002557
First Submitted: 2021-04-07 12:56
Last Updated: 2021-07-01 02:04

Latest Comments

joshuarubin commented on 2021-07-01 15:58

Worked great, thanks!

shuyuan-liu commented on 2021-07-01 02:08

@joshuarubin I implemented a temporary fix and initialized the array 'A' to all zeros; seems to be working fine for now. Could you try and see if it builds for you now?

shuyuan-liu commented on 2021-06-29 01:26

@joshuarubin I managed to reproduce the error with gcc 11.1.0; clang 12.0.0 compiles without problems. Will report upstream.

joshuarubin commented on 2021-06-28 17:43

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

@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

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

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

xiretza commented on 2021-04-24 12:15

This needs a -DCMAKE_INSTALL_PREFIX='/usr'.