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

Documentation installation sous windows #3446

Closed
wants to merge 6 commits into from
Closed

Documentation installation sous windows #3446

wants to merge 6 commits into from

Conversation

josephcab
Copy link
Contributor

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés Aucun

QA

Aucun

Objet

Correction (partielle) de la documentation pour l'installation de zds sur un ordinateur muni de Windows

Proposition de correction de la procédure d'installation sous Windows, après une installation laborieuse chez moi

.. Attention::

Il est fortement recommandé (pour des raisons de compatibilité), que tous les programmes à installés le soit en 32 bits.
Copy link
Member

Choose a reason for hiding this comment

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

à installer

@Situphen Situphen added the C-Docs Concerne la documentation label Mar 16, 2016
@Situphen
Copy link
Member

À part les quelques points à modifier ça m'a l'air bon ! :)

@SpaceFox
Copy link
Contributor

Est-ce que tu saurais faire un squash de tout ça @josephcab ? 5 commits, ça fait un peu beaucoup pour cette modification de doc :)

À noter que si tu ajoutes [ci-skip] dans le nom du commit, Travis ne passera pas (il est inutile ici).


.. Attention::

Il est fortement recommandé (pour des raisons de compatibilité), que tous les programmes à installer le soit en 32 bits.
Copy link
Contributor

Choose a reason for hiding this comment

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

Je suis toujours en désaccord avec cette ligne. Puis-je avoir une justification ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@SpaceFox , je ne sais pas faire, mais je peux essayer si tu m'explique.

@gustavi , en suivant exactement la même procédure avec les programmes en 64bits, j'ai eu des plantage à répétitions et n'ait jamais réussi à finir d'installer le site. Mais si vous pensez qu'il vaut mieux ne pas le mettre, aucun problème, je l'enlève.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oui enlève-le, on est en 2016.

Copy link
Member

Choose a reason for hiding this comment

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

Je suis d'accord qu'on est en 2016, mais il faut préciser que la Doc fonctionne avec des programmes 32bit et qu'on n'est pas sur pour 64bit.

Copy link
Contributor

Choose a reason for hiding this comment

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

Personnellement, à l'époque où j'avais testé sous Windows j'étais en 64 bits et ça passait très bien.

Copy link
Contributor

Choose a reason for hiding this comment

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

J'ai l'impression que vous parlez de 2 choses différentes là… si les programmes doivent être installés en 32 bits même sur un Windows 64 bits, ça doit être précisé. Même si c'est dommage, je préfère avoir une doc qui explique comment faire tourner ZdS en 32 bits sous Windows, que pas de doc du tout.

Copy link
Member

Choose a reason for hiding this comment

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

+1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

je fais quoi?

Copy link
Contributor

Choose a reason for hiding this comment

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

Je pense que Il est fortement recommandé (pour des raisons de compatibilité), que tous les programmes à installer le soient en 32 bits ou en 64 bits. Il est préférable de ne pas mélanger les deux. est bien.

@gustavi
Copy link
Contributor

gustavi commented Mar 29, 2016

À part les commentaires sur le 32/64 bits et les quelques fautes ça me semble correcte.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 87.889% when pulling dff9824 on josephcab:dev into 9a844fb on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 87.889% when pulling 8de22a3 on josephcab:dev into 9a844fb on zestedesavoir:dev.

@josephcab
Copy link
Contributor Author

j'ai essayé de faire un squash, mais sans succès, si ce n'est deux commit supplémentaires inutiles (les deux derniers)!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 87.889% when pulling 2ac0020 on josephcab:dev into 9a844fb on zestedesavoir:dev.

@gustavi
Copy link
Contributor

gustavi commented Mar 30, 2016

Tu peux essayer ça :

git reset --soft 41a1f5caf79928ad51bb08841d42ddf307099838
git add -A
git commit -m "Mise à jour de la documentation d'installation sous Windows"
git push -f origin dev

@josephcab
Copy link
Contributor Author

Merci beaucoup gustavi.

Juste pour essayer de comprendre :

  • je réinitialise mes commits avec la première ligne en désignant le plus vieux à reseter.
  • J'ajoute toutes les modifs encores non commitées (vu que j'ai reseté)
  • je crée mon commit avec toutes les modifs
  • j'envoie sur zds

C'est ça?

@gustavi
Copy link
Contributor

gustavi commented Apr 1, 2016

Pour te répondre :

git reset --soft 41a1f5caf79928ad51bb08841d42ddf307099838 # tu reviens au commit 41... sans annuler les changements sur les fichiers
git add -A # tu ajoutes tous les fichiers modifiés au prochain commit
git commit -m "Mise à jour de la documentation d'installation sous Windows" # tu fais le commit
git push -f origin dev # tu push avec -f car l'historique des commits a été modifié

Il faut néanmoins que tu rebase ta branche car il y a du conflit. Sais-tu faire ça ?

@josephcab
Copy link
Contributor Author

pas du tout (je ne sais en fait pas me servir de git).

