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

WIP: Migrate pynipap to setuptools #1419

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

garberg
Copy link
Member

@garberg garberg commented Jan 13, 2025

As distutils is being phased out, let's initiate the work with migrating to something more modern. Let's start with pynipap.

@garberg garberg changed the title Migrate pynipap to setuptools WIP: Migrate pynipap to setuptools Jan 13, 2025
@garberg
Copy link
Member Author

garberg commented Jan 13, 2025

There seems to be some bug in the build chain (pybuild, pyproject...) which is resolved in Ubuntu 24.04, thus the pipeline will need to be migrated to 24.04 in #1418 before this can be merged.

nipap/nipap/rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
tests/test_rest.py Outdated Show resolved Hide resolved
@garberg garberg force-pushed the repackage_pynipap branch 7 times, most recently from 798ac44 to ff08814 Compare January 15, 2025 21:20
Migrated to a more modern build-chain. Includes some changes to the
Debian packaging as well.
@garberg garberg merged commit b8fcb9f into SpriteLink:master Jan 16, 2025
4 of 6 checks passed
@garberg garberg deleted the repackage_pynipap branch January 16, 2025 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants