sourceryinstitute / OpenCoarrays

A parallel application binary interface for Fortran 2018 compilers.
http://www.opencoarrays.org
BSD 3-Clause "New" or "Revised" License
244 stars 58 forks source link

Defect: `caf` and `cafrun` scripts don't parse arguments/options correctly #467

Open zbeekman opened 6 years ago

zbeekman commented 6 years ago
Avg response time
Issue Stats

Defect/Bug Report

See also #460 for additional context. The caftan script requires -np and <N_IMAGES> as the first and second argument (when not passing --show. Worse yet, --show gives no indication that there will be problems if you add an additional argument to caftan ahead of -np <N>.

Observed Behavior

Users pass --oversubscribe (required for OpenMPI) ahead of -np 5 etc. and things blow up

Expected Behavior

Parse options with arguments correctly. This will make the code cleaner and more generic too.

Steps to Reproduce

cafrun --show --oversubscribe -np 4 FORTRAN/transpose-coarray 10 1024 32 cafrun --oversubscribe -np 4 FORTRAN/transpose-coarray 10 1024 32

Put @zbeekman in the corner of shell scripting shame

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.