Skip to content

Remove use of python setup.py develop/install in the project #2172

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ndgrigorian merged 23 commits into from
Nov 21, 2025
Merged

Remove use of python setup.py develop/install in the project #2172

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
164ba83
update build_locally script to avoid python setup.py develop call
ndgrigorian Oct 14, 2025
2e96092
refactor common build functionality out into separate file
ndgrigorian Oct 15, 2025
07590f7
remove --build-dir option and fix --clean option
ndgrigorian Oct 15, 2025
5b0bd76
do not return compiler root unnecessarily from resolve_compilers
ndgrigorian Oct 15, 2025
6ab378b
update gen_coverage script to align with build_locally
ndgrigorian Oct 21, 2025
df44252
resolve bin_llvm from default compiler layout when not provided
ndgrigorian Oct 28, 2025
23aad14
keep find_objects defined within main
ndgrigorian Oct 28, 2025
6ac779e
generalize err and warn utilities for different build scripts
ndgrigorian Oct 28, 2025
bfd5891
use common resolve_compilers utility
ndgrigorian Oct 28, 2025
a54e341
update gen_docs script
ndgrigorian Oct 28, 2025
0738341
try using pip install -e in place of setup.py develop in CI
ndgrigorian Oct 28, 2025
b84cdf8
use python setup.py build_ext in Cython extension building
ndgrigorian Oct 28, 2025
e675ff1
add types and descriptions for script args
ndgrigorian Oct 30, 2025
8ce292f
do not override CMAKE_ARGS in build scripts
ndgrigorian Oct 30, 2025
6b62a8e
fix typo in cmake arg
ndgrigorian Oct 30, 2025
8bfa41c
remove `--target-level-zero argument`
ndgrigorian Nov 5, 2025
ec29c49
raise RuntimeError from scripts with invalid arguments
ndgrigorian Nov 5, 2025
b0bbfb3
Update CONTRIBUTING.md
ndgrigorian Nov 6, 2025
17a09fc
Update docs and example build instructions to remove python setup.py …
ndgrigorian Nov 6, 2025
ab77357
remove adding scripts to sys.path
ndgrigorian Nov 6, 2025
82ea6d7
Use updated build scripts in CI
ndgrigorian Nov 6, 2025
d5f25cf
Merge pull request #2189 from IntelPython/use-build-scripts-in-public-ci
ndgrigorian Nov 6, 2025
7962541
address review comments
ndgrigorian Nov 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 2 additions & 9 deletions .github/workflows/generate-docs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,15 +75,8 @@ jobs:
source /opt/intel/oneapi/setvars.sh
wget https://github.com/vovkos/doxyrest/releases/download/doxyrest-2.1.2/doxyrest-2.1.2-linux-amd64.tar.xz
tar xf doxyrest-2.1.2-linux-amd64.tar.xz
python setup.py build_ext --inplace --generator=Ninja --build-type=Release \
-- \
-DCMAKE_C_COMPILER:PATH=$(which icx) \
-DCMAKE_CXX_COMPILER:PATH=$(which icpx) \
-DDPCTL_GENERATE_DOCS=ON \
-DDPCTL_ENABLE_DOXYREST=ON \
-DDoxyrest_DIR=`pwd`/doxyrest-2.1.2-linux-amd64 \
-DCMAKE_VERBOSE_MAKEFILE=ON
python -m pip install -e . --no-build-isolation
python scripts/gen_docs.py --c-compiler=$(which icx) --cxx-compiler=$(which icpx) \
Comment thread
ndgrigorian marked this conversation as resolved.
Outdated
--doxyrest-root=`pwd`/doxyrest-2.1.2-linux-amd64 --verbose || exit 1
python -c "import dpctl; print(dpctl.__version__)" || exit 1
pushd "$(find _skbuild -name cmake-build)" || exit 1
cmake --build . --target Sphinx || exit 1
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/os-llvm-sycl-build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -146,8 +146,8 @@ jobs:
shell: bash -l {0}
run: |
source set_allvars.sh
CC=clang CXX=clang++ python setup.py build_ext --inplace --generator=Ninja
python -m pip install -e . --no-build-isolation
python scripts/build_locally.py --c-compiler=clang --cxx-compiler=clang++ \
--compiler-root=${SYCL_BUNDLE_FOLDER}/dpcpp_compiler/bin || exit 1

- name: Run lsplatforms
shell: bash -l {0}
Expand Down
Loading