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

Modified abs reference of pycryptodomex==3.11.0 to nearest compatible. #73

Closed
wants to merge 1 commit into from

Conversation

loucksg
Copy link

@loucksg loucksg commented Dec 27, 2023

aiming to fix compatibility problem with the python crypto library install within the docker container.

@kilador
Copy link

kilador commented Jan 16, 2024

I tried your solution: changing "==" by "~=" but still get the same error. Any other idea?

@loucksg
Copy link
Author

loucksg commented Jan 17, 2024 via email

@oleg-d
Copy link

oleg-d commented Feb 4, 2024

Can this be merged and new release made with this fix? There's quite a few people waiting to try this integration out as the GoSungrow one has been broken.

@roabyd
Copy link

roabyd commented Feb 7, 2024

@oleg-d unfortunately this PR would not fix this issue (I cloned the repo and tried updating it myself). It looks like there is a bigger issue here as per @loucksg's comment. Here's hoping a fix can be found soon

@roabyd
Copy link

roabyd commented Feb 7, 2024

image

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.

5 participants