Skip to content

Fixes for Click 8.2 and 8.3#2065

Merged
juergbi merged 2 commits intomasterfrom
jbilleter/click
Sep 19, 2025
Merged

Fixes for Click 8.2 and 8.3#2065
juergbi merged 2 commits intomasterfrom
jbilleter/click

Conversation

@juergbi
Copy link
Contributor

@juergbi juergbi commented Sep 19, 2025

Two fixes to pass the test suite with Click 8.3.0 without breaking older versions of Click. We still have to support Click 8.1.x as Click 8.2.0 already dropped support for Python 3.9.

Fixes #121.

As per https://docs.python.org/3/library/constants.html#NotImplemented:

    A special value which should be returned by the binary special
    methods (e.g. __eq__(), __lt__(), __add__(), __rsub__(), etc.) to
    indicate that the operation is not implemented with respect to the
    other type

This fixes crashes with Click 8.3.0.
@juergbi juergbi merged commit a7455a3 into master Sep 19, 2025
35 of 36 checks passed
@juergbi juergbi deleted the jbilleter/click branch September 19, 2025 14:02
starnight added a commit to endlessm/eos-build-meta that referenced this pull request Sep 22, 2025
CI runner installed BuildStream hits issue mentioned by "Fixes for Click
8.2 and 8.3" [1]. However, the fix has not been released, yet. So, force
install Click 8.2.1 temporarily, until newer BuildStream version rolls
out.

[1]: apache/buildstream#2065

Fixes: #121
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.

Remove pre-/post- command prefix support

2 participants