Skip to content

Conversation

@BKPepe
Copy link
Member

@BKPepe BKPepe commented Dec 3, 2025

This looks like it is not maintained anymore.
The last commit in the repository [1] is from 2017, and the last tarball uploaded on PyPI is from 2015.

That tarball does not follow PEP625, because the
wheel and tarball begins with uppercase, it should be in lowercase and as well it is still using setup.py.

Python these days is using pyproject.toml.

[1] https://github.com/washort/parsley

This looks like it is not maintained anymore.
The last commit in the repository [1] is from 2017,
and the last tarball uploaded on PyPI is from 2015.

That tarball does not follow PEP625, because the
wheel and tarball begins with uppercase, it should be
in lowercase and as well it is still using setup.py.

Python these days is using pyproject.toml.

[1] https://github.com/washort/parsley

Signed-off-by: Josef Schlehofer <[email protected]>
Copilot AI review requested due to automatic review settings December 3, 2025 10:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the unmaintained python-parsley package from the repository. The package has not been updated since 2015 (PyPI) and 2017 (last repository commit), and does not follow modern Python packaging standards (PEP 625 compliance, pyproject.toml usage).

  • Removes the python-parsley package (version 1.3) along with all associated files
  • Eliminates two patch files that were previously applied to the package for Python 3 compatibility and test omission

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
lang/python/python-parsley/Makefile Removes the package definition for python3-parsley
lang/python/python-parsley/patches/001-py3-read-utf8.patch Removes the Python 3 UTF-8 compatibility patch
lang/python/python-parsley/patches/002-omit-tests.patch Removes the patch that excluded test packages from installation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BKPepe BKPepe merged commit 598dc27 into openwrt:master Dec 3, 2025
16 of 17 checks passed
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.

1 participant