From b1cc4ce906bbbe378cbc6c44dac123cc01a9e615 Mon Sep 17 00:00:00 2001 From: Emilio Serna Date: Tue, 6 Jun 2023 19:49:03 +0000 Subject: [PATCH] [FIX] letsencrypt: remove `cryptography` from python depends Such dependency is already included in Odoo requirements using a pinned version [1]. Adding here could cause to upgrade the library to an incompatible version. [1] https://github.com/odoo/odoo/blob/710a2b2a7af68e8f2f249ef9fc3146f44d3266a5/requirements.txt#L3 --- letsencrypt/__manifest__.py | 4 +--- requirements.txt | 1 - 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/letsencrypt/__manifest__.py b/letsencrypt/__manifest__.py index 69e6d0f73..25173b794 100644 --- a/letsencrypt/__manifest__.py +++ b/letsencrypt/__manifest__.py @@ -17,7 +17,5 @@ "demo": ["demo/ir_cron.xml"], "post_init_hook": "post_init_hook", "installable": True, - "external_dependencies": { - "python": ["acme<2.0.0", "cryptography", "dnspython", "josepy"] - }, + "external_dependencies": {"python": ["acme<2.0.0", "dnspython", "josepy"]}, } diff --git a/requirements.txt b/requirements.txt index 3265c19ca..dd70bcba9 100644 --- a/requirements.txt +++ b/requirements.txt @@ -1,7 +1,6 @@ # generated from manifests external_dependencies acme<2.0.0 astor -cryptography cryptography==2.6.1 dataclasses dnspython