makedep: Update interpreter directive to python3#953
Merged
Conversation
adcroft
approved these changes
Sep 9, 2025
adcroft
force-pushed
the
makedep_use_py3
branch
from
c38392e to
03eec0b
Compare
Hallberg-NOAA
force-pushed
the
makedep_use_py3
branch
from
03eec0b to
ba73e67
Compare
Member
|
This PR has passed pipeline testing at https://gitlab.gfdl.noaa.gov/ogrp/mom6ci/MOM6/-/pipelines/28776. |
The interpreter directive ("shebang") of makedep is updated to
`python3`, rather than the version-agnostic `python`.
Although we never invoke the shebang of the script, there are OS
environments out there which will object to any presence of a
versionless python. PEP 394 also strongly recommends the adoption of
python3 as the executable name, regardless of Py2 support.
Hallberg-NOAA
force-pushed
the
makedep_use_py3
branch
from
ba73e67 to
25a8b66
Compare
Member
|
After rebasing, this PR has passed pipeline testing again at https://gitlab.gfdl.noaa.gov/ogrp/mom6ci/MOM6/-/pipelines/28783. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The interpreter directive ("shebang") of makedep is updated to
python3, rather than the version-agnosticpython.Although we never invoke the shebang of the script, there are OS environments out there which will object to any presence of a versionless python. PEP 394 also strongly recommends the adoption of python3 as the executable name, regardless of Py2 support.