Derniers messages sur Zeste de Savoirhttps://zestedesavoir.com/forums/2015-04-28T13:05:36+02:00Les derniers messages parus sur le forum de Zeste de Savoir.geoip vs pygeoip, message #542812015-04-28T13:05:36+02:00SpaceFox/@SpaceFoxhttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p54281<p>Puisqu'il n'y a rien de clair sur Github et que personne n'arrive à retrouver le pourquoi du comment, fais une PR de revert. Au pire on re-basculera s'il y a une vraie raison derrière.</p>geoip vs pygeoip, message #542762015-04-28T12:08:14+02:00firm1/@firm1https://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p54276<figure><blockquote>
<p>Je suis pas sur (et la flemme de chercher sur portable) mais il y a pas une histoire de support de python3 ? (Anticipation toussa)
</p>
</blockquote>
<figcaption><p><a href="http://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53797">Eskimon</a></p></figcaption></figure><p>pygeoip est bien <a href="https://travis-ci.org/appliedsec/pygeoip/builds/39196535">compatible python 3</a>.</p>
<p>Sinon j'ai l'impression que gustavi est un peu injoignable en ce moment, et je ne sais pour combien de temps. ça fait 2 semaines qu'il ne s'est pas connecté sur le site (malgré mon MP <img alt=":(" src="/static/smileys/triste.png"> ).</p>
<p>En relisant les discussions github pour essayer de savoir le pourquoi du comment de ce passage, je ne trouve rien de pertinent.</p>
<p>On fait quoi, on attend toujours ?</p>geoip vs pygeoip, message #537972015-04-25T20:56:17+02:00Eskimon/@Eskimonhttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53797<p>Je suis pas sur (et la flemme de chercher sur portable) mais il y a pas une histoire de support de python3 ? (Anticipation toussa)</p>geoip vs pygeoip, message #533872015-04-23T21:46:13+02:00SpaceFox/@SpaceFoxhttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53387<figure><blockquote>
<p>tu peux pas attendre la justification de Gustavi ?
</p>
</blockquote>
<figcaption><p><a href="http://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53386">Kje</a></p></figcaption></figure><p>Ouais, j'aimerais bien avoir l'explication.</p>geoip vs pygeoip, message #533862015-04-23T21:30:45+02:00Kje/@Kjehttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53386<figure><blockquote>
<p>Je vais voir si je peux faire une PR de revert
</p>
</blockquote>
<figcaption><p><a href="http://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53324">firm1</a></p></figcaption></figure><p>tu peux pas attendre la justification de Gustavi ?</p>geoip vs pygeoip, message #533842015-04-23T21:12:04+02:00pierre_24/@pierre_24https://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53384<p>J'appuie fortement cette proposition, ne serait-ce que pour la doc.</p>geoip vs pygeoip, message #533342015-04-23T18:02:11+02:00firm1/@firm1https://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53334<figure><blockquote>
<p>Remarque au passage, quelqu'un a pensé a regarder si ils avaient pas la même interface ?
</p>
</blockquote>
<figcaption><p><a href="http://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53332">Kje</a></p></figcaption></figure><p>Pas la même interface. Et il n'y a pas vraiment d’intérêt de supporter les deux libs.</p>geoip vs pygeoip, message #533322015-04-23T17:50:25+02:00Kje/@Kjehttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53332<p>Remarque au passage, quelqu'un a pensé a regarder si ils avaient pas la même interface ? Car si oui on peut faire un :</p>
<table class="codehilitetable"><tr><td class="linenos"><div class="linenodiv"><pre>1
2
3
4</pre></div></td><td class="code"><div class="codehilite"><pre><span class="k">try</span><span class="p">:</span>
<span class="kn">import</span> <span class="nn">geoip</span>
<span class="k">except</span> <span class="ne">ImportError</span><span class="p">:</span>
<span class="kn">import</span> <span class="nn">pygeoip</span> <span class="kn">as</span> <span class="nn">geoip</span>
</pre></div>
</td></tr></table>
<p>et comme ça tout le monde est content</p>geoip vs pygeoip, message #533242015-04-23T17:37:38+02:00firm1/@firm1https://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53324<p>Bon du coup je ne vois pas de raison particulière de s'imposer geoip.</p>
<p>Je vais voir si je peux faire une PR de revert</p>geoip vs pygeoip, message #533172015-04-23T17:27:37+02:00Kje/@Kjehttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53317<p>Vivement que je puisse d'inclure numpy en dépendance que tu comprenne ce que "dépendance a compiler chiante veux dire" <img alt=":D" src="/static/smileys/heureux.png"></p>
<p>Plus sérieusement, pour windows au moins, ils existes des versions précompilé de beaucoup de lib python. </p>geoip vs pygeoip, message #533142015-04-23T17:15:54+02:00SpaceFox/@SpaceFoxhttps://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53314<p>Il faudrait demander directement à Gustavi, c'est lui qui a fait <a href="https://github.com/SpaceFox/zds-site/commit/f528058e0cceed08f8747bf7b2893ab9e4fa0676">ce commit</a> lequel a impliqué <a href="https://github.com/SpaceFox/zds-site/commit/0243b47e1819adefb1b655964203c17e8eb23102">cet autre commit</a>.</p>
<p>Avec un peu de chance, on peut revert le 2ème et Travis est d'accord… à voir.</p>geoip vs pygeoip, message #532922015-04-23T16:03:53+02:00firm1/@firm1https://zestedesavoir.com/forums/sujet/2962/geoip-vs-pygeoip/?page=1#p53292<p>Hello les jardiniers,</p>
<p>Depuis que la branche de dev à été taillée pour laisser place à Django 1.7, nous avons changé de dépendance pour la géolocalisation. Nous sommes passés grosso modo de la librairie <a href="https://github.com/appliedsec/pygeoip">pygeoip</a> à la librairie <a href="https://github.com/maxmind/geoip-api-python">geoip</a>.</p>
<p>Je ne sais pas trop si c'est une contrainte de django 1.7, mais le fait est que pygeoip qui fonctionnait bien jusqu'ici, avait le gros avantage d'être écrit en python pur. Ce qui signifie qu'il n'avait pas besoin de binding C. Aujourd'hui avec <code>geoip</code> on doit rajouter une dépendances système (<code>apt-get install geoip</code>). Ce qui nous ramène les problèmes suivants : </p>
<ul>
<li>Notre documentation sous ReadTheDocs, <a href="https://readthedocs.org/builds/zds-site/2615814/">ne compile plus</a> correctement parce que le pauvre n'a pas la fameuse dépendance système.</li>
<li>On alourdi la documentation d'installation sur les environnements (et c'est encore pire sur windows car ça nécessite d'avoir un compilateur C++ à disposition).</li>
</ul>
<p>J'ai cherché et je n'ai pas trouvé d’intérêt à ce passage à geoip, je me renseigne donc pour savoir si c'était nécessaire ? Auquel cas, je pense qu'il faudrait revenir à pygeoip</p>