firemodels / smv

Smokeview
Other
45 stars 177 forks source link

build error on ARM mac #2007

Closed ericvmueller closed 1 month ago

ericvmueller commented 1 month ago

In attempting to build gnu_osx_64 smokeview on an ARM (M3) mac, I get the following error in the linking stage:

Undefined symbols for architecture arm64:
  "_png_init_filter_functions_neon", referenced from:
      _png_read_filter_row in libpng.a[5](pngrutil.o)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [gnu_osx_64] Error 1

@marcosvanella has seen the same on his machine.

gforney commented 1 month ago

do we have any arm m3 macs at nist I can have access to?

ericvmueller commented 1 month ago

I think @marcosvanella said he should be able to set you up with access. Or you can take a look at the machine I'm using once I've finished setting things up.

gforney commented 1 month ago

hoping it is just a case of the png library needing to be updated

On Wed, Sep 4, 2024 at 1:04 PM Eric Mueller @.***> wrote:

I think @marcosvanella https://github.com/marcosvanella said he should be able to set you up with access. Or you can take a look at the machine I'm using once I've finished setting things up.

— Reply to this email directly, view it on GitHub https://github.com/firemodels/smv/issues/2007#issuecomment-2329576727, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC6UCRTS3F4FKMM6PYNHVXTZU44TLAVCNFSM6AAAAABNUW3332VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRZGU3TMNZSG4 . You are receiving this because you commented.Message ID: @.***>

-- Glenn Forney

marcosvanella commented 1 month ago

Glenn, let me know if you want to try updating the png lib. I have the arm mac here.


From: gforney @.> Sent: Wednesday, September 4, 2024 01:12 PM To: firemodels/smv @.> Cc: marcosvanella @.>; Mention @.> Subject: Re: [firemodels/smv] build error on ARM mac (Issue #2007)

hoping it is just a case of the png library needing to be updated

On Wed, Sep 4, 2024 at 1:04 PM Eric Mueller @.***> wrote:

I think @marcosvanella https://github.com/marcosvanella said he should be able to set you up with access. Or you can take a look at the machine I'm using once I've finished setting things up.

— Reply to this email directly, view it on GitHub https://github.com/firemodels/smv/issues/2007#issuecomment-2329576727, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC6UCRTS3F4FKMM6PYNHVXTZU44TLAVCNFSM6AAAAABNUW3332VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRZGU3TMNZSG4 . You are receiving this because you commented.Message ID: @.***>

-- Glenn Forney

— Reply to this email directly, view it on GitHubhttps://github.com/firemodels/smv/issues/2007#issuecomment-2329590149, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABY23VJP2JWXRTVJC5Y5NPDZU45QTAVCNFSM6AAAAABNUW3332VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRZGU4TAMJUHE. You are receiving this because you were mentioned.Message ID: @.***>

marcosvanella commented 1 month ago

Eric, do an update and try it. Glenn fixed it for the M3.

ericvmueller commented 1 month ago

Great - works now. Thanks! Thats the flag I was trying to implement but wasn't setting it in the right place