Opened 4 years ago

Closed 3 years ago

#61231 closed defect (fixed)

molden: error: implicit declaration of function

Reported by: p-walt Owned by: ryandesign (Ryan Carsten 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 4 years ago.
Log file of the failed building of the molden port
main.log (110.0 KB) - added by valerie-vallet 4 years ago.
compilation log file

Download all attachments as: .zip

Change History (11)

Changed 4 years ago by p-walt

Attachment: Error log.rtf added

Log file of the failed building of the molden port

comment:1 Changed 4 years ago by ryandesign (Ryan Carsten 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 4 years ago by ryandesign (Ryan Carsten 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 4 years ago by ryandesign (Ryan Carsten 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.

Last edited 4 years ago by ryandesign (Ryan Carsten Schmidt) (previous) (diff)

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

Resolution: fixed
Status: acceptedclosed

In 2ddc57db263fc9a0be09ea1888a4893d4f6e6435/macports-ports (master):

molden: Allow build with Xcode 12 on x86_64

Closes: #61231

comment:5 Changed 4 years ago by ryandesign (Ryan Carsten 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 4 years ago by valerie-vallet

Resolution: fixed
Status: closedreopened

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

Changed 4 years ago by valerie-vallet

Attachment: main.log added

compilation log file

comment:7 Changed 4 years ago by ryandesign (Ryan Carsten 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 3 years ago by ryandesign (Ryan Carsten 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 3 years ago by ryandesign (Ryan Carsten 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.