search

Ston3s / plugin.video.xstream

Video Addon for XBMC
GNU General Public License v3.0
10 stars 8 forks source link

2.1.16: Update-Branches #63

Closed Ston3s closed 8 years ago

Ston3s commented 8 years ago

Hallo zusammen,

wir warten zwar noch auf das eine oder andere Feature welches noch eingebaut werden muss. Trotzdem sollten wir uns so langsam eventuell schon Gedanken über das große Update in Richtung Master-Branch machen. Dazu ist es bei diesem Versionssprung noch einmal notwendig eigene Update-Braches zu erstellen um es Lynx187 so einfach wie nur möglich zu machen. Dieses Issue soll es uns ermöglichen für alle Änderungen ein geeignetes Brach zu definieren. Wenn dies erfolgt ist, kann man diese Plattform nutzen um sich bis zur finalen Bereitstellung entsprechend abzustimmen ;)

Benennung sollte allerdings eindeutig und logisch stattfinden. Ich dachte daran, dass der erste Teil des Namens immer gleich ist damit man auch noch später weiß welche Funktion das Addon hatte.

Bitte vervollständigen!!!

1. update-autoupdate (PULL REQUEST + MERGED)

Ston3s commented 8 years ago

Habe mal den ersten Beitrag entsprechend erweitert. Hab ich alles wichtige drin, fehlt noch was. Wenn nicht würde ich die Branches mal gleich erstellen. Dann könnte man sich schon fleißig ans werk machen...

Feedback ist immer gut ;)

ghost commented 8 years ago

So ich habe das gestern für HDFilme.TV mal gemacht. Wollen wollen wir das zum Testen mal zum Master mergen oder wollen wir alles zusammen machen?

(Ich hätte noch ein commit was eine nicht genutzte Variable entfernt, soll ich das noch machen? Oder wollen wir die Branche dann noch mal neu machen? ^^)

Ston3s commented 8 years ago

👍👍👍 Warten und zusammen mergen. Also kannst den Commit noch ruhig einbringen!

Werde zur Übersichtlichkeit alles was erledigt in der Issue-Beschreibung durchstreichen.

ghost commented 8 years ago

Dann bekommt der arme Lynx187 aber am Tag x nen riesen Berg ^^

Weil wenn man den Pull jetzt startet. Wird er ja sogar automatisch ergänzt (falls sich bis zum merge noch Änderungen ergeben)

Ich bin dafür die branches sofern, sofern sie gefüllt wurden. Als Pull los zu jagen.

ghost commented 8 years ago

Und wie machen wir es nun? ^^

Ston3s commented 8 years ago

Mein Gedanke ist dabei, dass ich möglichst alle Änderungen sauber übergeben möchte. Wenn dann einige übergeben und angenommen sind und andere nicht was dann?

Irgendwann will man ja weiter machen und dafür ist es notwendig einen Kahlschlag zu machen und die Master-Version zu übernehmen. Könnte ich aber nicht weil manches übergeben und anderes wiederum nicht übergeben ist.

Ston3s commented 8 years ago

Gibt es sonst schon Fortschritte, wollte mich heute Nachmittag etwas damit beschäftigen und dann auch den Status aktualisieren.

ghost commented 8 years ago

Eben drum würde ich halt die Branches (wenn sie fertig sind) am liebste direkt zur Master schicken. Damit @Lynx187 sie in ruhe annehmen kann. Das halt ich für sinniger als zu warten bis alles gemacht wurde und dann einen Berg zu kommen lassen.

Ggf. werde eh ein paar Sachen nicht angenommen.

Ston3s commented 8 years ago

Da bin ich ja sowieso mal gespannt drauf!

Von mir aus können wir es machen aber ich habe es im Vorfeld kommuniziert. Die bestehende Nightly Branch wird erst durch die neue Master ersetzt wenn alle Update-Branches entsprechend gepushed sind...

ghost commented 8 years ago

Muss ja. Nur wir müssen langsam alles rüber bekommen. Denn die Änderungen in der Master bräuchten wir auch in der nightly (grade den SSL-Part)

HD-Filme habe ich mal rüber geschickt und DDL.me mache ich nachher.

Aber ansich finde ich das mit dem Branches spitze. Falls @Lynx187 noch etwas zu beanstanden hat muss ich nur nen commit machen und der Pull bleibt trotzdem bestehen. Bis alle zufrieden sind und der Merge durchgeht.

sraedler commented 8 years ago

Hab auch den Updatemanager als Pull Request bereit gestellt.

Sollte normalerweise so passen

Ston3s commented 8 years ago

Könnte mir jemand noch bei den beiden offenen Optionen in den Einstellungen zu dem Global Search an erster Stelle und dem zentralen Settings Ordner weiterhelfen.

War ja noch nicht drin und Lynx187 meinte er hätte nichts dagegen wenn es optional wäre...

sraedler commented 8 years ago

Meinst du ein Setting, dass man wählen kann ob es ein Settingsfolder gibt oder nicht?

ghost commented 8 years ago

