Split PTW poszedł się...

[rysson]

Nożesz...

Wiem, że niewiele się dokładam, ale jednym z zadań jakich się pojąłem było rozdzielenie kodu PTW od FunFilm (gałąź split_ptw). O ile pamiętam się udało (głównie commit 0e6d49d)

Teraz patrzę a cały ten syf jest z powrotem.

@CherryKodi, czy jest on potrzebny w PTW? Zmieniła się koncepcja, czy coś przeoczyliśmy?

Pytam, bo dodałeś (7e5fd5b z 27.12.2017) z powrotem multum plików script.module.ptw/lib/ptw/libraries/, które wcześniej zostały przeniesione do plugin.video.fanfilm/resources/lib/libraries.

Teraz mamy je w dwóch miejscach, co jest fatalne IMO.

Chłopaki, @xulek, @notoco..., proszę o komentarz.

[xulek]

Były jakieś problemy z ładowaniem zrodel gdy biblioteka pomocnicza była oddzielną wtyczką, czasem ladowalo źródła w fanfilm a czasem nie wykrywało ani jednego jeśli dobrze pamiętam. Kolejnym powodem powrotu było łatwiejsze wrzucanie scrapperow z innych wtyczek do fanfilm (mniej zmian w importach). Ja się już nie bawię w aktualizowanie tej wtyczki publicznie, więc jeśli chcesz to masz wolną rękę w aktualizacjach :)

[rysson]

Czyli podejrzewam jakieś nieścisłości w importach gdzieś. Bo tu nie powinno być magii :-)
Lenistwo też nie jest wystarczającym argumentem (grep, sed, awk.. :-D)

Tiaaa... sporo jest do przepisania. Dzięki za info. Ty teraz już tylko działasz w tym repo, co mi kiedyś przesłałeś?

[xulek]

Były nieścisłości bo czasami biblioteka była importowana z ptw a czasem z fanfilma bo istniały w dwóch miejscach. Z braku czasu i właśnie lenistwa poszedłem na łatwiznę :P Tak, teraz tylko rozwijam prywatne repo na gitlab

[rysson]

No właśnie teraz jest w dwóch miejscach. Po split było w jednym (nowym).
Ale to już· z resztą, która tu działa.

To ja tam zajrzę kiedyś (ach ten wieczny brak czasu).

BTW. Możesz przy okazji zerknąć, czy mój PR #45 robiony na szybko ma sens?

[xulek]

Jutro mogę sprawdzić ale wygląda ciekawie. Teraz ważne żeby wszystko było kompatybilne z python 3 :) Bo za rok(albo krócej dla nightly) będzie płacz :P

[rysson]

Dzięki.

Python3 – jestem za!!! :-D