Skip to content

Commit

Permalink
Merge pull request #4716 from ESMCI/distutils_cleanup
Browse files Browse the repository at this point in the history
remove another reference to distutils
  • Loading branch information
jedwards4b authored Dec 11, 2024
2 parents b2f271b + 38e813a commit 7c0ca7d
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ jobs:
# https://docs.github.com/en/actions/managing-workflow-runs/downloading-workflow-artifacts
- name: Upload testing logs
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: testing-logs-${{ github.run_number }}-${{ matrix.model }}-${{ matrix.driver }}
path: /testing-logs-${{ github.run_number}}-${{ matrix.model }}-${{ matrix.driver }}.tar.gz
Expand Down
6 changes: 2 additions & 4 deletions CIME/code_checker.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import os
import json
from shutil import which

from CIME.XML.standard_module_setup import *

Expand All @@ -19,16 +20,13 @@

from multiprocessing.dummy import Pool as ThreadPool

# pylint: disable=import-error
from distutils.spawn import find_executable

logger = logging.getLogger(__name__)


###############################################################################
def _run_pylint(all_files, interactive):
###############################################################################
pylint = find_executable("pylint")
pylint = which("pylint")

cmd_options = (
" --disable=I,C,R,logging-not-lazy,wildcard-import,unused-wildcard-import"
Expand Down
6 changes: 2 additions & 4 deletions docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,7 @@ COPY cime.yaml /cime.yaml
RUN mamba env update -f /cime.yaml && \
rm -rf /opt/conda/pkgs/* && \
ln -sf /opt/conda/bin/x86_64-conda-linux-gnu-ar /opt/conda/bin/ar && \
ln -sf /opt/conda/bin/x86_64-conda-linux-gnu-ranlib /opt/conda/bin/ranlib && \
# need compilers
cpan install XML::LibXML Switch
ln -sf /opt/conda/bin/x86_64-conda-linux-gnu-ranlib /opt/conda/bin/ranlib

ARG PNETCDF_VERSION=1.12.3
ENV PNETCDF_VERSION=${PNETCDF_VERSION}
Expand Down Expand Up @@ -81,7 +79,7 @@ RUN apt-get update && \
apt-get install -y --no-install-recommends \
curl ca-certificates software-properties-common \
gcc make libtool libhwloc-dev libx11-dev libxt-dev libedit-dev \
libical-dev ncurses-dev perl python-dev tcl-dev tk-dev swig libexpat-dev libssl-dev \
libical-dev ncurses-dev python-dev tcl-dev tk-dev swig libexpat-dev libssl-dev \
libxext-dev libxft-dev autoconf automake \
postgresql-12 postgresql-server-dev-all postgresql-contrib \
expat libedit2 python3 sendmail-bin sudo tcl tk && \
Expand Down
1 change: 1 addition & 0 deletions docker/cime.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ dependencies:
- curl
- subversion
- m4
- pkg-config
- pytest
- pytest-cov
- pyyaml
Expand Down

0 comments on commit 7c0ca7d

Please sign in to comment.