Opened 2 years ago

Closed 5 months ago

#61231 closed defect (fixed)

molden: error: implicit declaration of function

Reported by: p-walt Owned by: ryandesign (Ryan Schmidt)
Priority: Normal Milestone:
Component: ports Version: 2.6.3
Keywords: catalina bigsur monterey Cc: vikilyc (yuchen liu), valerie-vallet
Port: molden

Description

Hi there, I have tried multiple times to install molden using MacPorts. I have tried multiple versions of gcc and still get the same errors generated.

Attachments (2)

Error log.rtf (115.0 KB) - added by p-walt 2 years ago.
Log file of the failed building of the molden port
main.log (110.0 KB) - added by valerie-vallet 19 months ago.
compilation log file

Download all attachments as: .zip

Change History (11)

Changed 2 years ago by p-walt

Attachment: Error log.rtf added

Log file of the failed building of the molden port

comment:1 Changed 2 years ago by ryandesign (Ryan Schmidt)

Owner: set to ryandesign
Status: newaccepted
Summary: Failure to install molden portmolden: error: implicit declaration of function

Looks like there are many errors in the log of this form:

src/xwin.c:8709:4: error: implicit declaration of function 'plinz_' [-Werror,-Wimplicit-function-declaration]
   plinz_(isel,istat,
   ^

Implicit function declaration errors are a new problem we're having to contend with since the introduction of Xcode 12. Prior to that, this was only considered a warning. We should fix these problems, but until we do, you can avoid them by downgrading to Xcode 11 and/or the Xcode 11 version of the command line tools.

comment:2 Changed 21 months ago by ryandesign (Ryan Schmidt)

Cc: vikilyc added
Keywords: bigsur catalina added

Has duplicate #61985.

It is too difficult for me to fix errors in molden (the file that these errors occur in is over 92,000 lines long) so I will report it to the developer of molden for them to fix.

comment:3 Changed 19 months ago by ryandesign (Ryan Schmidt)

Cc: valerie-vallet added

I did report the problems to the developer. They did not respond to my email but did add -Wno-implicit-function-declaration to the build system, which I promptly removed because that is not the correct fix (it is not going to work on Apple Silicon systems). However since it has now been awhile and the developer has not fixed it the correct way, I will allow -Wno-implicit-function-declaration back into the build system and will instead mark the port as incompatible with Apple Silicon.

Version 0, edited 19 months ago by ryandesign (Ryan Schmidt) (next)

comment:4 Changed 19 months ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: acceptedclosed

In 2ddc57db263fc9a0be09ea1888a4893d4f6e6435/macports-ports (master):

molden: Allow build with Xcode 12 on x86_64

Closes: #61231

comment:5 Changed 19 months ago by ryandesign (Ryan Schmidt)

Correction: the developer did respond and I had not replied to reiterate the unsuitability of -Wno-implicit-function-declaration. I've now done that.

comment:6 Changed 19 months ago by valerie-vallet

Resolution: fixed
Status: closedreopened

Despite your fix, there are still compiling issues. See the attached log file

Changed 19 months ago by valerie-vallet

Attachment: main.log added

compilation log file

comment:7 Changed 17 months ago by ryandesign (Ryan Schmidt)

The developer removed -Wno-implicit-function-declaration again, perhaps because of my email letting them know it was not a suitable fix, but has not implemented any other fix, so the problems will continue until the developer fixes the problem. If the problem affects you, you could email the developer and let them know.

comment:8 Changed 5 months ago by ryandesign (Ryan Schmidt)

Keywords: monterey added
Status: reopenedaccepted

I had not updated molden to a new version in a year because no new version appeared to be available. This turns out to have been because the homepage and download location changed. I'm now updating molden to the latest version (7.1) which still hasn't fixed the implicit function declaration problems. I'm therefore re-adding -Wno-implicit-function-declaration and marking the port as unsuitable for use with the arm64 architecture. Users who try to install the port on Apple Silicon machines will get the x86_64 version running in emulation (assuming MacPorts is able to install all of its dependencies with the +universal variant first). Users who wish to have an Apple Silicon-native version should contact the developer and ask them to fix the implicit function declarations.

comment:9 Changed 5 months ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: acceptedclosed

In 067f7f34a8a540aea559615220a163934f62a997/macports-ports (master):

molden: Update to 7.1

Update to 7.1. Update homepage and master_sites and fix livecheck.

Remove stealth update provisions; let's see if the developers have
learned to refrain from doing stealth updates by now.

Use -fallow-argument-mismatch to allow building with gcc10 and later.

Use -Wno-implicit-function-declaration to allow building with Xcode 12
and later on x86_64, therefore mark the port as incompatible with arm64.

Closes: #61231

Note: See TracTickets for help on using tickets.