Opened 5 days ago

Last modified 4 days ago

#71003 assigned defect

source-extractor @2.28.0: error: call to undeclared library function 'finite'

Reported by: cantiello (Michele Cantiello) Owned by: lpsinger (Leo Singer)
Priority: Normal Milestone:
Component: ports Version: 2.10.1
Keywords: arm64 Cc:
Port: source-extractor

Description (last modified by ryandesign (Ryan Carsten Schmidt))

Installing source-extractor on a macbook iOS with M3, I get the following error:

 sudo port install source-extractor 
--->  Computing dependencies for source-extractor
--->  Fetching archive for source-extractor
--->  Attempting to fetch source-extractor-2.28.0_0.darwin_23.arm64.tbz2 from https://packages.macports.org/source-extractor
--->  Attempting to fetch source-extractor-2.28.0_0.darwin_23.arm64.tbz2 from https://fra.de.packages.macports.org/source-extractor
--->  Attempting to fetch source-extractor-2.28.0_0.darwin_23.arm64.tbz2 from https://nue.de.packages.macports.org/source-extractor
--->  Building source-extractor
Error: Failed to build source-extractor: command execution failed
Error: See /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_science_source-extractor/source-extractor/main.log for details.
Error: Follow https://guide.macports.org/#project.tickets if you believe there is a bug.
Error: Processing of port source-extractor failed

Mik

Attachments (1)

main.log (677.1 KB) - added by cantiello (Michele Cantiello) 5 days ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 5 days ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)
Keywords: arm64 added
Owner: set to lpsinger
Status: newassigned

source-extractor also fails to build on our arm64 build machines, but please attach the main.log file so that we can verify if it's the same problem.

Changed 5 days ago by cantiello (Michele Cantiello)

Attachment: main.log added

comment:2 in reply to:  1 Changed 5 days ago by cantiello (Michele Cantiello)

Replying to ryandesign:

source-extractor also fails to build on our arm64 build machines, but please attach the main.log file so that we can verify if it's the same problem.

Done!

Thanks

comment:3 Changed 4 days ago by ryandesign (Ryan Carsten Schmidt)

Summary: install source-extractorsource-extractor @2.28.0: error: call to undeclared library function 'finite'
Version: 2.10.1

Yup, same problem we see:

In file included from lm.c:81:
./lm_core.c:206:7: error: call to undeclared library function 'finite' with type 'int (double)'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  if(!LM_FINITE(p_eL2)) stop=7;
      ^
./compiler.h:63:19: note: expanded from macro 'LM_FINITE'
#define LM_FINITE finite // ICC, GCC
                  ^
./lm_core.c:206:7: note: include the header <math.h> or explicitly provide a declaration for 'finite'
./compiler.h:63:19: note: expanded from macro 'LM_FINITE'
#define LM_FINITE finite // ICC, GCC
                  ^
In file included from lmbc.c:84:
./lmbc_core.c:324:13: error: call to undeclared library function 'finite' with type 'int (double)'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
              if (!LM_FINITE(fpls)) {
                   ^
./compiler.h:63:19: note: expanded from macro 'LM_FINITE'
#define LM_FINITE finite // ICC, GCC
                  ^
./lmbc_core.c:324:13: note: include the header <math.h> or explicitly provide a declaration for 'finite'
./compiler.h:63:19: note: expanded from macro 'LM_FINITE'
#define LM_FINITE finite // ICC, GCC
                  ^
1 error generated.
1 error generated.

This was also filed upstream as https://github.com/astromatic/sextractor/issues/24 and again as https://github.com/astromatic/sextractor/issues/27. finite became obsolete in 1999 and is not included on Apple Silicon systems. The code needs to be changed to use isfinite instead. This was done in https://github.com/astromatic/sextractor/commit/ced65570cb5b7073361dbf2c3c60631c3f54d0f9 so we could backport that patch.

Note: See TracTickets for help on using tickets.