From a7cb321db04a97ee5f125c4db7aa869d20357ea3 Mon Sep 17 00:00:00 2001 From: Jiaming You Date: Thu, 26 Sep 2024 18:27:31 -0700 Subject: [PATCH] Fixed the setup.py issue Summary: setup.py are not able to parse url https://capi-automation.s3.us-east-2.amazonaws.com/public/python/capi_param_builder/dist/capi_param_builder-0.1.0.dev0.tar.gz from requirements.txt this diff fix that source https://www.activestate.com/resources/quick-reads/how-to-package-python-dependencies-with-pip-setuptools/ Reviewed By: liliarizona Differential Revision: D63491696 fbshipit-source-id: 82b5738590050d68fa7ae4943b1ba9ae89cd5d7a --- requirements.txt | 3 ++- setup.py | 11 ++++++++++- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/requirements.txt b/requirements.txt index 7e35b91d..e5d6c84e 100644 --- a/requirements.txt +++ b/requirements.txt @@ -3,4 +3,5 @@ six >= 1.7.3 curlify >= 2.1.0 pycountry >= 19.8.18 aiohttp; python_version >= '3.5.3' -https://capi-automation.s3.us-east-2.amazonaws.com/public/python/capi_param_builder/dist/capi_param_builder-0.1.0.dev0.tar.gz +https://capi-automation.s3.us-east-2.amazonaws.com/public/python/capi_param_builder/dist/capi_param_builder-0.1.0.dev0-py3-none-any.whl +capi_param_builder >= 0.1.0-dev0 diff --git a/setup.py b/setup.py index c369de62..32df0307 100644 --- a/setup.py +++ b/setup.py @@ -54,7 +54,15 @@ PACKAGE_LONG_DESCRIPTION = f.read() with open(requirements_filename) as f: - PACKAGE_INSTALL_REQUIRES = [line[:-1] for line in f] + PACKAGE_INSTALL_REQUIRES = [] + DEPENDENCY_LINKS = [] + + for line in f: + line = line.strip() + if line.lower().startswith(('http://', 'https://')): + DEPENDENCY_LINKS.append(line) + else: + PACKAGE_INSTALL_REQUIRES.append(line) setup( name=PACKAGE_NAME, @@ -70,4 +78,5 @@ long_description=PACKAGE_LONG_DESCRIPTION, install_requires=PACKAGE_INSTALL_REQUIRES, long_description_content_type="text/markdown", + dependency_links=DEPENDENCY_LINKS, )