From 79745021553f227b2ed8a5a3508ddb7bda939c54 Mon Sep 17 00:00:00 2001 From: Chuck Hastings <45364586+ChuckHastings@users.noreply.github.com> Date: Wed, 3 Mar 2021 12:28:11 -0500 Subject: [PATCH] =?UTF-8?q?update=20default=20path=20of=20setup=20to=20use?= =?UTF-8?q?=20the=20new=20directory=20paths=20in=20build=20=E2=80=A6=20(#1?= =?UTF-8?q?425)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit A recent change caused the `setup.py` running in a clean checkout to fail unless either `RAFT_PATH` or `CUGRAPH_BUILD_PATH` were set. These are typically set if you run the build scripts, but if you try running the build steps individually by hand the script would fail unless you set one of these environment variables correctly. This PR fixes the default path (if neither environment variable is specified) to construct the proper location for looking up raft. Closes #1428 Authors: - Chuck Hastings (@ChuckHastings) Approvers: - Rick Ratzel (@rlratzel) - Brad Rees (@BradReesWork) URL: https://github.com/rapidsai/cugraph/pull/1425 --- python/setuputils.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/setuputils.py b/python/setuputils.py index 47eaf74d4b6..d93051d05ef 100644 --- a/python/setuputils.py +++ b/python/setuputils.py @@ -152,7 +152,7 @@ def clone_repo_if_needed(name, cpp_build_path=None, repo_path = ( _get_repo_path() + '/python/_external_repositories/' + name + '/') else: - repo_path = os.path.join(cpp_build_path, name + '/src/' + name + '/') + repo_path = os.path.join(cpp_build_path, '_deps', name + '-src') return repo_path, repo_cloned