[Bug #127739] Bugfix facile

[DktrKranz]

Su Launchpad è stato aperto il bug #127739, il quale richiede di inserire un'icona nel pacchetto nvtv. Oltre all'attività di bugfix in quanto tale, occorre anche cercare un'icona da poter utilizzare, siccome il pacchetto non la fornisce. Chi si vuol cimentare?

[DktrKranz]

Per quanto riguarda l'icona, ne ho trovate due che potrebbero andare bene:

• http://www.it.com.cn/f/diy/0411/15/20041111_NVTV_447_12.jpg
• http://www.hwupgrade.org/userimages/NVTV_3D-125.jpg

Occorre esportarle in formato xpm con risoluzione 32x32, usando GIMP non dovrebbe essere un problema.

Ovviamente se avete altre soluzioni in merito (usando l'icona presente in un altro pacchetto relativo ai driver nVidia, per esempio) o se trovate l'icona "ufficiale", è meglio

[Rospo Zoppo]

questo bug si riferisce a feisty vero ?

[DktrKranz]

I bug si riferiscono sempre alla versione di sviluppo, a meno di procedure particolari.

[Rospo Zoppo]

perchè dice

On Kubuntu feisty with nvtv 0.4.7-5

e il pacchetto lo ho trovato solo per Feisty 

[DktrKranz]

L'utente lamenta il fatto che non ci sia l'icona per Feisty, ma siccome il pacchetto è il medesimo per Gutsy, il problema è ancora presente.
In questo caso c'è poco che si possa fare per Feisty (uno SRU è fuori luogo, al limite un backport), ma deve essere risolto in Gutsy (e segnalato a Debian).

[Rospo Zoppo]

ho capito... comunque ho provato (per adesso in feisty ad aggiungere queste righe al file rules

Codice: Seleziona tutto

install -D -m 644 $(CURDIR)/debian/nvtv.desktop \
   $(CURDIR)/debian/xcdroast/usr/share/applications/nvtv.desktop
install -D -m 644 $(CURDIR)/debian/nvtv.png \
   $(CURDIR)/debian/xcdroast/usr/share/pixmaps/nvtv.png

ma mi dice missing separator.. inoltre il file rules è strano http://paste.ubuntu-nl.org/34035/

[DktrKranz]

La quasi totalità dei pacchetti usa debhelper per la gestione della pacchettizzazione. Questo usa cdbs, che è più semplice ma meno potente.

In questo caso per installare uno o più file è necessario agire sul file debian/nvtv.install aggiungendo in fondo la riga

Codice: Seleziona tutto

debian/nvtv.xpm usr/share/pixmaps/

Ovviamente va modificato anche debian/nvtv.dirs, in modo che venga creta pixmaps:

Codice: Seleziona tutto

usr/share/pixmaps

Questo pacchetto richiede ulteriore lavoro. Dovrai modificare il file .desktop. Ad una prima analisi, devi aggiungere la voce

Codice: Seleziona tutto

Icon=nvtw

per visualizzare l'icona, ma desktop-file-validate ti indicherà anche di rimuovere Encoding=UTF-8 e Application; (da Categories=), obsoleti. Un file desktop corretto (salvo mie sviste, ma controlla sempre con desktop-file-validate) dovrebbe essere il seguente:

Codice: Seleziona tutto

[Desktop Entry]
Version=1.0
Name=Nvtv TV Out
Comment=Control the TV output of NVidia video cards
Comment[es]=Controle la salida de TV de tarjetas de video NVidia
Exec=nvtv
Icon=nvtv
Type=Application
Categories=AudioVideo;

Per quanto riguarda la dimensione dell'icona, non sono sicuro che la dimensione corretta sia 32x32, controllerò.

[Rospo Zoppo]

Ok, va bene se l'icona è png ?

[DktrKranz]

Certo, l'importante è che sia in un formato leggibile dal desktop environment, e PNG lo è.

[Rospo Zoppo]

comunque desktop-file-validate mi chiede l'encoding

[DktrKranz]

Probabile, se hai la versione di Feisty.

Codice: Seleziona tutto

root@moses:/# apt-cache policy desktop-file-utils
desktop-file-utils:
  Installed: 0.13-0ubuntu2
  Candidate: 0.13-0ubuntu2
  Version table:
 *** 0.13-0ubuntu2 0
        500 http://archive.ubuntu.com gutsy/main Packages
        100 /var/lib/dpkg/status
root@moses:/#

Questo è l'output di Gutsy con il .desktop originale:

Codice: Seleziona tutto

root@moses:/# cat nvtv.desktop
[Desktop Entry]
Version=1.0
Encoding=UTF-8
Name=Nvtv TV Out
Comment=Control the TV output of NVidia video cards
Comment[es]=Controle la salida de TV de tarjetas de video NVidia
Exec=nvtv
Type=Application
Categories=Application;AudioVideo;TV
root@moses:/# desktop-file-validate nvtv.desktop
nvtv.desktop: warning: key "Encoding" in group "Desktop Entry" is deprecated
nvtv.desktop: error: value "Application;AudioVideo;TV" for string list key "Categories" in group "Desktop Entry" does not have a semicolon (';') as trailing character
root@moses:/#

e questo con quello modificato:

Codice: Seleziona tutto

root@moses:/# cat nvtv.desktop
[Desktop Entry]
Version=1.0
Name=Nvtv TV Out
Comment=Control the TV output of NVidia video cards
Comment[es]=Controle la salida de TV de tarjetas de video NVidia
Exec=nvtv
Icon=nvtv
Type=Application
Categories=AudioVideo;TV;
root@moses:/# desktop-file-validate nvtv.desktop
root@moses:/#

[Rospo Zoppo]

tra poco provo da gutsy, comunque per il file rules ho sbagliato ?

[DktrKranz]

Il rules in questo caso non va toccato.

[Rospo Zoppo]

ho appena virtualizzato gutsy ma il pacchetto non è presente nei repo, uso quello di feisty ?

[DktrKranz]

E' il medesimo, usa pure quello.

[Rospo Zoppo]

mi viene scritto:

Codice: Seleziona tutto

dpkg-source: cannot represent change to debian/nvtv.png: binary file content changed

[DktrKranz]

Le immagini PNG sono trattate come oggetti binari e non possono far parte del diff.gz. Devi usare XPM o SVG.
Sorry, l'ho dimenticato

[Rospo Zoppo]

oh ecco perchè 

[Rospo Zoppo]

ho modificato i file e ho dato debuild -S... devo usare pbuilder ora ?