Skip to content

add cysignals dependency#314

Merged
malb merged 1 commit into
fplll:masterfrom
keltecc:feature/add-cysignals-dependency
Jun 15, 2026
Merged

add cysignals dependency#314
malb merged 1 commit into
fplll:masterfrom
keltecc:feature/add-cysignals-dependency

Conversation

@keltecc

@keltecc keltecc commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

fpylll requires cysignals at runtime, but the published wheels do not declare it as a dependency

$ pip install fpylll
$ python3 -c "import fpylll"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
    import fpylll
  File "/usr/local/lib/python3.14/dist-packages/fpylll/__init__.py", line 3, in <module>
    from .fplll.integer_matrix import IntegerMatrix
  File "src/fpylll/fplll/integer_matrix.pyx", line 1, in init fpylll.fplll.integer_matrix
    # -*- coding: utf-8 -*-
ModuleNotFoundError: No module named 'cysignals'

fix: add cysignals as a dependency in setup.py

@malb malb merged commit 516ea76 into fplll:master Jun 15, 2026
37 checks passed
@malb

malb commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Ta!

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.

2 participants