25.2.2021, 10:45
@Saljack: Proč bych měl čekat na merge PR, který ten problém s compat_path neřeší? Také nesouhlasím s tím, že se compat_path nepoužívá, protože minimálně při startu pluginu se hned použije a proto to na Windows spadne. compat_path řeší problém s encodingem názvů souborů, kde asi Windows moc nechutnají názvy obsahujíci binární data v kódovaní UTF-8. Dohledal jsem, že Python na to má funkci, která to řeší (zdroj: https://www.python.org/dev/peps/pep-0529/ a https://docs.python.org/3/library/os.html).
Správná OS nezávislá implementace je podle mne tato:
Co se týká abortu, tak nevidím důvod nefixnout něco, co házi chyby do logu, je to evidentně špatně a je jasné, jak se to má opravit
.
Nehledě na to, že díky té chybě v abortu na pozadí neběží služba, která obnovuje subscription.
Správná OS nezávislá implementace je podle mne tato:
Kód:
def compat_path(path):
return os.fsencode(path);Co se týká abortu, tak nevidím důvod nefixnout něco, co házi chyby do logu, je to evidentně špatně a je jasné, jak se to má opravit
.Nehledě na to, že díky té chybě v abortu na pozadí neběží služba, která obnovuje subscription.