Skip to content

Commit

Permalink
{Packaging} Use proper PEP 508 environment marker for dependencies (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
jiasli authored Mar 17, 2022
1 parent 7cac0c8 commit 83c11a4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 9 deletions.
7 changes: 2 additions & 5 deletions src/azure-cli-core/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,16 +57,13 @@
'packaging>=20.9,<22.0',
'paramiko>=2.0.8,<3.0.0',
'pkginfo>=1.5.0.1',
# psutil can't install on cygwin: https://github.com/Azure/azure-cli/issues/9399
'psutil~=5.9; sys_platform != "cygwin"',
'PyJWT>=2.1.0',
'pyopenssl>=17.1.0', # https://github.com/pyca/pyopenssl/pull/612
'requests[socks]'
]

# dependencies for specific OSes
if not sys.platform.startswith('cygwin'):
DEPENDENCIES.append('psutil~=5.9')


with open('README.rst', 'r', encoding='utf-8') as f:
README = f.read()

Expand Down
6 changes: 2 additions & 4 deletions src/azure-cli/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,6 +139,8 @@
'azure-synapse-spark~=0.2.0',
'chardet~=3.0.4',
'colorama~=0.4.4',
# On Linux, the distribution (Ubuntu, Debian, etc) and version are checked for `az feedback`
'distro; sys_platform == "linux"',
'fabric~=2.4',
'javaproperties~=0.5.1',
'jsondiff~=1.3.0',
Expand All @@ -154,10 +156,6 @@
'xmltodict~=0.12'
]

# On Linux, the distribution (Ubuntu, Debian, etc) and version are checked
if sys.platform == 'linux':
DEPENDENCIES.append('distro')

with open('README.rst', 'r', encoding='utf-8') as f:
README = f.read()

Expand Down

0 comments on commit 83c11a4

Please sign in to comment.