Skip to content
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.

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

Setup binder as a Python package and publish pre-built wheels to PyPI #303

Open
wants to merge 62 commits into
base: master
Choose a base branch
from

Conversation

haiiliin
Copy link
Contributor

@haiiliin haiiliin commented Jun 19, 2024

Description of the implementation

  • The python package is placed in the python directory to avoid conflicts.
  • The llvm-project branch v6.x is submoduled in python/llvm-project and is included in python/CMakeLists.txt with add_subdirectory.
  • The source directory is symbolically linked to python/llvm-project/clang-tools-extra/binder in the llvm-project submodule.
  • The version of the python package is automatically inferred from the git tag using setuptools-scm.
  • scikit-build-core is used to build the Python package with cmake.
  • cibuildwheel is used to build wheels on different platforms, and archs using GitHub actions, and they will be published to PyPI when a release is published.

@haiiliin haiiliin force-pushed the master branch 2 times, most recently from 78cbdf6 to edd669d Compare June 19, 2024 18:52
Revert "Update README and LICENSE in pyproject.toml"

This reverts commit edd669d.

Update symbolic links

Not using symbolic links for README and LICENSE
@haiiliin
Copy link
Contributor Author

Linux build failed because it is built in a docker container. Part of (the python directory) the project was copied in the container and the symbolic link for the binder source broke.

@haiiliin haiiliin closed this Jul 29, 2024
@haiiliin haiiliin reopened this Oct 21, 2024
@haiiliin
Copy link
Contributor Author

The LLVM project is now cloned at build time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants