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

catkin_lint not compatible with Python 2 anymore #113

Open
rhaschke opened this issue Oct 6, 2024 · 1 comment
Open

catkin_lint not compatible with Python 2 anymore #113

rhaschke opened this issue Oct 6, 2024 · 1 comment

Comments

@rhaschke
Copy link

rhaschke commented Oct 6, 2024

Latest PyPi release of catkin_lint is not compatible with Python 2.7 anymore:

  Traceback (most recent call last):
    File "/usr/local/bin/catkin_lint", line 52, in <module>
      from catkin_lint.main import main
    File "/usr/local/lib/python2.7/dist-packages/catkin_lint/__init__.py", line 39, in <module>
      from ._version import version as __version__
    File "/usr/local/lib/python2.7/dist-packages/catkin_lint/_version.py", line 10
      version: str
             ^
  SyntaxError: invalid syntax

It would be great if a version compatible with Python 2 would still be available via pip install.

@rhaschke
Copy link
Author

rhaschke commented Oct 8, 2024

Looks like the Python 3 requirement comes from (a new release of) setuptools-scm, which generates _version.py.
Adding a python_requires=">=3" in setup(...) should resolve the issue.
industrial_ci pinned the version manually for now.

toliver added a commit to shadow-robot/sr-build-tools that referenced this issue Oct 25, 2024
For kinetic (python2) systems, we've bumped into this issue with catkin_lint fkie/catkin_lint#113

That makes the style checks fail.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant