Skip to content

patch_debian.md

Latest commit

 

History

History
1403 lines (1147 loc) · 56.5 KB

patch_debian.md

File metadata and controls

1403 lines (1147 loc) · 56.5 KB
Raw

Patch Linux/Debian — SongpressPlusPlus

Questa guida elenca tutte le patch applicate al codice sorgente per la compatibilità con Linux/Debian, come verificarle e come applicarle manualmente se necessario.

⚠️ Percorso di installazione cambiato. Fino alle versioni precedenti il pacchetto installava sotto /usr/local/lib/python3.13/dist-packages/. La Debian Policy riserva /usr/local all'amministratore locale, quindi build_deb.sh ora normalizza l'albero in:

/usr/lib/python3/dist-packages/songpressplusplus/   (moduli)
/usr/bin/SongpressPlusPlus                          (eseguibile)

Il percorso è senza numero di versione di Python: è l'unica directory di sistema realmente presente in sys.path su Debian e rende il pacchetto immune agli aggiornamenti di minor version. Tutti i comandi «applicazione sul file installato» in questa guida usano il nuovo percorso.

La migrazione dal vecchio layout è automatica: lo script preinst del .deb rimuove i residui in /usr/local prima dello scompattamento, dopo aver verificato con dpkg-query che nessun pacchetto li rivendichi. Per sapere in ogni momento dove sono i file installati:

python3 -c "import songpressplusplus, os; print(os.path.dirname(songpressplusplus.__file__))"

Patch 1 — Testo nero nella finestra "Informazioni su"

File: src/songpressplusplus/SongpressFrame.py

Problema: Su Linux con tema scuro, il testo della finestra About è bianco su sfondo bianco e risulta invisibile. wxPython non eredita il colore del testo da SetBackgroundColour(wx.WHITE).

Fix: Aggiunge SetForegroundColour(wx.BLACK) su ogni wx.StaticText della finestra About.

Verifica

python3 - <<'PY'
with open("src/songpressplusplus/SongpressFrame.py", "r") as f:
    content = f.read()

checks = [
    '            title_lbl.SetForegroundColour(wx.BLACK)',
    '            lbl.SetForegroundColour(wx.BLACK)',
]

for c in checks:
    print(f"{'✅' if c in content else '❌'} {c.strip()!r}")
PY

Output atteso:

✅ 'title_lbl.SetForegroundColour(wx.BLACK)'
✅ 'lbl.SetForegroundColour(wx.BLACK)'

Applicazione manuale

python3 - <<'PY'
with open("src/songpressplusplus/SongpressFrame.py", "r") as f:
    content = f.read()

fixes = [
    (
        '            title_lbl.SetFont(font_title)\n            hbox_title.Add(title_lbl, 0, wx.ALIGN_CENTER_VERTICAL)',
        '            title_lbl.SetFont(font_title)\n            title_lbl.SetForegroundColour(wx.BLACK)\n            hbox_title.Add(title_lbl, 0, wx.ALIGN_CENTER_VERTICAL)'
    ),
    (
        '        except Exception:\n            title_lbl = wx.StaticText(panel, label=u"Songpress++ - The Song Editor {}".format(glb.VERSION))\n            vbox.Add(title_lbl, 0, wx.ALIGN_CENTER | wx.ALL, 10)',
        '        except Exception:\n            title_lbl = wx.StaticText(panel, label=u"Songpress++ - The Song Editor {}".format(glb.VERSION))\n            title_lbl.SetForegroundColour(wx.BLACK)\n            vbox.Add(title_lbl, 0, wx.ALIGN_CENTER | wx.ALL, 10)'
    ),
    (
        '        def add_text(text):\n            lbl = wx.StaticText(panel, label=text)\n            vbox.Add(lbl, 0, wx.ALIGN_CENTER | wx.LEFT | wx.RIGHT, 15)',
        '        def add_text(text):\n            lbl = wx.StaticText(panel, label=text)\n            lbl.SetForegroundColour(wx.BLACK)\n            vbox.Add(lbl, 0, wx.ALIGN_CENTER | wx.LEFT | wx.RIGHT, 15)'
    ),
]

count = sum(1 for old, new in fixes if old in content)
for old, new in fixes:
    content = content.replace(old, new)

with open("src/songpressplusplus/SongpressFrame.py", "w") as f:
    f.write(content)
print(f"Fix applicati: {count}/3")
PY

Applicazione sul file installato (senza ricostruire il .deb)

sudo python3 - <<'PY'
path = "/usr/lib/python3/dist-packages/songpressplusplus/SongpressFrame.py"

with open(path, "r") as f:
    content = f.read()

fixes = [
    (
        '            title_lbl.SetFont(font_title)\n            hbox_title.Add(title_lbl, 0, wx.ALIGN_CENTER_VERTICAL)',
        '            title_lbl.SetFont(font_title)\n            title_lbl.SetForegroundColour(wx.BLACK)\n            hbox_title.Add(title_lbl, 0, wx.ALIGN_CENTER_VERTICAL)'
    ),
    (
        '        except Exception:\n            title_lbl = wx.StaticText(panel, label=u"Songpress++ - The Song Editor {}".format(glb.VERSION))\n            vbox.Add(title_lbl, 0, wx.ALIGN_CENTER | wx.ALL, 10)',
        '        except Exception:\n            title_lbl = wx.StaticText(panel, label=u"Songpress++ - The Song Editor {}".format(glb.VERSION))\n            title_lbl.SetForegroundColour(wx.BLACK)\n            vbox.Add(title_lbl, 0, wx.ALIGN_CENTER | wx.ALL, 10)'
    ),
    (
        '        def add_text(text):\n            lbl = wx.StaticText(panel, label=text)\n            vbox.Add(lbl, 0, wx.ALIGN_CENTER | wx.LEFT | wx.RIGHT, 15)',
        '        def add_text(text):\n            lbl = wx.StaticText(panel, label=text)\n            lbl.SetForegroundColour(wx.BLACK)\n            vbox.Add(lbl, 0, wx.ALIGN_CENTER | wx.LEFT | wx.RIGHT, 15)'
    ),
]

count = sum(1 for old, new in fixes if old in content)
for old, new in fixes:
    content = content.replace(old, new)

with open(path, "w") as f:
    f.write(content)
print(f"Fix applicati: {count}/3")
PY

Patch 2 — Crash colore non valido nel selettore colori

File: src/songpressplusplus/MyPreferencesDialog.py

Problema: Su Linux GetCustomColour(i) può restituire un colore non valido (slot vuoto). Chiamare Red() su un colore non valido causa un crash wx._core.wxAssertionError.

Fix: Aggiunge un controllo IsOk() prima di leggere i componenti RGB.

Verifica

python3 - <<'PY'
with open("src/songpressplusplus/MyPreferencesDialog.py", "r") as f:
    content = f.read()

check = "        if not colour.IsOk():"
print(f"{'✅' if check in content else '❌'} Controllo IsOk() presente")
PY

Applicazione manuale

python3 - <<'PY'
with open("src/songpressplusplus/MyPreferencesDialog.py", "r") as f:
    content = f.read()

old = """    def _colour_to_hex(self, colour):
        return '#{:02X}{:02X}{:02X}'.format(colour.Red(), colour.Green(), colour.Blue())"""

new = """    def _colour_to_hex(self, colour):
        if not colour.IsOk():
            return '#FFFFFF'
        return '#{:02X}{:02X}{:02X}'.format(colour.Red(), colour.Green(), colour.Blue())"""

if old in content:
    content = content.replace(old, new)
    with open("src/songpressplusplus/MyPreferencesDialog.py", "w") as f:
        f.write(content)
    print("OK")
else:
    print("ERRORE - testo non trovato")
PY

Applicazione sul file installato

sudo python3 - <<'PY'
path = "/usr/lib/python3/dist-packages/songpressplusplus/MyPreferencesDialog.py"

with open(path, "r") as f:
    content = f.read()

old = """    def _colour_to_hex(self, colour):
        return '#{:02X}{:02X}{:02X}'.format(colour.Red(), colour.Green(), colour.Blue())"""

new = """    def _colour_to_hex(self, colour):
        if not colour.IsOk():
            return '#FFFFFF'
        return '#{:02X}{:02X}{:02X}'.format(colour.Red(), colour.Green(), colour.Blue())"""

if old in content:
    content = content.replace(old, new)
    with open(path, "w") as f:
        f.write(content)
    print("OK")
else:
    print("ERRORE - testo non trovato")
PY

Patch 3 — Pulsanti associazione file disabilitati su Linux

File: src/songpressplusplus/PreferencesDialog.py

Problema: Su Linux i pulsanti "Associa tutto", "Disassocia tutto" e "Applica ora" creano file locali in ~/.local/share/ che entrano in conflitto con le associazioni di sistema installate dal pacchetto .deb. Anche i checkbox delle singole estensioni devono essere disabilitati per evitare modifiche accidentali.

Fix: Disabilita i tre pulsanti e tutti i checkbox delle estensioni automaticamente quando il sistema è Linux.

Verifica

python3 - <<'PY'
with open("src/songpressplusplus/PreferencesDialog.py", "r") as f:
    content = f.read()

checks = [
    ("btnAssocAll.Disable()",   "Disabilita btnAssocAll"),
    ("btnUnassocAll.Disable()", "Disabilita btnUnassocAll"),
    ("btnApply.Disable()",      "Disabilita btnApply"),
    ("cb.Disable()",            "Disabilita checkbox estensioni"),
]

for code, desc in checks:
    print(f"{'✅' if code in content else '❌'} {desc}")
PY

Output atteso:

✅ Disabilita btnAssocAll
✅ Disabilita btnUnassocAll
✅ Disabilita btnApply
✅ Disabilita checkbox estensioni

Applicazione manuale

python3 - <<'PY'
with open("src/songpressplusplus/PreferencesDialog.py", "r") as f:
    content = f.read()

fixes = [
    (
        "            self._btnAssocAll   = btnAssocAll\n            self._btnUnassocAll = btnUnassocAll",
        "            self._btnAssocAll   = btnAssocAll\n            self._btnUnassocAll = btnUnassocAll\n            import platform as _pl\n            if _pl.system() == 'Linux':\n                btnAssocAll.Disable()\n                btnUnassocAll.Disable()"
    ),
    (
        "            self._btnApplyFileAssoc = btnApply",
        "            self._btnApplyFileAssoc = btnApply\n            import platform as _pl2\n            if _pl2.system() == 'Linux':\n                btnApply.Disable()"
    ),
]

count = sum(1 for old, new in fixes if old in content)
for old, new in fixes:
    content = content.replace(old, new)

with open("src/songpressplusplus/PreferencesDialog.py", "w") as f:
    f.write(content)
print(f"Fix applicati: {count}/3")
PY

Applicazione manuale checkbox estensioni

python3 - <<'PY'
with open("src/songpressplusplus/PreferencesDialog.py", "r") as f:
    content = f.read()

old = """            for ext in self._fileAssocExts:
                cb = wx.CheckBox(self.fileAssocPanel, wx.ID_ANY, u"." + ext)
                cb.SetToolTip(_(u"Associate the .%s file extension with Songpress++.") % ext)
                self._fileAssocCBs[ext] = cb
                bSizerFA.Add(cb, 0, wx.ALL, 4)"""

new = """            import platform as _plcb
            for ext in self._fileAssocExts:
                cb = wx.CheckBox(self.fileAssocPanel, wx.ID_ANY, u"." + ext)
                cb.SetToolTip(_(u"Associate the .%s file extension with Songpress++.") % ext)
                self._fileAssocCBs[ext] = cb
                if _plcb.system() == 'Linux':
                    cb.Disable()
                bSizerFA.Add(cb, 0, wx.ALL, 4)"""

if old in content:
    content = content.replace(old, new)
    with open("src/songpressplusplus/PreferencesDialog.py", "w") as f:
        f.write(content)
    print("OK")
else:
    print("ERRORE - testo non trovato")
PY

Applicazione sul file installato

sudo python3 - <<'PY'
path = "/usr/lib/python3/dist-packages/songpressplusplus/PreferencesDialog.py"

with open(path, "r") as f:
    content = f.read()

fixes = [
    (
        "            self._btnAssocAll   = btnAssocAll\n            self._btnUnassocAll = btnUnassocAll",
        "            self._btnAssocAll   = btnAssocAll\n            self._btnUnassocAll = btnUnassocAll\n            import platform as _pl\n            if _pl.system() == 'Linux':\n                btnAssocAll.Disable()\n                btnUnassocAll.Disable()"
    ),
    (
        "            self._btnApplyFileAssoc = btnApply",
        "            self._btnApplyFileAssoc = btnApply\n            import platform as _pl2\n            if _pl2.system() == 'Linux':\n                btnApply.Disable()"
    ),
]

count = sum(1 for old, new in fixes if old in content)
for old, new in fixes:
    content = content.replace(old, new)

with open(path, "w") as f:
    f.write(content)
print(f"Fix applicati: {count}/2")
PY

Patch 4 — Testo e sfondo nella finestra "Statistiche brano"

File: src/songpressplusplus/SongpressFrame.py

Problema: Su Linux con tema scuro:

  • Lo sfondo del dialog è bianco hardcoded (wx.Colour(250,250,252)), ignorando il tema di sistema.
  • Il testo delle righe chiave/valore (Struttura, Testo, Accordi, Metadati) non ha colore esplicito e risulta invisibile.
  • Le stelle e il verdetto ("Ottimo per principianti" ecc.) erano in un wx.Panel con sfondo bianco hardcoded.
  • Il testo del verdetto non aveva colore esplicito e risultava invisibile su tema scuro.

Fix:

  • BG → wx.SystemSettings.GetColour(wx.SYS_COLOUR_WINDOW) (segue il tema).
  • wx.WHITE e wx.Colour(240,245,255) → SYS_COLOUR_WINDOW (finestra About, eval_panel, tastiera).
  • eval_panel rimosso: stelle e verdetto aggiunti direttamente al sizer senza pannello con sfondo fisso.
  • wx.BLACK → SYS_COLOUR_WINDOWTEXT su tutti i SetForegroundColour hardcoded.
  • SetForegroundColour(SYS_COLOUR_WINDOWTEXT) esplicito su k_lbl, v_lbl e _lbl_verdict.

Verifica

Usa la sezione Verifica rapida in fondo al documento (controlla Patch 4a–4e).

Applicazione sul file installato (senza ricostruire il .deb)

SPPY=/usr/lib/python3/dist-packages/songpressplusplus/SongpressFrame.py
sudo sed -i \
  -e 's/SetForegroundColour(wx\.BLACK)/SetForegroundColour(wx.SystemSettings.GetColour(wx.SYS_COLOUR_WINDOWTEXT))/g' \
  -e 's/SetBackgroundColour(wx\.WHITE)/SetBackgroundColour(wx.SystemSettings.GetColour(wx.SYS_COLOUR_WINDOW))/g' \
  -e 's/wx\.Colour(240, 245, 255)/wx.SystemSettings.GetColour(wx.SYS_COLOUR_WINDOW)/g' \
  "$SPPY"

