[pkg-bacula-devel] bacula-tray-monitor

[Carsten Leonhardt]

Hi,

> I tested this and it look really nice. I also changed the
> bacula-tray-monitor.desktop to reference the Icon by name instead of
> path, so the correct Icon will always be selected from
> /usr/share/icons/*/apps/.

> I also choose to include a pre-exported PNG to get rid of the
> imagemagick build-dep, because including this monstrosity just to
> convert one file from SVG to PNG feels wrong. It also could create
> reproducibility issues if imagemagick decides to change their code in
> the future, resulting in slightly different images.

ok, I just have two questions:

 - Why do we include the icon in debian/additions when it's already in
   src/qt-console/images?

 - We should consider if it's ok for users to have different icons on
   different distributions. So far it seems that only Fedora is shipping
   the tray-monitor and they use the "generic.xpm" (judging from my list
   at the end of debian/TODO)

(I'm also amazed at how much time, energy and discussion we can put into
an icon...)

 - Carsten