Das gehört hier nich rein. Aber ich habe nen Bug mit dem Settings-Folder und zwar wenn mein nen neue Site auswählt trägt er die nicht direkt rein. Ich muss aus xStream raus und wieder rein damit die Liste neu aufgebaut wird.

Ston3s commented 8 years ago

Jo, Setting für Setting Folder und Setting für Global Search auf erster Position

Ston3s commented 8 years ago

War noch nicht an Rechner und konnte nicht schauen wie die PRs von euch erstellt wurden. Habt ihr die von mir erstellen Branches verwendet, hoffe hat alles von meiner Seite aus so gepasst

ghost commented 8 years ago

So habe meine Site-Addons rübergeschickt und HDFilme.tv ist schon drin. Bei den andern beiden muss ich nachher noch mal ran etwas aufräumen.

Könntest du mal nen Merge von der Hauptforke in die NIghtly machen? Grade die SSL Sachen könnte ggf. Leute stören wenn sie die Nighly benutzen.

sraedler commented 8 years ago

@Viper2k4 kannst du beschreiben wie du zu dem Bug im Settings-Folder kommst?

Ich hab es versucht, bei mir geht es. Es wird nämlich anschließend oGui.updateDirectory() aufgerufen und dann ist die neue Seite da.

P.s. mach bitte einen Bug auf....

Ston3s commented 8 years ago

Werde heute Abend den letzten Stand vom Master updaten. Bitte um anschließende Unterstützung bei auftretenden Konflikten.

Ston3s commented 8 years ago

Eben gemerged und jetzt geht gar nichts mehr...

ghost commented 8 years ago

So Pulls sind gemerged (update-manager, ddl.me, animie-loads, gute-filme.to). Könnte man also auch wieder richtung nightly holen. Dann haben wir schon mal nen schöne Basis.

sraedler commented 8 years ago

ja "frischer" Beta und nightly wäre was...

und dann häufigere pull requests und update Branchen schließen

Ston3s commented 8 years ago

Wollte erst schließen wenn alles übergeben ist. Leider bin aktuell ziemlich häufig dienstlich unterwegs und komme zu nichts!

Wie sieht es eigentlich um die fixes für die SitePlugins, die unter dem Branch "update-siteplugins" aufgezählt sind. Muss man da noch was übergeben oder hat Lynx187 schon alles in der Master?

Ston3s commented 8 years ago

Kann mir noch jemand mit den entsprechenden Einstellungen für die "Central Settings" Sache und die Position der Globalen Suche weiterhelfen?

Sonst bin ich mir da ziemlich sicher das Lynx187 das nicht übernehmen wird. Hatte er in den eigentlichen Issues so kommuniziert.

ghost commented 8 years ago

BurningSeries fix (HTTPS bzw. API) KinoLeak fix #55, #9 (Hier gab es keine Changes an der Kino-Leak-Datei, nur das unbennen was ich mal außen vorlasse) SE: Fix series only showing first page #35 BurningSeries Random Feature #50 fix GlobalSearch

Bei den Dickgeschrieben Sachen weiß ich das sie gefixt sind.

sraedler commented 8 years ago

random kann getestet werden #50

Ston3s commented 8 years ago

Ok, soeben wurde die Master in meinem Fork aktualisiert und alle von Lynx187 angenommen Pos ebenfalls in die Nightly eingebaut.

Next Step wäre die update-siteplugins zu pushen...

Ston3s commented 8 years ago

Also habe jetzt erstmal "update-siteplugins" soweit angepasst. Vielleicht könnt ihr mal einen Blick drauf werfen ob alles soweit ok ist. Sage schon mal danke!

Ston3s commented 8 years ago

PR für obiges Thema wurde eingereicht.

Wie siehts mit den den Anpassungen in den Optionen aus, würde mich über ein kurzes Feedback freuen. Gibt es bei den Punkten für update-Basics was zu berücksichtigen?!

Ston3s commented 8 years ago

Lynx meinte das in Update-Basic noch einige ident errors bei BS und SE. Habe allerdings nur die Veränderungen von "euch" in der Nightly eingepflegt.

Ston3s commented 8 years ago

Könntet ihr da mal vielleicht bitte einen Blick drauf werfen?

ghost commented 8 years ago

Also bei BS ist das komplett bei randomEpisode,randomSeason,randomSerie und showRandom und vereinzelt auch mitten drin. Scheint wohl mit #50 rein gekommen sein.

Und bei SW in Zeile 153 und 154. Scheint #34 gewesen zu sein.

Habe grade kein Kodi zur Hand zum Testen. Aber im Notepad++ (Ansicht => Nicht druckbare Zeichen => Leerzeichen und Tabulatoren) sieht man das dann ganz gut. Weil Pyton kein Tab mag zum einrücken.

ghost commented 8 years ago

So ich glaube ich habe alle erwischt. Aber @sraedler bitte mal drüber gucken ob es jetzt passt/läuft

sraedler commented 8 years ago

Ich hab es getestet, es funktioniert. Ich weiß nicht warum das passiert ist, bei mir in Pycharm sieht beides gleich aus (auch beim vergleichen der commits). Es geht aber.

