Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mom5: new switch --no_version is required #51

Merged
merged 1 commit into from
Nov 6, 2023
Merged

mom5: new switch --no_version is required #51

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion packages/mom5/package.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
version("master", branch="master")

variant("deterministic", default=False, description="Deterministic build.")
variant("optimisation_report", default=False, description="Generate optimisation reports.")

Check failure on line 22 in packages/mom5/package.py

View workflow job for this annotation

GitHub Actions / Check syntax

Ruff (E501) 

packages/mom5/package.py:22:89: E501 Line too long (95 > 88 characters)

# Depend on virtual package "mpi".
depends_on("mpi")
Expand Down Expand Up @@ -51,7 +51,7 @@

# NOTE: The order of the libraries matter during the linking step!
# NOTE: datetime-fortran is a dependency of libaccessom2.
ldeps = ["oasis3-mct", "libaccessom2", "netcdf-c", "netcdf-fortran", "datetime-fortran"]

Check failure on line 54 in packages/mom5/package.py

View workflow job for this annotation

GitHub Actions / Check syntax

Ruff (E501) 

packages/mom5/package.py:54:89: E501 Line too long (96 > 88 characters)
libs = " ".join([(spec[d].libs).ld_flags for d in ldeps])

# TODO: https://github.com/ACCESS-NRI/ACCESS-OM/issues/12
Expand All @@ -78,7 +78,7 @@

FPPFLAGS :=

FFLAGS := -fcray-pointer -fdefault-real-8 -ffree-line-length-none -fno-range-check -Waliasing -Wampersand -Warray-bounds -Wcharacter-truncation -Wconversion -Wline-truncation -Wintrinsics-std -Wsurprising -Wno-tabs -Wunderflow -Wunused-parameter -Wintrinsic-shadow -Wno-align-commons -fallow-argument-mismatch -fallow-invalid-boz

Check failure on line 81 in packages/mom5/package.py

View workflow job for this annotation

GitHub Actions / Check syntax

Ruff (E501) 

packages/mom5/package.py:81:89: E501 Line too long (329 > 88 characters)
FFLAGS += {incs}
FFLAGS += -DGFORTRAN

Expand Down Expand Up @@ -162,10 +162,10 @@
INCLUDE := {incs}

FPPFLAGS := -fpp -Wp,-w $(INCLUDE)
FFLAGS := -fno-alias -safe-cray-ptr -fpe0 -ftz -assume byterecl -i4 -r8 -nowarn -check noarg_temp_created -assume nobuffered_io -convert big_endian -grecord-gcc-switches -align all

Check failure on line 165 in packages/mom5/package.py

View workflow job for this annotation

GitHub Actions / Check syntax

Ruff (E501) 

packages/mom5/package.py:165:89: E501 Line too long (180 > 88 characters)
FFLAGS_OPT := {FFLAGS_OPT}
FFLAGS_REPORT := -qopt-report=5 -qopt-report-annotate
FFLAGS_DEBUG := -g3 -O0 -debug all -check -check noarg_temp_created -check nopointer -warn -warn noerrors -ftrapuv -traceback

Check failure on line 168 in packages/mom5/package.py

View workflow job for this annotation

GitHub Actions / Check syntax

Ruff (E501) 

packages/mom5/package.py:168:89: E501 Line too long (125 > 88 characters)
FFLAGS_REPRO := -fp-model precise -fp-model source -align all
FFLAGS_VERBOSE := -v -V -what

Expand Down Expand Up @@ -310,12 +310,16 @@
if "+optimisation_report" in self.spec:
build.add_default_env("REPORT", "true")

# The MOM5 commit d7ba13a3f364ce130b6ad0ba813f01832cada7a2
# requires the --no_version switch to avoid git hashes being
# embedded in the binary.
build(
"--type",
self._mom_type,
"--platform",
self._platform,
"--no_environ"
"--no_environ",
"--no_version"
)

def install(self, spec, prefix):
Expand Down
Loading