Bug#983218: cockpit: please use correct Debian logo

[Martin Pitt]

Control: reassign -1 debconf
Control: forcemerge 983200 -1

Hello again,

Martin Pitt [2021-02-23 15:48 +0100]:
> Philip Hands [2021-02-21 10:28 +0100]:
> > Rather than waiting for debconf to be updated to provide you with a new 
> > image to
> > play with, I'd suggest going to the source, and using the SVG here:
> > 
> >   https://www.debian.org/logos/openlogo-nd.svg
> 
> Thanks, very well spotted! That's what I did, and I rendered it (in GIMP) as
> 48x48. I submitted it here: 
> https://github.com/cockpit-project/cockpit/pull/15408

Sorry, this was a thinko. Our *source code* contains a
src/branding/debian/favicon.ico, but it's not actually being used. The deb just
installs it as a symlink to the generic distro-wide icon from debconf:

   lrwxrwxrwx 1 root root 32 Feb  4 14:48 
/usr/share/cockpit/branding/debian/favicon.ico -> 
../../../pixmaps/debian-logo.png

So I'm marking this a duplicate of the debconf bug, and will submit an upstream
PR to just remove the obsolete logo file from the source.

Thanks!

Martin

Bug#983218: cockpit: please use correct Debian logo

[Martin Pitt]

Hello Philip,

Philip Hands [2021-02-21 10:28 +0100]:
> Rather than waiting for debconf to be updated to provide you with a new image 
> to
> play with, I'd suggest going to the source, and using the SVG here:
> 
>   https://www.debian.org/logos/openlogo-nd.svg

Thanks, very well spotted! That's what I did, and I rendered it (in GIMP) as
48x48. I submitted it here: 
https://github.com/cockpit-project/cockpit/pull/15408

It is now 9 kB, instead of the previous 1, but it looked rather crappy when I
lowered the BPP and/or alpha depth. If you know how to make a better/smaller
one, I'd appreciate of course!

Thanks,

Martin

Bug#983218: cockpit: please use correct Debian logo

[Philip Hands]

Package: cockpit
Version: 238-1
Severity: minor

Dear Maintainer,

I notice that you're using a version of the Debian Logo that has been rendered
with a "Fat Tail".

I was surprised to note that this broken version was to be found in the debconf
package (which really ought to know better ;-) ) so it's hardly your fault.

As you can see, I've also reported a bug against debconf: #983200

You can clearly see the difference I'm talking about in the associated merge
request, here:

  
https://salsa.debian.org/pkg-debconf/debconf/-/merge_requests/6/diffs?commit_id=ec5148f84860643c4058c283a0c9cd94217d0849

Rather than waiting for debconf to be updated to provide you with a new image to
play with, I'd suggest going to the source, and using the SVG here:

  https://www.debian.org/logos/openlogo-nd.svg

to a) complete your debian branding, by adding an SVG version, and b) to
generate a PNG of the size you require (rather than apparently scaling the image
somehow, judging from what I see in the webUI).

As mentioned in the MR, I used inkscape to do the new PNG for debconf, and it
was no effort, so I'd cheerfully do the same here if you tell me what you need.

HTH

Cheers, Phil.

-- System Information:
Debian Release: 10.8
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable'), (99, 'unstable'), (99, 
'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 5.9.0-0.bpo.5-amd64 (SMP w/4 CPU cores)
Kernel taint flags: TAINT_CPU_OUT_OF_SPEC, TAINT_OOT_MODULE, 
TAINT_UNSIGNED_MODULE
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_GB:en (charmap=UTF-8)
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages cockpit depends on:
ii  cockpit-bridge  238-1
ii  cockpit-system  238-1
ii  cockpit-ws  238-1

Versions of packages cockpit recommends:
ii  cockpit-networkmanager  238-1
ii  cockpit-packagekit  238-1
ii  cockpit-storaged238-1

Versions of packages cockpit suggests:
pn  cockpit-doc   
pn  cockpit-machines  
pn  cockpit-pcp   
ii  xdg-utils 1.1.3-1+deb10u1

-- no debconf information