Nota: questa patch è applicata automaticamente da build_deb.sh (sezione 1e).

Patch 5 — Crash _mgr al cambio lingua / chiusura

File: src/songpressplusplus/SongpressFrame.py e src/songpressplusplus/SongpressToolbars.py

Problema: Cambiando lingua (es. da Italiano a Inglese) l'app fa un restart: chiude la finestra e rilancia il processo. Mentre la finestra si chiude GTK emette un EVT_SIZE che programma un aggiornamento differito delle toolbar (wx.CallAfter(_deferred_tb_update)). Quando la callback viene eseguita l'AUI manager è già stato smontato (UnInit) e l'attributo _mgr non esiste più sul frame, quindi:

AttributeError: 'SongpressFrame' object has no attribute '_mgr'
  File ".../SongpressToolbars.py", line 202, in _FinalizeToolbarLayout
    pane = self._mgr.GetPane(tb)

La guardia if self.frame: non basta: self.frame resta "truthy" durante la chiusura mentre _mgr è già sparito. È una race di teardown, non un bug di i18n.

Fix:

  • _deferred_tb_update non tocca il layout se _mgr è assente o se il frame è in chiusura (_closing).
  • OnClose imposta self._closing = True come prima istruzione.
  • _FinalizeToolbarLayout esce subito se _mgr è None o se le toolbar non esistono più (protegge anche gli altri chiamanti).

Verifica

python3 - <<'PY'
checks = [
    ("SongpressFrame.py",    "getattr(self, '_mgr', None) is not None"),
    ("SongpressFrame.py",    "self._closing = True"),
    ("SongpressToolbars.py", "if getattr(self, '_mgr', None) is None:"),
]
for fn, needle in checks:
    with open("src/songpressplusplus/" + fn) as f:
        ok = needle in f.read()
    print(f"{'✅' if ok else '❌'} {fn}: {needle!r}")
PY

Applicazione manuale (sorgente)

python3 - <<'PY'
FR = "src/songpressplusplus/SongpressFrame.py"
TB = "src/songpressplusplus/SongpressToolbars.py"

with open(FR) as f: fr = f.read()
with open(TB) as f: tb = f.read()

# --- Frame: guardia in _deferred_tb_update ---
old_a = '''                    def _deferred_tb_update():
                        self._tb_layout_pending = False
                        if self.frame:
                            self._FinalizeToolbarLayout()'''
new_a = '''                    def _deferred_tb_update():
                        self._tb_layout_pending = False
                        # Durante la chiusura/riavvio l'AUI manager puo' essere
                        # gia' stato smontato (UnInit): _mgr non esiste piu'.
                        # self.frame resta "truthy" ma il layout non va toccato.
                        if (self.frame
                                and getattr(self, '_mgr', None) is not None
                                and not getattr(self, '_closing', False)):
                            self._FinalizeToolbarLayout()'''

# --- Frame: _closing in OnClose ---
old_b = '''    def OnClose(self, evt):
        if hasattr(self, '_lockKeysTimer') and self._lockKeysTimer.IsRunning():'''
new_b = '''    def OnClose(self, evt):
        self._closing = True
        if hasattr(self, '_lockKeysTimer') and self._lockKeysTimer.IsRunning():'''

# --- Toolbars: guardia in _FinalizeToolbarLayout ---
old_c = '''        self._tb_finalizing = True
        try:
            for tb in (self.mainToolBar, self.formatToolBar,
                       self.insertToolBar, self.viewToolBar):
                tb.SetGripperVisible(False)'''
new_c = '''        # Se l'AUI manager e' gia' stato smontato (chiusura/riavvio) o le
        # toolbar non esistono piu', non c'e' layout da ricalcolare.
        if getattr(self, '_mgr', None) is None:
            return
        if not all(getattr(self, name, None) is not None for name in
                   ('mainToolBar', 'formatToolBar',
                    'insertToolBar', 'viewToolBar')):
            return
        self._tb_finalizing = True
        try:
            for tb in (self.mainToolBar, self.formatToolBar,
                       self.insertToolBar, self.viewToolBar):
                tb.SetGripperVisible(False)'''

count = 0
for old, new, tgt in ((old_a, new_a, 'fr'), (old_b, new_b, 'fr'), (old_c, new_c, 'tb')):
    src = fr if tgt == 'fr' else tb
    if old in src:
        if tgt == 'fr': fr = fr.replace(old, new)
        else:           tb = tb.replace(old, new)
        count += 1

with open(FR, "w") as f: f.write(fr)
with open(TB, "w") as f: f.write(tb)
print(f"Fix applicati: {count}/3")
PY

Applicazione sul file installato

sudo python3 - <<'PY'
BASE = "/usr/lib/python3/dist-packages/songpressplusplus/"
FR = BASE + "SongpressFrame.py"
TB = BASE + "SongpressToolbars.py"

with open(FR) as f: fr = f.read()
with open(TB) as f: tb = f.read()

old_a = '''                    def _deferred_tb_update():
                        self._tb_layout_pending = False
                        if self.frame:
                            self._FinalizeToolbarLayout()'''
new_a = '''                    def _deferred_tb_update():
                        self._tb_layout_pending = False
                        # Durante la chiusura/riavvio l'AUI manager puo' essere
                        # gia' stato smontato (UnInit): _mgr non esiste piu'.
                        # self.frame resta "truthy" ma il layout non va toccato.
                        if (self.frame
                                and getattr(self, '_mgr', None) is not None
                                and not getattr(self, '_closing', False)):
                            self._FinalizeToolbarLayout()'''

old_b = '''    def OnClose(self, evt):
        if hasattr(self, '_lockKeysTimer') and self._lockKeysTimer.IsRunning():'''
new_b = '''    def OnClose(self, evt):
        self._closing = True
        if hasattr(self, '_lockKeysTimer') and self._lockKeysTimer.IsRunning():'''

old_c = '''        self._tb_finalizing = True
        try:
            for tb in (self.mainToolBar, self.formatToolBar,
                       self.insertToolBar, self.viewToolBar):
                tb.SetGripperVisible(False)'''
new_c = '''        # Se l'AUI manager e' gia' stato smontato (chiusura/riavvio) o le
        # toolbar non esistono piu', non c'e' layout da ricalcolare.
        if getattr(self, '_mgr', None) is None:
            return
        if not all(getattr(self, name, None) is not None for name in
                   ('mainToolBar', 'formatToolBar',
                    'insertToolBar', 'viewToolBar')):
            return
        self._tb_finalizing = True
        try:
            for tb in (self.mainToolBar, self.formatToolBar,
                       self.insertToolBar, self.viewToolBar):
                tb.SetGripperVisible(False)'''

count = 0
for old, new, tgt in ((old_a, new_a, 'fr'), (old_b, new_b, 'fr'), (old_c, new_c, 'tb')):
    src = fr if tgt == 'fr' else tb
    if old in src:
        if tgt == 'fr': fr = fr.replace(old, new)
        else:           tb = tb.replace(old, new)
        count += 1

with open(FR, "w") as f: f.write(fr)
with open(TB, "w") as f: f.write(tb)
print(f"Fix applicati: {count}/3")
PY

Patch 6 — Warning "Cannot set locale" con locale di sistema mancante

File: src/songpressplusplus/i18n.py

Problema: Cambiando lingua in inglese su un sistema dove il locale C corrispondente (es. en_US.UTF-8) non è stato generato con locale-gen, wx.Locale(langid) non riesce a chiamare setlocale() e wxWidgets emette un wxLogWarning che, col log target GUI di default, appare come finestra "Cannot set locale to language 'English'.". Inoltre, se FindLanguageInfo(l) restituisce None per un codice sconosciuto, l'accesso a .Language provoca un AttributeError.

Fix:

  • Guardia su FindLanguageInfo: fallback a wx.LANGUAGE_DEFAULT se None.
  • Costruzione di wx.Locale avvolta in wx.LogNull(): il warning non genera più la finestra. I cataloghi .mo si caricano comunque (la traduzione dipende dalla lingua impostata in wx.Locale, non dal locale C) e l'oggetto mylocale resta valido per wx.GetLocale() usato altrove.

Verifica

python3 - <<'PY'
with open("src/songpressplusplus/i18n.py") as f:
    c = f.read()
checks = [
    "info = wx.Locale.FindLanguageInfo(l)",
    "info.Language if info is not None else wx.LANGUAGE_DEFAULT",
    "_nolog = wx.LogNull()",
]
for n in checks:
    print(f"{'✅' if n in c else '❌'} {n!r}")
PY

Applicazione manuale (sorgente)

python3 - <<'PY'
path = "src/songpressplusplus/i18n.py"
with open(path) as f:
    content = f.read()

old = '''def setLang(l):
    global current_language, mylocale, _
    current_language = l
    langid = wx.Locale.FindLanguageInfo(l).Language
    mylocale = wx.Locale(langid)
    localedir = os.path.join(glb.path, "locale")'''

new = '''def setLang(l):
    global current_language, mylocale, _
    current_language = l
    info = wx.Locale.FindLanguageInfo(l)
    langid = info.Language if info is not None else wx.LANGUAGE_DEFAULT

    # Su sistemi dove il locale C della lingua richiesta (es. en_US.UTF-8)
    # non e' stato generato con locale-gen, wx.Locale non riesce a chiamare
    # setlocale() ed emette un wxLogWarning che, col log target GUI di
    # default, compare come finestra "Cannot set locale to language ...".
    # La traduzione (cataloghi .mo) funziona comunque, perche' dipende dalla
    # lingua impostata in wx.Locale e non dal locale C: silenziamo quindi
    # solo quel warning durante la costruzione, senza perdere le traduzioni
    # ne' l'oggetto mylocale (usato da wx.GetLocale() altrove).
    _nolog = wx.LogNull()
    try:
        mylocale = wx.Locale(langid)
    finally:
        del _nolog

    localedir = os.path.join(glb.path, "locale")'''

if old in content:
    content = content.replace(old, new)
    with open(path, "w") as f:
        f.write(content)
    print("OK")
else:
    print("Patch già presente o testo non trovato")
PY

Applicazione sul file installato

sudo python3 - <<'PY'
path = "/usr/lib/python3/dist-packages/songpressplusplus/i18n.py"
with open(path) as f:
    content = f.read()

old = '''def setLang(l):
    global current_language, mylocale, _
    current_language = l
    langid = wx.Locale.FindLanguageInfo(l).Language
    mylocale = wx.Locale(langid)
    localedir = os.path.join(glb.path, "locale")'''

new = '''def setLang(l):
    global current_language, mylocale, _
    current_language = l
    info = wx.Locale.FindLanguageInfo(l)
    langid = info.Language if info is not None else wx.LANGUAGE_DEFAULT

    # Su sistemi dove il locale C della lingua richiesta (es. en_US.UTF-8)
    # non e' stato generato con locale-gen, wx.Locale non riesce a chiamare
    # setlocale() ed emette un wxLogWarning che, col log target GUI di
    # default, compare come finestra "Cannot set locale to language ...".
    # La traduzione (cataloghi .mo) funziona comunque, perche' dipende dalla
    # lingua impostata in wx.Locale e non dal locale C: silenziamo quindi
    # solo quel warning durante la costruzione, senza perdere le traduzioni
    # ne' l'oggetto mylocale (usato da wx.GetLocale() altrove).
    _nolog = wx.LogNull()
    try:
        mylocale = wx.Locale(langid)
    finally:
        del _nolog

    localedir = os.path.join(glb.path, "locale")'''

if old in content:
    content = content.replace(old, new)
    with open(path, "w") as f:
        f.write(content)
    print("OK")
else:
    print("Patch già presente o testo non trovato")
PY

Nota — locale di sistema (facoltativo): la patch fa sì che l'inglese compaia senza avvisi anche se il locale C non è generato. Per avere anche la formattazione di numeri/date corretta si può generare il locale a livello di sistema (operazione manuale dell'utente, non inclusa nel .deb):

sudo sed -i 's/^# *\(en_US.UTF-8 UTF-8\)/\1/' /etc/locale.gen
sudo locale-gen

Patch 7 — Pulsante "copia" anteprima: icona PNG e copia immagine su Wayland

File: src/songpressplusplus/PreviewCanvas.py, src/songpressplusplus/SongpressFrame.py

Problema (tre parti):

  1. Icona. L'icona del pulsante "copia" nella toolbar dell'anteprima era disegnata a mano; si voleva usare il PNG del progetto img/copy_2.png.
  2. Il pulsante non copiava. Il pulsante rilanciava un wx.adv.HyperlinkEvent sintetico confidando nella propagazione dell'evento fino a main_panel: su wxGTK questa propagazione dal BitmapButton non è affidabile, quindi OnCopyAsImage non veniva mai chiamato.
  3. La copia immagine non funziona su Wayland. Anche chiamando direttamente CopyAsImage, la clipboard immagine di wxGTK su Wayland non registra i formati immagine (wl-paste --list-types mostra solo testo), pur riportando successo. È un limite noto di wxGTK/Wayland.

Fix:

  1. PreviewCanvas._load_copy_bitmap() carica img/copy_2.png (con glb.AddPath), scalata a 16px, e ripiega silenziosamente sull'icona disegnata se il file manca (controllo os.path.isfile + wx.LogNull() per non far comparire il popup d'errore di wxLog).
  2. PreviewCanvas espone SetCopyCallback() e _OnCopyButton chiama la callback diretta (niente più eventi wx). SongpressFrame la registra con self.previewCanvas.SetCopyCallback(self.CopyAsImage).
  3. CopyAsImage su Wayland genera il PNG e lo mette negli appunti con wl-copy --type image/png (pacchetto wl-clipboard). Su X11/macOS resta la clipboard di wx (SVG + PNG); su Windows invariato (metafile). Se wl-copy manca, avvisa di installare wl-clipboard.

Packaging: wl-clipboard è aggiunto tra i Recommends del .deb (vedi build_deb.sh), così su Wayland viene installato di default da apt. Nota: il .deb avvia comunque l'app con GDK_BACKEND=x11 (XWayland); il compositor (KDE) fa da ponte tra la clipboard Wayland di wl-copy e le app X11, quindi la copia funziona in entrambi i casi.

Verifica

python3 - <<'PY'
import os
BASE = "src/songpressplusplus"
checks = [
    ("PreviewCanvas.py",  "def _load_copy_bitmap",   "Patch 7a — icona copy_2.png"),
    ("PreviewCanvas.py",  "def SetCopyCallback",     "Patch 7b — callback copia esposta"),
    ("PreviewCanvas.py",  "self._on_copy_callback",  "Patch 7c — _OnCopyButton usa la callback"),
    ("SongpressFrame.py", "SetCopyCallback(self.CopyAsImage)", "Patch 7d — callback registrata"),
    ("SongpressFrame.py", "def _copy_bytes_wayland", "Patch 7e — copia via wl-copy su Wayland"),
    ("SongpressFrame.py", "wl-copy",                 "Patch 7f — uso di wl-copy"),
]
for filename, needle, desc in checks:
    path = os.path.join(BASE, filename)
    try:
        with open(path) as f:
            found = needle in f.read()
        print(f"{'✅' if found else '❌'} {desc}")
    except FileNotFoundError:
        print(f"⚠️  File non trovato: {path}")
PY

Applicazione

Questa patch riscrive interi metodi e ne aggiunge di nuovi in due file: non è pratica come blocco content.replace(). I file sorgente (PreviewCanvas.py, SongpressFrame.py) contengono già il fix e sono la fonte di verità. Per applicarla al file installato senza ricostruire il .deb, copiare i due file:

SRC=~/Songpress_DEFINitiVO3/SongpressPlusPlus/src/songpressplusplus
DST=/usr/lib/python3/dist-packages/songpressplusplus
sudo cp "$SRC/PreviewCanvas.py"  "$DST/PreviewCanvas.py"
sudo cp "$SRC/SongpressFrame.py" "$DST/SongpressFrame.py"

Serve inoltre il PNG dell'icona e il pacchetto wl-clipboard:

# icona (se non già presente nell'installato)
sudo cp "$SRC/img/copy_2.png" "$DST/img/copy_2.png"
# strumento clipboard Wayland
sudo apt install wl-clipboard

Test funzionale (Wayland)

Per prima cosa, verifica se la sessione è X11 o Wayland (il percorso wl-copy si attiva solo su Wayland):

# metodo rapido (variabile d'ambiente della sessione grafica)
echo "$XDG_SESSION_TYPE"      # stampa  wayland  oppure  x11

# metodo autorevole (systemd-logind, consigliato sulle distro recenti)
loginctl show-session "$(loginctl --no-legend list-sessions | awk -v u="$USER" '$3==u {print $1; exit}')" -p Type --value

Poi prova la copia:

# dopo aver premuto "copia" in Songpress, con l'app ancora aperta:
wl-paste --list-types                 # deve comparire  image/png
wl-paste --type image/png > /tmp/t.png && xdg-open /tmp/t.png

Patch 8 — "Apri cartella template": crash explorer e cartella non scrivibile

File: src/songpressplusplus/MyPreferencesDialog.py, src/songpressplusplus/Globals.py, build_deb.sh

Problema (tre parti):

  1. Comando Windows su Linux. MyPreferencesDialog.OnOpenTemplatesFolder() lanciava subprocess.Popen(['explorer', path]), comando che su Linux non esiste: premendo Opzioni → Generale → "Apri cartella template" compariva il dialogo di errore [Errno 2] File o directory non esistente: 'explorer'.
  2. Cartella di destinazione sbagliata. La funzione puntava a templates/ accanto al pacchetto, cioè /usr/lib/python3/dist-packages/songpressplusplus/templates/: con l'installazione .deb quella cartella appartiene a root ed è in sola lettura per l'utente, che quindi non potrebbe comunque salvarvi i propri template. Peggio, os.makedirs(..., exist_ok=True) non solleva errore su una cartella già esistente di root, quindi il controllo di scrivibilità non scattava.
  3. Crash su templates/slides mancante. Globals.ListLocalGlobalDir() chiamava os.listdir() su entrambe le radici senza verificarne l'esistenza: se templates/slides/ non era presente nella cartella dati utente (perché templates/local_dir/ non era stata inclusa nella wheel), l'esportazione PowerPoint sollevava FileNotFoundError.

Fix:

  1. OnOpenTemplatesFolder() è ora multipiattaforma: os.startfile() su Windows (gestisce spazi e caratteri non ASCII e non lascia processi appesi, a differenza di Popen(['explorer', ...])), open su macOS, xdg-open su Linux/BSD (standard freedesktop: su KDE inoltra a Dolphin, su GNOME a Nautilus), lanciato con start_new_session=True e output su DEVNULL. Se xdg-open manca, ripiega su wx.LaunchDefaultApplication() e in ultima istanza suggerisce sudo apt install xdg-utils.
  2. Nuovo MyPreferencesDialog._get_templates_dir(), allineato alle radici che SongpressFrame._PopulateTemplateMenu() già scandisce: sceglie la prima cartella scrivibile tra glb.data_path/templates (cartella dati utente), glb.path/templates (pacchetto, valida solo per sorgenti/venv/portable) e ~/.Songpress++/templates (rete di sicurezza). Il nuovo helper _is_writable_templates_dir() fa un test esplicito con os.access(d, W_OK | X_OK), perché makedirs(exist_ok=True) da solo non rileva una cartella di sistema già esistente.
  3. Globals.ListLocalGlobalDir() salta le cartelle mancanti o illeggibili e gestisce data_path non ancora inizializzato (stessa tolleranza di _PopulateTemplateMenu()). Globals.InitDataPath() crea inoltre sempre templates/songs e templates/slides dentro la cartella dati utente, anche se templates/local_dir/ è incompleta o assente.

Percorso finale su Debian. Con il .deb, la cartella aperta dal pulsante è la cartella dati utente ricavata da wx.StandardPaths.GetUserDataDir():

~/.Songpress++/templates/
├── songs/     ← template di brani (.crd) → menu "File → Nuovo da template"
└── slides/    ← template PowerPoint (.pptx) → esportazione presentazioni

I file qui presenti hanno la precedenza sugli omonimi installati a livello di sistema dal pacchetto.

Packaging: xdg-utils (che fornisce xdg-open) è aggiunto tra i Depends del .deb. build_deb.sh verifica inoltre, dopo l'installazione della wheel, che templates/local_dir/templates/{songs,slides} e templates/{songs,slides} esistano nel pacchetto: se mancano le crea con un file segnaposto .keep (git non versiona le directory vuote) e stampa un avviso con lo snippet pyproject.toml per la correzione permanente.

Verifica

python3 - <<'PY'
import os
BASE = "src/songpressplusplus"
checks = [
    ("MyPreferencesDialog.py", "def _get_templates_dir",        "Patch 8a — helper cartella template"),
    ("MyPreferencesDialog.py", "def _is_writable_templates_dir","Patch 8b — test di scrivibilità"),
    ("MyPreferencesDialog.py", "xdg-open",                      "Patch 8c — apertura cartella su Linux"),
    ("Globals.py",             "if not os.path.isdir(folder):", "Patch 8d — ListLocalGlobalDir tollerante"),
    ("Globals.py",             "'templates', 'slides'",         "Patch 8e — sottocartelle create in InitDataPath"),
]
for filename, needle, desc in checks:
    path = os.path.join(BASE, filename)
    try:
        with open(path) as f:
            found = needle in f.read()
        print(f"{'✅' if found else '❌'} {desc}")
    except FileNotFoundError:
        print(f"⚠️  File non trovato: {path}")

# 'explorer' non deve più comparire da nessuna parte
p = os.path.join(BASE, "MyPreferencesDialog.py")
if os.path.isfile(p):
    bad = "'explorer'" in open(p).read()
    print(f"{'❌' if bad else '✅'} Patch 8f — nessun residuo di 'explorer'")
PY

Verifica che la dipendenza sia dichiarata nel pacchetto costruito:

grep -n "xdg-utils" build_deb.sh
dpkg-deb -f build_deb/songpressplusplus_*.deb Depends   # deve contenere xdg-utils

Applicazione sul file installato (senza ricostruire il .deb)

La patch riscrive interi metodi in due file: i sorgenti sono la fonte di verità.

SRC=~/Songpress_DEFINitiVO3/SongpressPlusPlus/src/songpressplusplus
DST=/usr/lib/python3/dist-packages/songpressplusplus
sudo cp "$SRC/MyPreferencesDialog.py" "$DST/MyPreferencesDialog.py"
sudo cp "$SRC/Globals.py"             "$DST/Globals.py"
sudo apt install xdg-utils

Test funzionale

# 1. xdg-open è presente?
command -v xdg-open || sudo apt install xdg-utils

# 2. quale gestore file verrebbe usato?
xdg-mime query default inode/directory      # es. org.kde.dolphin.desktop

# 3. la cartella dati utente esiste con le sue sottocartelle?
ls -la ~/.Songpress++/templates/            # deve mostrare songs/ e slides/

Poi, in Songpress++: Strumenti → Opzioni → Generale → "Apri cartella template". Deve aprirsi Dolphin (o il gestore file predefinito) su ~/.Songpress++/templates/, senza alcun dialogo di errore. Copiando un file .crd in songs/, questo compare al riavvio nel menu File → Nuovo da template.

Aggiornamento: la scelta della cartella aperta dal pulsante su Linux è stata rivista dalla Patch 9 (vedi sotto): ora viene mostrata la cartella templates/ del pacchetto installato. La cartella dati utente resta quella scrivibile usata per salvare temi e template personali.

Patch 9 — "Apri cartella template" su Linux: mostrare la cartella del pacchetto

File: src/songpressplusplus/MyPreferencesDialog.py

Problema: su Linux il pulsante Opzioni → Generale → "Apri cartella template" non apriva la cartella di installazione dei template, cioè

/usr/lib/python3/dist-packages/songpressplusplus/templates/

ma ripiegava sulla cartella dati utente (o comunque su un percorso diverso, mai la radice templates/ del pacchetto — e in nessun caso deve finire in una sua sottocartella come templates/local_dir/, che è solo lo scheletro copiato nella cartella dati al primo avvio).

La causa è nella Patch 8: OnOpenTemplatesFolder() usava _get_templates_dir(), che seleziona la prima radice scrivibile. Con l'installazione .deb la cartella del pacchetto è di root, quindi il test os.access(d, W_OK | X_OK) fallisce e la funzione ripiega sulla cartella dati utente. Ma per ispezionare una cartella nel file manager basta il permesso di lettura: la scrivibilità è un requisito del salvataggio (temi), non dell'apertura.

Fix: si separano le due esigenze invece di modificare _get_templates_dir() (che è usato anche da _get_themes_dir() per scrivere i temi, e che quindi deve continuare a restituire una cartella scrivibile).

Nuovo metodo MyPreferencesDialog._get_templates_dir_to_open():

  • su Linux, se glb.path/templates esiste (os.path.isdir()), la restituisce così com'è — nessun makedirs(), nessun test W_OK, quindi nessun ripiego indesiderato;
  • su Windows/macOS e come rete di sicurezza (albero sorgenti senza templates/), delega a _get_templates_dir(), mantenendo il comportamento precedente.

OnOpenTemplatesFolder() chiama ora _get_templates_dir_to_open() al posto di _get_templates_dir(). Nient'altro cambia: xdg-open/os.startfile/open, i fallback e la gestione degli errori restano quelli della Patch 8.

Conseguenza sui percorsi (Debian):

Azione Cartella
Pulsante "Apri cartella template" /usr/lib/python3/dist-packages/songpressplusplus/templates/ (sola lettura, contiene gli esempi distribuiti)
Salvataggio temi / template personali ~/.Songpress++/templates/ (scrivibile)

La lettura non cambia: _theme_roots() e SongpressFrame._PopulateTemplateMenu() scandiscono comunque entrambe le radici, con quella utente che ha la precedenza su quella di sistema.

Packaging: nessuna modifica a build_deb.sh — la patch vive interamente nei sorgenti.

Verifica

python3 - <<'PY'
import os, re
p = "src/songpressplusplus/MyPreferencesDialog.py"
src = open(p).read()
checks = [
    ("def _get_templates_dir_to_open", "Patch 9a — helper cartella da aprire"),
    ("path = self._get_templates_dir_to_open()", "Patch 9b — handler aggiornato"),
]
for needle, desc in checks:
    print(f"{'✅' if needle in src else '❌'} {desc}")
# il vecchio helper deve restare (serve a _get_themes_dir per SCRIVERE)
print(f"{'✅' if 'def _get_templates_dir(self)' in src else '❌'} Patch 9c — _get_templates_dir() conservato")
PY

Applicazione sul file installato (senza ricostruire il .deb)

SRC=~/Songpress_DEFINitiVO3/SongpressPlusPlus/src/songpressplusplus
DST=/usr/lib/python3/dist-packages/songpressplusplus
sudo cp "$SRC/MyPreferencesDialog.py" "$DST/MyPreferencesDialog.py"

Test funzionale

# la cartella che il pulsante deve aprire
ls -la /usr/lib/python3/dist-packages/songpressplusplus/templates/
# → fonts/  local_dir/  slides/  songs/  themes/

In Songpress++: Strumenti → Opzioni → Generale → "Apri cartella template". Dolphin deve aprirsi esattamente su quella cartella (non su .../templates/local_dir/, non su ~/.Songpress++/templates/), senza alcun dialogo di errore.

Patch 10 — Template: seeding della cartella utente al primo avvio

File: src/songpressplusplus/TemplateSeed.py (nuovo), src/songpressplusplus/MyPreferencesDialog.py, src/songpressplusplus/Globals.py

Sostituisce la Patch 9, che apriva la cartella templates/ del pacchetto: quella cartella mostra sì gli esempi, ma appartiene a root — l'utente non può crearci un proprio modello.

Problema: la cartella dei template del pacchetto e quella scrivibile dell'utente erano in conflitto irriducibile.

cartella contenuto scrivibile?
/usr/lib/python3/dist-packages/songpressplusplus/templates/ gli esempi distribuiti ❌ è di root
~/.Songpress++/templates/ vuota ✅

Aprire la prima → non ci si può creare nulla. Aprire la seconda → è deserta. Nessuna delle due è la risposta giusta, e spostare i file in templates/local_dir/ non cambierebbe nulla: resta sotto l'albero di sistema, resta di root, e per giunta local_dir/ ha già un suo significato (è lo scheletro local_dir/templates/{songs,slides} che build_deb.sh verifica).

Fix — seeding al primo avvio. Si popola la cartella utente con i template del pacchetto, così è al tempo stesso completa e scrivibile, e il pulsante torna ad aprire quella.

Perché non nel postinst del .deb. dpkg gira come root, prima che esista una sessione utente: non sa quale home usare (la macchina può avere più utenti) e i file che creasse in ~/.Songpress++/ sarebbero di proprietà di root, cioè di nuovo non scrivibili — esattamente il problema da cui si sta scappando. Il seeding va fatto dall'applicazione, al primo avvio per quell'utente. Dal punto di vista dell'utente il risultato è identico: dopo l'installazione trova tutto in ~/.Songpress++/templates/.

  1. Nuovo modulo TemplateSeed.py con seed_user_templates(pkg_templates, user_templates, force=False). Copia ricorsivamente tutto il contenuto di <pacchetto>/templates/ nella cartella utente:

    templates/            →  ~/.Songpress++/templates/
├── fonts/            →  ├── fonts/
├── slides/           →  ├── slides/
├── songs/            →  ├── songs/
├── themes/           →  ├── themes/
└── local_dir/        →  ├── .seeded   ← marcatore di idempotenza
    └── templates/       └── (il contenuto di local_dir/templates/ è fuso qui)
        ├── songs/
        └── slides/

    Garanzie:

    • non sovrascrive mai un file già presente → le personalizzazioni dell'utente sono intoccabili;
    • idempotente grazie al marcatore .seeded: i file che l'utente cancella volutamente non ricompaiono a ogni avvio (per rifare il seeding basta cancellare .seeded);
    • os.chmod(d, mode | 0o200) dopo ogni shutil.copy2(), perché copy2 preserva i permessi della sorgente (0644 di root nel .deb);
    • local_dir/ non viene ricreata nella home: sarebbe una duplicazione priva di senso. Il suo sotto-albero local_dir/templates/* viene fuso nella destinazione;
    • salta il caso pkg_templates == user_templates (portable / venv / albero sorgenti);
    • non solleva mai eccezioni: il seeding è un'agevolazione, non deve poter impedire l'avvio.

  2. MyPreferencesDialog: _get_templates_dir_to_open() non punta più al pacchetto — restituisce _get_templates_dir() (la radice scrivibile) dopo aver invocato il nuovo helper _seed_templates_dir(). Il seeding è ripetuto qui, oltre che all'avvio, perché è idempotente e a costo nullo: garantisce la cartella popolata anche aggiornando il solo MyPreferencesDialog.py.

  3. Globals: nuovo metodo SeedUserTemplates(force=False), invocato in coda a InitDataPath() (quindi a ogni avvio, ma effettivo solo la prima volta). Nuova costante USER_TEMPLATE_SUBDIRS = ('fonts', 'slides', 'songs', 'themes'): è TEMPLATE_SUBDIRS meno local_dir, perché quella cartella è lo scheletro che vive dentro il pacchetto e ricrearne una copia vuota nella home dell'utente sarebbe un doppione. _TEMPLATE_SUBDIRS di MyPreferencesDialog è allineata di conseguenza.

Packaging

TemplateSeed.py sta dentro il package, quindi entra nella wheel senza modifiche a pyproject.toml (i moduli .py del package sono inclusi automaticamente). Nessuna modifica a build_deb.sh.

Verifica

python3 - <<'PY'
import os
BASE = "src/songpressplusplus"
checks = [
    ("TemplateSeed.py",        "def seed_user_templates",          "Patch 10a — modulo di seeding"),
    ("TemplateSeed.py",        "_MARKER",                          "Patch 10b — marcatore di idempotenza"),
    ("MyPreferencesDialog.py", "def _seed_templates_dir",          "Patch 10c — helper nel dialogo"),
    ("MyPreferencesDialog.py", "path = self._get_templates_dir()", "Patch 10d — pulsante → cartella scrivibile"),
    ("Globals.py",             "seed_user_templates",              "Patch 10e — seeding all'avvio"),
]
for filename, needle, desc in checks:
    path = os.path.join(BASE, filename)
    try:
        found = needle in open(path).read()
        print(f"{{'✅' if found else '❌'}} {{desc}}")
    except FileNotFoundError:
        print(f"⚠️  File non trovato: {{path}}")
PY

Applicazione sul file installato (senza ricostruire il .deb)

SRC=~/Songpress_DEFINitiVO3/SongpressPlusPlus/src/songpressplusplus
DST=/usr/lib/python3/dist-packages/songpressplusplus
sudo cp "$SRC/TemplateSeed.py"        "$DST/TemplateSeed.py"
sudo cp "$SRC/MyPreferencesDialog.py" "$DST/MyPreferencesDialog.py"
sudo cp "$SRC/Globals.py"             "$DST/Globals.py"

Test funzionale

# 1. stato di partenza pulito (ATTENZIONE: cancella i template personali!)
rm -rf ~/.Songpress++/templates

# 2. avvia Songpress++, poi chiudi. La cartella deve essersi popolata:
find ~/.Songpress++/templates -maxdepth 2 | sort
#    → fonts/  slides/  songs/  themes/  .seeded  + i file distribuiti
#      (NIENTE local_dir/)

# 3. i file devono essere TUOI e scrivibili
ls -l ~/.Songpress++/templates/songs/
touch ~/.Songpress++/templates/songs/mio_modello.crd     # deve funzionare

# 4. idempotenza: cancella un file distribuito e riavvia — non deve tornare.
#    Per rifare il seeding completo:  rm ~/.Songpress++/templates/.seeded

In Songpress++: Strumenti → Opzioni → Generale → "Apri cartella template" apre ora ~/.Songpress++/templates/, piena dei template distribuiti e scrivibile: ci puoi creare i tuoi modelli, che sopravvivono agli aggiornamenti del .deb e hanno la precedenza sugli omonimi di sistema.

Patch 11 — Gtk-CRITICAL sui menu: SetBitmap prima di Append

File: tutti i sorgenti in src/songpressplusplus/ (la patch è generica)

Problema: all'avvio la console si riempie di messaggi come

(SongpressPlusPlus_bin:4662): Gtk-CRITICAL **: gtk_image_menu_item_set_image:
    assertion 'GTK_IS_IMAGE_MENU_ITEM (image_menu_item)' failed
(SongpressPlusPlus_bin:4662): GLib-GObject-WARNING **:
    invalid cast from 'GtkMenuItem' to 'GtkImageMenuItem'

uno per ogni voce di menu con icona. La documentazione di wxMenuItem è esplicita: SetBitmap() va chiamato prima che la voce venga aggiunta al menu; aggiungerla senza bitmap e impostarla dopo non è garantito che funzioni. Con l'ordine invertito wxGTK ha già creato un GtkMenuItem normale e poi prova a trattarlo come GtkImageMenuItem, da cui l'assertion fallita.

Nota: su GTK3 quelle icone non erano comunque visibili. GTK usa l'impostazione globale gtk-menu-images per decidere se mostrare immagini nei menu, ed è disattivata di default sui desktop moderni. La patch non cambia quindi nulla di visibile su Linux, mentre su Windows le icone restano funzionanti.

Fix: riordino della coppia di chiamate.

# prima
item = menu.Append(id, testo, help)
item.SetBitmap(bmp)

# dopo
item = wx.MenuItem(menu, id, testo, help)
item.SetBitmap(bmp)
menu.Append(item)

La trasformazione è sicura perché le firme combaciano: wxMenu::Append(id, text, help, kind) e wxMenuItem(parentMenu, id, text, help, kind) differiscono solo per il menu in prima posizione, quindi gli argomenti si trasferiscono invariati qualunque essi siano.

Verifica

A differenza delle altre, questa patch va verificata cercando ciò che non deve più esserci: la coppia Append → SetBitmap su righe consecutive.

python3 - <<'PY'
import re, subprocess

r = subprocess.run(["find", "src/songpressplusplus", "-name", "*.py"],
                   capture_output=True, text=True)
PAT = re.compile(
    r'^(?P<ind>[ \t]*)(?P<item>[A-Za-z_][\w.]*)[ \t]*=[ \t]*'
    r'(?P<menu>[A-Za-z_][\w.]*)\.Append\([^\n]*\)[ \t]*$\n'
    r'(?P=ind)(?P=item)\.SetBitmap\(', re.MULTILINE)

residui = 0
for path in r.stdout.split():
    with open(path, encoding='utf-8', errors='surrogateescape') as f:
        n = len(PAT.findall(f.read()))
    if n:
        print(f"KO {path}: {n} occorrenze da riordinare")
        residui += n
print("OK Patch 11 - nessun SetBitmap dopo Append" if not residui
      else f"KO Patch 11 - {residui} occorrenze residue")
PY

Output atteso:

OK Patch 11 - nessun SetBitmap dopo Append

Controprova a runtime, che disattiva il filtro del wrapper e mostra l'output grezzo:

SONGPRESS_VERBOSE=1 SongpressPlusPlus 2>&1 | grep -c gtk_image_menu_item
# atteso: 0

Applicazione manuale

La patch è applicata automaticamente da build_deb.sh (passo 1h) su tutti i .py del progetto. Per rifarla a mano è sufficiente lo stesso blocco di verifica con PAT.subn() al posto di PAT.findall(), oppure riordinare a mano le poche occorrenze che il blocco segnala.

Nota sulle codifiche. Non tutti i sorgenti sono UTF-8 validi (qualcuno è in latin-1). La patch li legge e riscrive con errors='surrogateescape', che garantisce un round-trip byte-per-byte lossless: nessun file cambia codifica. Per elencare i file non UTF-8:

for f in $(find src/songpressplusplus -name '*.py'); do
    iconv -f utf-8 -t utf-8 "$f" >/dev/null 2>&1 || echo "NON UTF-8: $f"
done

Rete di sicurezza nel wrapper

Se qualche chiamata sfugge alla patch (per esempio un Append spezzato su più righe), il wrapper installato scarta dalla console solo queste quattro righe:

messaggio perché è innocuo
gtk_image_menu_item_set_image GtkImageMenuItem è deprecato in GTK3 e gtk-menu-images è off di default
invalid cast from 'GtkMenuItem' to 'GtkImageMenuItem' stessa causa, altro messaggio
ScreenToClient cannot work when toplevel window is not shown log di livello Debug di wx, emesso chiedendo coordinate prima di Show()
gtk_combo_box_text_insert ... GTK_IS_COMBO_BOX_TEXT emesso solo alla chiusura: wxGTK ripopola/distrugge un ComboBox mentre il widget GTK sottostante è già in teardown, quindi non è più un combo valido

Non è un 2>/dev/null: qualsiasi altro output — errori, eccezioni, traceback Python — passa intatto, e il codice di uscita dell'applicazione è preservato. SONGPRESS_VERBOSE=1 disattiva il filtro.

Verifica rapida — tutte le patch in un colpo solo

python3 - <<'PY'
import os

BASE = "src/songpressplusplus"
FG = "wx.SystemSettings.GetColour(wx.SYS_COLOUR_WINDOWTEXT)"

checks = [
    ("SongpressFrame.py",      f'            title_lbl.SetForegroundColour({FG})',            "Patch 1a — titolo About leggibile"),
    ("SongpressFrame.py",      f'            lbl.SetForegroundColour({FG})',                   "Patch 1b — testo About leggibile"),
    ("MyPreferencesDialog.py", "        if not colour.IsOk():",                              "Patch 2  — crash colore"),
    ("PreferencesDialog.py",   "btnAssocAll.Disable()",                                      "Patch 3a — disabilita AssocAll"),
    ("PreferencesDialog.py",   "btnApply.Disable()",                                         "Patch 3b — disabilita Apply"),
    ("PreferencesDialog.py",   "cb.Disable()",                                               "Patch 3c — disabilita checkbox ext"),
    ("SongpressFrame.py",      "BG      = wx.SystemSettings.GetColour(wx.SYS_COLOUR_WINDOW)","Patch 4a — BG statistiche tema sistema"),
    ("SongpressFrame.py",      f"                    _lbl_verdict.SetForegroundColour({FG})","Patch 4b — verdetto multi-tab leggibile"),
    ("SongpressFrame.py",      f"            _lbl_verdict.SetForegroundColour({FG})",        "Patch 4c — verdetto singolo leggibile"),
    ("SongpressFrame.py",      f"                    k_lbl.SetForegroundColour({FG})",       "Patch 4d — testo _row multi-tab leggibile"),
    ("SongpressFrame.py",      f"            k_lbl.SetForegroundColour({FG})",               "Patch 4e — testo _row singolo leggibile"),
    ("SongpressFrame.py",      "getattr(self, '_mgr', None) is not None",                    "Patch 5a — guardia _mgr in _deferred_tb_update"),
    ("SongpressFrame.py",      "self._closing = True",                                       "Patch 5b — _closing in OnClose"),
    ("SongpressToolbars.py",   "if getattr(self, '_mgr', None) is None:",                    "Patch 5c — guardia _mgr in _FinalizeToolbarLayout"),
    ("i18n.py",                "_nolog = wx.LogNull()",                                      "Patch 6  — warning locale silenziato"),
    ("PreviewCanvas.py",       "def _load_copy_bitmap",                                      "Patch 7a — icona copy_2.png"),
    ("PreviewCanvas.py",       "def SetCopyCallback",                                        "Patch 7b — callback copia esposta"),
    ("PreviewCanvas.py",       "self._on_copy_callback",                                     "Patch 7c — _OnCopyButton usa la callback"),
    ("SongpressFrame.py",      "SetCopyCallback(self.CopyAsImage)",                          "Patch 7d — callback registrata"),
    ("SongpressFrame.py",      "def _copy_bytes_wayland",                                    "Patch 7e — copia via wl-copy su Wayland"),
    ("MyPreferencesDialog.py", "def _get_templates_dir",                                     "Patch 8a — helper cartella template"),
    ("MyPreferencesDialog.py", "def _is_writable_templates_dir",                             "Patch 8b — test di scrivibilità"),
    ("MyPreferencesDialog.py", "xdg-open",                                                   "Patch 8c — apertura cartella su Linux"),
    ("Globals.py",             "if not os.path.isdir(folder):",                              "Patch 8d — ListLocalGlobalDir tollerante"),
    ("Globals.py",             "'templates', 'slides'",                                      "Patch 8e — sottocartelle create in InitDataPath"),
    ("MyPreferencesDialog.py", "def _get_templates_dir_to_open",                             "Patch 9  — helper cartella da aprire"),
    ("TemplateSeed.py",        "def seed_user_templates",                                    "Patch 10a — modulo di seeding template"),
    ("MyPreferencesDialog.py", "def _seed_templates_dir",                                    "Patch 10b — seeding invocato dal dialogo"),
    ("Globals.py",             "seed_user_templates",                                        "Patch 10c — seeding all'avvio"),
]

for filename, needle, desc in checks:
    path = os.path.join(BASE, filename)
    try:
        with open(path) as f:
            found = needle in f.read()
        print(f"{'✅' if found else '❌'} {desc}")
    except FileNotFoundError:
        print(f"⚠️  File non trovato: {path}")
PY

Note sulle associazioni file

Il pacchetto .deb registra automaticamente tutte le estensioni ChordPro (.crd, .cho, .chordpro, .chopro, .pro, .sng) tramite il file /usr/share/mime/packages/songpressplusplus.xml e il campo MimeType= nel file .desktop. Non è necessario usare la scheda "Associazioni file" dentro Songpress++ — i pulsanti sono disabilitati automaticamente su Linux.

Build e installazione

chmod +x ~/Songpress_DEFINitiVO3/SongpressPlusPlus/build_deb.sh
cd ~/Songpress_DEFINitiVO3/SongpressPlusPlus
./build_deb.sh
sudo dpkg -i build_deb/songpressplusplus_*.deb

Note

  • Tutte le patch sono idempotenti: se già presenti nel sorgente, lo script le rileva e non le applica due volte.
  • Poiché le patch modificano l'albero sorgente in modo permanente, dalla seconda build in poi è normale leggere 0/3 fix o 1/3 fix: significa "già applicata", non "fallita". Per una conferma indipendente usa la sezione Verifica rapida qui sopra, oppure git diff.
  • Il build_deb.sh applica automaticamente tutte le patch prima di costruire la wheel (ordine: patch → wheel → install). Non è necessario applicare nessuna patch manualmente.
  • Importante: le patch devono essere applicate prima di pip wheel, non dopo. Se la wheel viene costruita prima delle patch, le modifiche al sorgente non entrano nel pacchetto installato.
  • Dopo aver applicato una patch al file installato (senza ricostruire il .deb), il fix è attivo immediatamente al prossimo avvio di Songpress++, senza reinstallare nulla.