README erweitert und example config geändert um Verwirrung zu vermeiden

die example config listet wird ja nicht ausgelesen und listete bis eben als default, dass Daten nicht initiiert werden. Das verwirrt, weil ohne config das Default ist, die Daten mit zu initiieren.

Statt das in der README zu beschreiben ist es sicherlich besser, das Default zu ändern ;)

Außerdem hab ich noch Infos hinzugefügt, wie man den Upstream konfiguriert.

Mentioned in issue #77 (closed)

Added 1 commit:

• caace678 - Konfigurierbarkeit der IP-Adresse, auf der gelauscht wird

Hi,
merke gerade, dass ich für unterschiedliche Mergerequests unterschiedliche branches verwenden muss
der hat den zweiten Commit direkt nachgezogen

der ist an sich aber auch ganz sinnvoll

Er erlaubt es die Server-IP von 127.0.0.1 in der Config auf 0.0.0.0 zu ändern, damit man mit anderen Geräten im Netzwerk testen kann
Die Config wird man dann später auch bei der Liveschaltung brauchen können.

EDIT:
ich plane noch, die README später zu erweitern, um zu zeigen, wie man auch aus einer Virtualbox per Netzwerkbrücke den Server im Netzwerk erreichbar zu machen

Die Config wird man dann später auch bei der Liveschaltung brauchen können.

Im Livebetrieb verwenden wir nicht die run.py, sondern uwsgi direkt (siehe die nginx config, die ich vor Kurzem hinzugefügt hatte). Zum Debuggen finde ich die Option aber ganz sinnvoll.

Ich denke aber nicht, dass wir im README eine Anleitung für irgendein VM-Setup brauchen. Das ist viel zu speziell.

ok dann kannst du das ganze ja mergen
Die Anleitung für ein VM Setup hab ich ja noch gar nicht erstellt

Hat es einen Grund, warum du nur mache der Python-Pakete von python- in python3- umbenennst? Ich bin mir nicht so sicher, ob man sich an der Stelle an Debian anpassen sollte. In anderen Distros ist python3 schon länger der Standard.

Ohne diese Paket-Umbenennungen würde ich das mergen.

ähm, ich nutze Ubuntu, da scheint 2 auch noch der Standard zu sein alle Pakete bei denen keine 3 steht, gibt es in dieser Version nicht über apt

ich dachte, dass würde bedeuten, dass sie dann für beide Versionen funktionieren

wenn dem nicht so ist, muss man sie wohl über pip3 laden, kann das sein? wenn ja, werde ich das natürlich anpassen

außerdem betrifft das wie ich gerade sehe nur das ldap Paket den mysql connector lade ich ja schon über pip3, dann wird er ja wohl in version 3 vorliegen

Die beiden Pakete heißen bei Debian (bzw. Ubuntu, das mach keinen großen Unterschied): python3-ldap3, python3-mysql.connector

Bei Archlinux heißt python3-mysql.connector aber python-mysql-connector. Das wird also schenll sehr kompliziert, wenn man sich an irgendwelche Distros anpasst. Mein Vorschlag wäre: Reverte die Änderungen bei der Paketliste und vervollständige die "Kurzform unter Ubuntu"-Zeile.

Wobei man das ldap-Paket wirklich in python-ldap3 umbenennen sollte.

Added 1 commit:

• 8c638533 - gewünschter Fix

Wie ausgeführt, brauch man unter Debian/Ubuntu eigentlich kein pip dafür.

Added 1 commit:

• b116b3c4 - der echte Fix

ja, hab deine Nachricht erst nach dem "gewünschten" Fix gelesen ^ ^

hab nur die Schreibweisen aus eurem README genommen und deshalb nicht die echten Pakete gefunden

jetzt solltest du eigentlich mergen können

Wie gesagt, ich will in dieser Liste nicht die Debian/Ubuntu-Paketnamen sondern die Projektnamen bzw. die pip-Paket namen (also python-ldap3 und python-mysql-connector), da jede Distro andere Schreibweisen verwendet.

Bevor ich mich weiter wiederhole mache das lieber selbst. Ich merge das gleich ...

Status changed to merged

achso jetzt hab ichs auch verstanden, naja egal
Danke! :)

Ein schönes Wochenende noch.
Ich hör für heute auf.

Mentioned in merge request !5 (closed)