OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
292 stars 520 forks source link

[16.0] cryptography wrong version in modules manifest #3507

Closed iTecan closed 7 months ago

iTecan commented 8 months ago

Module

l10n_es_aeat l10n_es_facturae

Describe the bug

There is a wrong version in the manifest from the modules, being in branch 15.0 version not requiered an specific version but in branch 16.0 is required version 3.4.8. Even if this cryptography version is installed, the modules can't be installed given you an error message that requires the version already installed

To Reproduce

Affected versions: 16.0

Steps to reproduce the behavior:

  1. Install Odoo 16
  2. Install requirements and l10n_spain
  3. Try to install the modules

Expected behavior The modules should be installed

Additional context In 15.0 the manifest from l10n_es_aeat is the following: "external_dependencies": {"python": ["unidecode", "cryptography"]},

While in 16.0 is the following: "external_dependencies": {"python": ["unidecode", "cryptography==3.4.8"]},

pedrobaeza commented 8 months ago

@etobella si no recuerdo mal, lo incluiste tú para evitar una incompatibilidad.

etobella commented 8 months ago

No entiendo el problema :thinking: que tiene que ver el manifest de la 15 si estas instalando 16.

De hecho, usamos la misma versión que define Odoo en sus requirements. https://github.com/odoo/odoo/blob/16.0/requirements.txt#L5

Así que para mi no hay ningún bug.

Debemos recordar que por eso hay cambio de versiones... Las dependencias pueden cambiar entre versiones mayores (lo intentamos evitar en menores...)

iTecan commented 8 months ago

@etobella la indicación a la rama 15 es por indicar las diferencias entre una rama que no da ningún problema (la cual en su manifest no solicita ninguna version de cryptography en concreto) realizando la instalación con una que si las esta dando.

La única diferencia apreciable entre versiones es la obligación de versión de la dependencia que aunque estando la misma versión que requiere odoo y en el sistema la misma, esa indicación de version es la que genera el fallo que salta el error de que no esta instalada (aunque si lo esta)

Como se ha indicado, los pasos han sido los siguientes:

Instalación de odoo 16 CE con sus requirements, su funcionamiento es correcto Descarga del repositorio l10n_spain e instalación de sus requirements, cuando se va a los modulos mencionados salta el mensaje de que esa versión no esta instalada.

Si eliminamos del manifest la versión, dejando solo indicado la dependencia cryptography funciona correctamente la instalación

Queremos dejar claro que este problema no es a nivel del fichero requirements.txt sino en los archivos manifest.py

etobella commented 8 months ago

El problema esta en tu instalacion. Odoo requiere esa version, por eso esta congelado.

pedrobaeza commented 7 months ago

@iTecan efectivamente si se instala con las versiones indicadas tanto en el requirements.txt de Odoo como en el manifest, todo funciona correctamente, así que cierro. Revisa por favor tu procedimiento de despliegue para ver si estás instalando por otro lado una versión sin congelar de cryptography, y tendrás que indicar dicha versión. Me suena que utilizas Doodba, así que sería en el archivo pip.txt indicar cryptography==3.4.8 en lugar de solo cryptography.

Cierro entonces la issue.