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

[patch] correctifs ompilation des exercices du bouquin #1

Open
ebachard opened this issue Jan 12, 2018 · 1 comment
Open

[patch] correctifs ompilation des exercices du bouquin #1

ebachard opened this issue Jan 12, 2018 · 1 comment

Comments

@ebachard
Copy link

ebachard commented Jan 12, 2018

Bonjour,

Sans connaître le bouquin, et tombé par hasard sur ce dépôt, j'ai essayé les différents exercices proposés. Comme il y avait quelques petits problèmes de compilation, je me suis permis d'en corriger certains (principalement sous Linux x86_64)

=> voir le patch en attachement. N.B. : il est au format .txt, car j'avais la flemme de cloner le dépôt et de faire un PR ;-)

Félicitation pour le livre, et meilleurs voeux pour 2018 :-)

correctifs_compilation_exos_bouquin_lberger.diff.txt

P.S. : tout ne compile pas, il reste un ou deux problèmes je crois.

@LaurentBerger
Copy link
Contributor

Bonsoir,
Je vous réponds avec un peu de retard.
Merci pour l'intérêt que vous portez à ce livre.

Pour les corrections que vous apportez à propos de cx11, je pense que vous avez une version de compilateur antérieure à gcc version 6.3.0 ou bien est-ce cmake qui crée le problème. Je ne sais pas. Je pense que je vais créer temporairement une branche avec vos modifs, voir si cela marche avec un compilateur récent et sous windows et fusionner un peu plus tard....

Par contre vous avez trouvé quelques caractères unicode qui bloquait la compilation : je vais rapidement les enlever. Pour l'espace après ADD_EXECUTABLE j'avoue que je ne comprends pas cela marche et quelquefois cela bloque. Je suis perplexe.

Encore merci pour vos remarques.

Je ne ferme pas l'issue car je vais essayer d'intégrer vos remarques dans la semaine.

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

No branches or pull requests

2 participants