Au passage, si tu as un tuto, je suis preneur!

@coveralls
Copy link

Coverage Status

Coverage increased (+16.5%) to 87.401% when pulling 14aa1ec on josephcab:dev into 504cc3c on zestedesavoir:dev.

@Situphen
Copy link
Member

Situphen commented Apr 2, 2016


Vérifiez de posséder la version 2.5.0 de npm (et non la version 1.4.X que peut installer l'installeur), pour connaitre votre version : ``npm -v``. Pour mettre à jour npm, vous avez `trois possibilités <https://github.com/npm/npm/wiki/Troubleshooting#upgrading-on-windows)>`_, l'une d'elle est de faire ``npm install npm`` dans le dossier ``C:\Program Files (x86)\nodejs``.
Vérifiez de posséder la version 2.5.0 de npm (et non la version 1.4.X que peut installer l'installeur), pour connaitre votre version : ``npm -v``.
Copy link

Choose a reason for hiding this comment

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

Il peux être utile d'indiquer que la commande : npm install npm@latest effectuera les mise à jour

@@ -41,6 +49,7 @@ Prérequis
- ``pip install virtualenvwrapper-powershell``
- Désactivez la sécurité sur les script powershell (mode administrateur obligatoire) : ``Set-ExecutionPolicy RemoteSigned``, pour autoriser ``Import-Module``.
- Créez votre workspace dédié à ZdS (mode administrateur obligatoire).
- Créez le dossier ``.virtualenvs`` avec la commande ``mkdir .virtualenvs`` dans le dossier où vous souhaitez voir installer votre environnement virtuel.
- ``set $env:WORKON_HOME``
Copy link

Choose a reason for hiding this comment

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

C'est vraiment évident qu'il faut rentrer dans le dossier .virtualenvs que l'on vient de créer ou c'est moi qui suis bête?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

non, ce n'est pas nécessairement évident, m'étant aussi fait avoir!

Copy link
Contributor

Choose a reason for hiding this comment

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

N'hésitez pas à donner un max d'infos sur vos retours, même si ça semble stupide !

@orandin
Copy link
Contributor

orandin commented Apr 4, 2016

Après les commandes pour installer virtualenv, il est possible de procéder autrement :

  • cloner le dépôt git
  • Se placer dans le dossier du dépôt cloné zds-site/
  • Taper virtualenv zdsenv pour créer le dossier zdsenv contenant l'environnement.
  • Exécuter le fichier zdsenv/bin/activate.ps1 pour activer l'environnement.

Cela me semble davantage plus simple et plus stable que les commandes proposées.

@gustavi
Copy link
Contributor

gustavi commented Apr 19, 2016

@josephcab il faudrait rebase tout ça ! Tu as des liens avec de la documentation ci dessus, au besoin n'hésite pas ;)

@josephcab
Copy link
Contributor Author

josephcab commented Apr 19, 2016

@orandin j'ajoute les commandes à partir du clone du dépot zds, mais faut-il supprimer une ou plusieurs lignes avant?

et je n'ai pas réussi pour le rebase, j'ai encore bien du mal avec git!

@@ -41,6 +49,7 @@ Prérequis
- ``pip install virtualenvwrapper-powershell``
- Désactivez la sécurité sur les script powershell (mode administrateur obligatoire) : ``Set-ExecutionPolicy RemoteSigned``, pour autoriser ``Import-Module``.
- Créez votre workspace dédié à ZdS (mode administrateur obligatoire).
- Créez le dossier ``.virtualenvs`` avec la commande ``mkdir .virtualenvs`` dans le dossier où vous souhaitez voir installer votre environnement virtuel.
- ``set $env:WORKON_HOME``
- ``Import-Module virtualenvwrapper``, *la console PowerShell pourrait hurler à cette commande, mais normalement ça passe, en cas de deuxième essai vous ne devriez plus avoir d'erreur*.
- ``New-VirtualEnvironment zdsenv --no-site-packages``
Copy link
Contributor

@orandin orandin Apr 19, 2016

Choose a reason for hiding this comment

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

Pour moi, les lignes de 52 à 55 rendent l'installation compliquée et vraiment particulière vis-à-vis de Linux par exemple. Alors qu'il est possible d'avoir sensiblement la même chose en une ligne : virtualenv zdsenv

Il faut donc penser à rajouter vers la ligne 57 cette instruction :

Copy link
Contributor

Choose a reason for hiding this comment

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

D'autant que quand j'étais sous Windows, ça ne fonctionnait pas.

@josephcab
Copy link
Contributor Author

Suite à la catastrophe que devenait mon fork a cause de mes tentatives échouées de rebase, et autres, j'ai recréé mon fork et donc une PR ici.

Cette PR peut donc être supprimée, tous les changements ayant été repris sur l'autre, avec vos dernières remarques.

@SpaceFox
Copy link
Contributor

OK, je ferme ici donc.

@SpaceFox SpaceFox closed this Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Docs Concerne la documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants