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

Suppression du support de NVM #1726

Closed
jpm-cbna opened this issue Feb 18, 2022 · 3 comments
Closed

Suppression du support de NVM #1726

jpm-cbna opened this issue Feb 18, 2022 · 3 comments

Comments

@jpm-cbna
Copy link
Contributor

jpm-cbna commented Feb 18, 2022

Version
develop

Description du bug
L'utilisation de NVM a été supprimé du fichier install/04_install_frontend.sh dans le commit suivant.
La documentation sur les dépendances nécessaire à GeoNature n'a pas été mise à jour et le fichier frontend/.nvmrc est toujours présent.
À noter que la suppression de l'utilisation de NVM risque de poser des problèmes. Les paquets Debian sont loin d'avoir toutes les versions de Node disponible. A priori, Node v12.22.5 dans Bullseye et v10.24.0 dans Buster.
Nous utilisons actuellement la version v16.13.2 via NVM.
Angular CLI nécessite les versions v12.14 ou v14.15.
Du coup, cela risque de poser problème avec Buster...

Comportement attendue
Idéalement, le script 04_install_frontend.sh devrait indiquer que Node nécessite d'être mis à jour. Ou mieux détecter la présence de NVM et l'utiliser dans ces cas là...

Comment reproduire
Essayer de lancer le script install/04_install_frontend.sh avec une version inférieure de Node (par exemple v10.15.3).
Puis essayer de lancer la commande ng dans le frontend. Elle n'existe pas à cet emplacement ./node_modules/.bin/ng
Mettre à jour vers la version v16.13.2 puis relancer 04_install_frontend.sh résout le problème.

@jpm-cbna jpm-cbna added the bug label Feb 18, 2022
@Gaetanbrl
Copy link
Contributor

Gaetanbrl commented Feb 18, 2022

L'utilisation d'apt (linux) est en général évité pour l'install de nodejs.
Taxhub et Usershub sont surtout dépendants de nodejs 10.x (avec la version npm + les paquets liés) alors qu'une version >=14x est utilisée pour l'install GeoNature.

Je fait un lien avec cette issue PnX-SI/UsersHub#149

Pour notre dernière install (develop) on a bien utilisé NVM qui permet bien le switch entre les modules.

@camillemonchicourt
Copy link
Member

Fait dans e9dba8a

@camillemonchicourt
Copy link
Member

OK dans 2.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants