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

Added swig to swig requirements_dev.txt #1019

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

Conversation

jesper-friis
Copy link
Collaborator

Description

Install swig with pip instead of via the linux distribution to ensure a consistent swig version.

Type of change

  • Bug fix & code cleanup
  • New feature
  • Documentation update
  • Test update

Checklist for the reviewer

This checklist should be used as a help for the reviewer.

  • Is the change limited to one issue?
  • Does this PR close the issue?
  • Is the code easy to read and understand?
  • Do all new feature have an accompanying new test?
  • Has the documentation been updated as necessary?

@jesper-friis jesper-friis mentioned this pull request Nov 27, 2024
9 tasks
@jesper-friis jesper-friis added the documentation Improvements or additions to documentation label Nov 27, 2024
@jesper-friis jesper-friis removed the request for review from francescalb November 27, 2024 15:19
@jesper-friis jesper-friis added Keep for reference This PR might not be merged, but has valuable content superseeded Superseded by another PR. Should not be merged and removed documentation Improvements or additions to documentation Keep for reference This PR might not be merged, but has valuable content labels Nov 27, 2024
jesper-friis added a commit that referenced this pull request Nov 28, 2024
Updated the weekly build of docker containers:
- change to relative path in the COPY statements of the Dockerfiles generated with gen_dockerfile.sh
- reduce number of build containers to 4: manylinux/musllinux, i686/x86_64
- parametrised python versions
- installed rust and cargo on musllinux and manylinux2014
- updated ci_build_wheels to match the reduced number of containers that are build
- install SWIG with pip instead of via the linux distribution to ensure a consistent swig version
- avoid that split_url() and split_meta_uri() returns and extra None argument with SWIG 4.3.0

This PR combines PR #1017 and #1019. It should be followed up by a PR adding support for Python 3.13 and more included dependencies like pydantic, paramiko, pandas, ... (except for manylinux2014)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
superseeded Superseded by another PR. Should not be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant