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

Ajout d'un script d'installation pour Windows #5390

Merged
merged 12 commits into from
Feb 23, 2020

Conversation

A-312
Copy link
Contributor

@A-312 A-312 commented Aug 6, 2019

fix: #5388 Le script est prêt!

Q/A :

Je vous conseille d'utiliser python 3.6.8 : https://www.python.org/ftp/python/3.6.8/python-3.6.8-amd64.exe

Avec git pr :

  • git pr 5390
  • git checkout pr/5390
  • cd /ton/path/zds-site/
  • powershell .\scripts\win\install_zds.ps1 +full

@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling d7148ee on A-312:install-zds_win into dd74144 on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 6, 2019

Coverage Status

Coverage remained the same at ?% when pulling b78165a on A-312:install-zds_win into 180cd29 on zestedesavoir:dev.

@A-312
Copy link
Contributor Author

A-312 commented Aug 6, 2019

J'ai utilisé python 3.6.8 : https://www.python.org/ftp/python/3.6.8/python-3.6.8-amd64.exe

@A-312 A-312 force-pushed the install-zds_win branch 2 times, most recently from d20d28f to 7c3ed78 Compare August 6, 2019 18:47
@A-312
Copy link
Contributor Author

A-312 commented Aug 6, 2019

scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
Copy link
Contributor

@helmasaur helmasaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour plus de clarté, il vaut mieux ajouter un nom aux processus de zmarkdown (surtout si le fichier s'appelle index.js).

scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
doc/source/install/install-windows.rst Outdated Show resolved Hide resolved
scripts/win/install_zds.ps1 Outdated Show resolved Hide resolved
@A-312
Copy link
Contributor Author

A-312 commented Aug 12, 2019

Il faudra que je mette yarn au passage mais j'avais peur que ça bug sous windows. :S

@helmasaur
Copy link
Contributor

C'est possible d'installer yarn en local plutôt qu'en général si ce dernier n'est pas présent ?

@A-312
Copy link
Contributor Author

A-312 commented Aug 14, 2019

C'est possible d'installer yarn en local plutôt qu'en général si ce dernier n'est pas présent ?

Trop compliqué, il faudrait l'ajouter au virtualenv.

@helmasaur
Copy link
Contributor

yarn peut être installé de plusieurs façons d'où ma question. Si c'est ajouté à l'environnement virtuel, hormis le potentiel doublon, ça ne créera pas de conflits.

@A-312
Copy link
Contributor Author

A-312 commented Aug 14, 2019

Si je mets yarn ça sera avec npm install -g yarn, je ne vais pas me prendre la tête à faire autrement ce qui sera le cas si je sors de l'installation normal présente dans la doc. De toute façon, on a NPM donc je ne vais pas gaspiller du temps à essayer de scoper yarn, ce qui a très peu d'intérêt en soit et source de problème.

@helmasaur
Copy link
Contributor

Et si la personne a installé yarn d'une autre manière, ça fera quoi ?

@A-312
Copy link
Contributor Author

A-312 commented Aug 14, 2019

Et si la personne a installé yarn d'une autre manière, ça fera quoi ?

Rien c'est son espace de dev, il le customise comme il veut.

@gustavi
Copy link
Contributor

gustavi commented Aug 25, 2019

Il faut changer Python 3.6.8 par 3.6.9 qui est la dernière version en date.

@A-312
Copy link
Contributor Author

A-312 commented Aug 26, 2019

@gustavi Je n'ai pas trouvé d'installateur pour Windows ou en tout cas, l'installation et le téléchargement semble plus complexe. À moins, la présence d'un fix de sécurité, la 3.6.8 convient

@gustavi
Copy link
Contributor

gustavi commented Aug 26, 2019

@A-312
Copy link
Contributor Author

A-312 commented Nov 14, 2019

Je dois intégrer #5480 ou dans une autre PR.

@A-312
Copy link
Contributor Author

A-312 commented Dec 16, 2019

Cette PR va prendre froid :( , ça fait 2-3 mois qu'elle est prête. 😕

@helmasaur
Copy link
Contributor

Je pensais qu'elle a été déjà testé. Je veux bien faire la PR mais il faudrait me dire ce que je dois faire ou vérifier.

J'imagine que je dois installer un nouveau projet ZdS sur Windows à l'aide de ce script.

@A-312
Copy link
Contributor Author

A-312 commented Dec 19, 2019

@helmasaur Oui normalement ça ne rempli pas les fichiers temporaires globaux du système, donc il n'y a pas vraiment de contraintes.

@A-312 A-312 force-pushed the install-zds_win branch 4 times, most recently from 145b9d1 to a932c0f Compare January 19, 2020 20:18
@A-312
Copy link
Contributor Author

A-312 commented Jan 26, 2020

Il faudra Q/A sous linux l'installation de pm2 : #5390

  1. npm remove pm2 -g
  2. Essayer: pm2 qui devrait retourner command not found
  3. Ouvrir le zdsenv
  4. Supprimer package-lock et node_modules dans le dossier zmd/
  5. faire make zmd-install à la racine de zds-site/
  6. Lancer zmd zmd-start
    • Si 6. ne fonctionne pas (et que pm2 est introuvable) ajouter : export PATH="$PATH:$ZDS_ENV/../zmd/node_modules/pm2/bin" dans la commande du makefile, et refaire make zmd-install (et me dire que c'est nécessaire)

@philippemilink
Copy link
Member

Je viens d'essayer l'installation sous Linux, mais en partant d'une nouvelle installation. J'ai donc fait:

  1. cloner ton dépôt et utilisation de cette branche
  2. npm remove pm2 -g
  3. pm2 est inconnu: ✔️
  4. make install-linux
  5. sourcer l'environnement Python
  6. make zmd-start: ✔️
  7. make run-back: le site fonctionne correctement.

La version Linux (debian 10) du script d'installation semble donc correcte.

Copy link
Contributor

@helmasaur helmasaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fonctionne sur mon ordinateur portable.

@A-312 A-312 merged commit a4bec36 into zestedesavoir:dev Feb 23, 2020
@A-312 A-312 deleted the install-zds_win branch April 12, 2020 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants