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

Révision de la procédure d'installation windows #5211

Merged
merged 1 commit into from May 1, 2019
Merged

Conversation

artragis
Copy link
Member

@artragis artragis commented Jan 4, 2019

J'ai pu tester tout ce qui est noté sauf la partie elastic.

désolé pour la branche sur le dépôt, j'ai fait une mauvaise manip, mais sans trop de conséquence.

@artragis artragis added the C-Docs Concerne la documentation label Jan 4, 2019
@coveralls
Copy link

Coverage Status

Coverage increased (+14.9%) to 89.242% when pulling 5161f91 on update_windows_doc into 9f4f36a on dev.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+14.9%) to 89.242% when pulling 5161f91 on update_windows_doc into 9f4f36a on dev.

@coveralls
Copy link

coveralls commented Jan 4, 2019

Coverage Status

Coverage remained the same at 89.171% when pulling c339c8e on update_windows_doc into 1265876 on dev.

sudo apt install build-essential git
git clone https://github.com/<votre-pseudo>/zds-site
cd zds-site
./install-linux -packages ubuntu
Copy link
Member

Choose a reason for hiding this comment

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

à priori, le ubuntu à la fin ne fait rien, à moins que #5128 ne soit modifiée ;)

Copy link
Member

@pierre-24 pierre-24 Jan 4, 2019

Choose a reason for hiding this comment

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

(du coup le -package signifie "n'installe pas les packages", donc ... il manque pas des trucs ?)

Copy link
Member Author

Choose a reason for hiding this comment

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

ah, peut être que c'est "+packages", ça fait un moment quej e l'ai fait. Et le sous-système linux de windows ne permet pas de lancer make install-linux

Copy link
Member

Choose a reason for hiding this comment

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

c'est bien +packages, à mon avis.

Copy link
Member Author

Choose a reason for hiding this comment

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

eh bah non c'est -packages si on se fit à l'erreur qu'il donne

@Situphen
Copy link
Member

Situphen commented Jan 5, 2019

Il y a des contributeurs sous Windows actuellement ? Si ce n'est pas le cas, je me demande si ça vaut le coup de tenir ça à jour. Il n'est déjà pas simple d'améliorer l'installation sur les distributions Linux, alors si on doit avoir comme contrainte que ça fonctionne sous Windows... :/

@pierre-24
Copy link
Member

@artragis dévellope régulièrement sous windows ^^

@artragis
Copy link
Member Author

artragis commented Jan 5, 2019

Oui @Situphen il y a moi.

@artragis
Copy link
Member Author

On peut merger ici?

@artragis artragis added this to in review in Objectif v28.2 Apr 17, 2019
@artragis artragis merged commit 3681389 into dev May 1, 2019
Objectif v28.2 automation moved this from in review to Done May 1, 2019
@artragis artragis deleted the update_windows_doc branch May 1, 2019 14:37
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
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants