Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#56938 closed defect (fixed)

gcc7 fortran fails simple tests

Reported by: ogourgue (Olivier Gourgue) Owned by: cjones051073 (Chris Jones)
Priority: Normal Milestone:
Component: ports Version: 2.5.3
Keywords: Cc: eborisch (Eric A. Borisch), iqgrande, maehne (Torsten Maehne)
Port: gcc7

Description (last modified by eborisch (Eric A. Borisch))

gfortran-mp-7 is failing simple tests.

Trying to compile simple testfile conftest.f (as created by mpich during configure):

        program main

        end

fails when called with /opt/local/bin/gfortran-mp-7 -o conftest conftest.f for 7.3.0 revision 2, but works with 7.3.0 revision 1. (As tested by deactivating / activating using the pre-compiled packages from macports.)

I don't know if this is something that changed outside of gcc7 in the interim that broke the new builds, but the end result appears broken.

Attachments (4)

main.log (92.2 KB) - added by ogourgue (Olivier Gourgue) 6 years ago.
config.log (97.7 KB) - added by ogourgue (Olivier Gourgue) 6 years ago.
working.log (4.4 KB) - added by eborisch (Eric A. Borisch) 6 years ago.
7.3.0_1: working compilation
broken.log (2.7 KB) - added by eborisch (Eric A. Borisch) 6 years ago.
7.3.0_2: Broken compilation

Download all attachments as: .zip

Change History (15)

Changed 6 years ago by ogourgue (Olivier Gourgue)

Attachment: main.log added

Changed 6 years ago by ogourgue (Olivier Gourgue)

Attachment: config.log added

comment:1 Changed 6 years ago by mf2k (Frank Schima)

Cc: eborisch removed
Owner: set to eborisch
Status: newassigned

comment:2 Changed 6 years ago by eborisch (Eric A. Borisch)

Cc: eborisch added; seanfarley removed
Description: modified (diff)
Owner: changed from eborisch to cjones051073
Port: gcc7 added; mpich-default removed
Summary: Processing of port mpich-default failedgcc7 fortran fails simple tests

Changed 6 years ago by eborisch (Eric A. Borisch)

Attachment: working.log added

7.3.0_1: working compilation

Changed 6 years ago by eborisch (Eric A. Borisch)

Attachment: broken.log added

7.3.0_2: Broken compilation

comment:3 Changed 6 years ago by eborisch (Eric A. Borisch)

The main difference between the logs (where things diverge) is that the new gfortran-mp-7 is adding -mmacosx-version-min=10.13 to the call to the assembler, and leading to FATAL:/opt/local/bin/../libexec/as/x86_64/as: I don't understand 'm' flag!

comment:4 Changed 6 years ago by iqgrande

Cc: iqgrande added

comment:5 Changed 6 years ago by cjones051073 (Chris Jones)

This is an issue with cctools.

The problem is cctools was recently updated to make a new Xcode variant the default on macOS systems that use Xcode 9 or newer, to fix issues with the fact the port at the moment provides a tool kit based on Xcode 8, which in some situations was causing issues. The new Xcode variant instead just installs wrapper scripts.

The issue is MacPorts maintains previous variants, and as cctools had a different variant enabled by default before this is maintained.

The work around is to force uninstall cctools, then reinstall it with the new defaults.

sudo port -f uninstall cctools
sudo port install cctools

After this, check your cctools installation is using the Xcode variant.

This is not the first time I have seen this issue in the last few days. We need to look into a way of forcing the use of the Xcode variant.

Chris

Last edited 6 years ago by cjones051073 (Chris Jones) (previous) (diff)

comment:6 Changed 6 years ago by cjones051073 (Chris Jones)

FYI See #56916

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

comment:7 Changed 6 years ago by cjones051073 (Chris Jones)

comment:8 Changed 6 years ago by eborisch (Eric A. Borisch)

Resolution: fixed
Status: assignedclosed

mpich-default builds successfully after the steps outlined above to catch the new default variant for cctools. Once the PR is merged (and percolates out) this will hopefully resolve itself without further user action.

Thanks, Chris!

comment:9 Changed 6 years ago by ogourgue (Olivier Gourgue)

The work around suggested by cjones051073 worked for me! Thanks a lot for your help guys!

comment:10 Changed 6 years ago by cjones051073 (Chris Jones)

PR now merged.

comment:11 Changed 6 years ago by maehne (Torsten Maehne)

Cc: maehne added
Note: See TracTickets for help on using tickets.