This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: Cygwin Terminal icon broken on case-sensitive systems
- From: Andy Koppe <andy dot koppe at gmail dot com>
- To: cygwin-apps at cygwin dot com
- Date: Sat, 31 Dec 2011 10:15:31 +0000
- Subject: Re: Cygwin Terminal icon broken on case-sensitive systems
- References: <CAHWeT-b8q2dbsJ4T6UVAtYpzPaCzaemmD=1wP1AR1-zEfmQhrA@mail.gmail.com> <CAHWeT-ZX6sL1LMJsHQu1L7azLhwH1wwHMTLKpVCiQFNZBuAX4g@mail.gmail.com>
On 24 November 2011 16:10, Andy Koppe wrote:
> On 24 November 2011 15:42, Andy Koppe wrote:
>> I'm afraid there's a problem with the new "Cygwin Terminal" icon
>> (which I'm sorry not to have spotted when reviewing the change).
>>
>> Setup.exe creates the mintty shortcuts with '-i /Cygwin-Terminal.ico',
>> yet the icon file it creates is called 'cygwin-terminal.ico', i.e. all
>> lowercase. This means that users of systems with case sensitivity
>> enabled just get to see a dialog box saying "/usr/bin/mintty: could
>> not load icon from '/Cygwin-Terminal.ico'".
>
> Actually, I take it all back. I'd confused myself by still having a
> 'cygwin-terminal.ico' lying around from earlier experiments. Setup.exe
> does create it with the correct name.
>
>> Apparently at least two users have fallen foul of this already:
>>
>> http://georgik.sinusgear.com/2011/11/23/mintty-resizable-terminal-for-windows/
>
> Still leaves the question what's going on there.
I think I found what triggered this: with a non-default dpi setting,
for some reason the ExtractIconEx function used to load the icon
doesn't return a small version of the icon along with the normal one.
I (somewhat accidentally) fixed this in 1.0.3 by no longer insisting
on a small icon being returned, as it's optional when creating a
window class anyway.
Andy