Skip to content

Commit

Permalink
Remove distutils references
Browse files Browse the repository at this point in the history
  • Loading branch information
jsiirola committed Dec 8, 2023
1 parent fd5b4da commit e3df7bd
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 11 deletions.
9 changes: 3 additions & 6 deletions pyomo/common/cmake_builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,8 @@ def handleReadonly(function, path, excinfo):
def build_cmake_project(
targets, package_name=None, description=None, user_args=[], parallel=None
):
# Note: setuptools must be imported before distutils to avoid
# warnings / errors with recent setuptools distributions
from setuptools import Extension
import distutils.core
from distutils.command.build_ext import build_ext
from setuptools import Extension, Distribution
from setuptools.command.build_ext import build_ext

class _CMakeBuild(build_ext, object):
def run(self):
Expand Down Expand Up @@ -122,7 +119,7 @@ def __init__(self, target_dir, user_args, parallel):
'ext_modules': ext_modules,
'cmdclass': {'build_ext': _CMakeBuild},
}
dist = distutils.core.Distribution(package_config)
dist = Distribution(package_config)
basedir = os.path.abspath(os.path.curdir)
try:
tmpdir = os.path.abspath(tempfile.mkdtemp())
Expand Down
3 changes: 1 addition & 2 deletions pyomo/contrib/appsi/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,7 @@ def get_appsi_extension(in_setup=False, appsi_root=None):

def build_appsi(args=[]):
print('\n\n**** Building APPSI ****')
import setuptools
from distutils.dist import Distribution
from setuptools import Distribution
from pybind11.setup_helpers import build_ext
import pybind11.setup_helpers
from pyomo.common.envvar import PYOMO_CONFIG_DIR
Expand Down
6 changes: 3 additions & 3 deletions pyomo/contrib/mcpp/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,8 @@ def _generate_configuration():


def build_mcpp():
import distutils.core
from distutils.command.build_ext import build_ext
from setuptools import Distribution
from setuptools.command.build_ext import build_ext

class _BuildWithoutPlatformInfo(build_ext, object):
# Python3.x puts platform information into the generated SO file
Expand All @@ -87,7 +87,7 @@ def get_ext_filename(self, ext_name):
print("\n**** Building MCPP library ****")
package_config = _generate_configuration()
package_config['cmdclass'] = {'build_ext': _BuildWithoutPlatformInfo}
dist = distutils.core.Distribution(package_config)
dist = Distribution(package_config)
install_dir = os.path.join(envvar.PYOMO_CONFIG_DIR, 'lib')
dist.get_command_obj('install_lib').install_dir = install_dir
try:
Expand Down
2 changes: 2 additions & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,10 @@
from setuptools import setup, find_packages, Command

try:
# This works beginning in setuptools 40.7.0 (27 Jan 2019)
from setuptools import DistutilsOptionError
except ImportError:
# Needed for setuptools prior to 40.7.0
from distutils.errors import DistutilsOptionError


Expand Down

0 comments on commit e3df7bd

Please sign in to comment.