Jedoch besteht ein Bug, der mir zuvor nicht aufgefallen ist. Er ist bezüglich utf-8, gleich wie #64.

Er kann so er zeugt werden: Zufalls Episode wählen und die Serie "100 Code" wählen. Anschließend die Staffel. Ab und an kommt nun ein UnicodeEncodeError.

Werde es gleich fixen, war ein blöder kleiner bug

ghost commented 8 years ago

Donno ich habe deine Version ins Notepad++ kopiert und dort sieht man das sehr gut (so fern die Option eingestellt ist). Aber Pycharm muss ich mir mal angucken. Sieht echt gut aus. Wobei ich gesehen habe das VisualStudio wohl auch Pyton support hat, das wollte ich mir die tage mal ansehen.

Aber wir weichen ab :)

ghost commented 8 years ago

Du solltest btw. auch noch folgendes beim Title berücksichtigen:

if episode['german']:
     title += episode['german'].encode('utf-8')
else:
     title += episode['english'].encode('utf-8')

so wird es eigentlich überall in BS gemacht. Nur bei randomEpisode fehlt das noch.

Ston3s commented 8 years ago

Danke fürs reinarbeiten!!!

ghost commented 8 years ago

Ich habe das mit dem Fallback mal eingebaut. Sprich wenn kein Deutsch Title vorhanden ist wird der englische benutzt.

Ston3s commented 8 years ago

Top, hoffentlich wars das mit "update-siteplugins". Ich wollte jetzt gerne bis zum WE die "updates-basics" soweit auf Stand bringen. Gibt es dort was wichtiges berücksichtigen? Kann man noch mit den Anpassung in den Optionen zwecks "Central Options" und "GlobalSrearch at first" rechnen?

sraedler commented 8 years ago

das global search find ich sinnvoll, das Central folder eigentlich nicht...

wenn ich Zeit finde mach ich es heute

Ston3s commented 8 years ago

Danke, kann man ja dann in den Einstellungen selber entscheiden ;)

sraedler commented 8 years ago

So jetzt hast deine Settings ;) c69833f

Evtl. müssen die Texte noch für Englisch und Deutsch geändert werden, mir fiel nämlich nichts besseres ein.

Ston3s commented 8 years ago

Ok super!

Mir ist aufgefallen das die Änderung in der Nightly und nicht in der Update-Branch ist. Werde ich dann später übertragen und hoffe das die xstream.py identisch/ähnlich ist, sodass keine Fehler dadurch übergeben werden.

Ston3s commented 8 years ago

Muss ich eigentlich was berücksichtigen wenn ich die Position der Einstellungen verändern will?

ghost commented 8 years ago

Direkt nicht. Aber die Frage ist eher wieso? :)

Lynx187 commented 8 years ago

Das einzige was man beachten sollte ist, dass die enable Tags sich relativ von der Position auf andere Settings beziehen. Mehr Infos, Kodi Wiki. Das bringt mich zur Frage wozu sollen die enable Tags in c69833f gut sein? PS: GlobalSearch kann auch fix als erstes stehen da muss man nicht noch extra eine Option einführen.

Ston3s commented 8 years ago

Man könnte es ja die Option GlobalSearch at First standardmäßig aktivieren. Finde jedoch die Möglichkeit die Suche vor den Einstellungen zu platzieren durchaus interessant. Ich z.B. Nutze diese globale Suche für alle Seiten extrem selten.

Muss ich bei einer überarbeiteten Sortierung der Einstellungen nur die Position anpassen oder auch die jeweiligen Labels? Oder wird über die Labels die Verknüpfung zur eigentlichen Funktion hergestellt? Bin erst morgen Abend in der Lage in wie Kodi Wiki zu schauen, wollte vorab aber schon mal nachfragen...

ghost commented 8 years ago

Was muss denn hier noch gemacht werden? Wäre nämlich schön wenn wir die Nightly wieder auf Stand der Master bekommen.

btw was haltet ihr eigentlich von folgenden Plan für den Updater:

Master => quasi das was in der Repro ist Beta => git-stand von Lynx1781 Nightly => diese Repro hier

Dann sollten wir für unsere Änderungen immer Branches anlegen und diese nach Lynx1781 pullen. Die Nightly kann sich die Änderungen der Brances ja immer direkt holen (kann man das Automatisieren?). Das heißt wir haben keine Direkte Entwicklung in der Nightly.

Aber so bleibt der Code-Verlauf von Nightly => Beta => Master wengistens konstant soweit.

Ston3s commented 8 years ago

Update-Basic müsste noch auf Stand gebracht werden bevor der PR erzeugt werden kann. Wollte ich eigentlich schon am Wochenende angehen aber gerade komme ich zeitlich nicht dazu...

Lynx187 commented 8 years ago

Also nur zu Info ich werde spätestens am 30.4. den Stand und damit eine neue Version ins xStreamRepo übertragen. Was momentan noch unter Update Basics steht:

Alles was noch offen ist eher optional. Wenns es noch rechtzeitig kommt ok, wenn nicht dann später, da für die Funktion irrelevant.