GNU bug report logs - #12993
Wrong icon for Cygw32-Emacs

Previous Next

Full log

Message #87 received at 12993-done <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 12993-done <at> debbugs.gnu.org
Subject: Re: bug#12993: Wrong icon for Cygw32-Emacs
Date: Mon, 08 Apr 2013 22:43:55 -0400

On 4/8/2013 4:26 PM, Ken Brown wrote:
> On 4/8/2013 1:20 PM, Eli Zaretskii wrote:
>>> Date: Mon, 08 Apr 2013 12:58:40 -0400
>>> From: Ken Brown <kbrown <at> cornell.edu>
>>> CC: dancol <at> dancol.org, 12993 <at> debbugs.gnu.org
>>>
>>> Thank you!  So I assume the correct fix is the following:
>>>
>>> === modified file 'nt/emacs.rc'
>>> --- nt/emacs.rc 2013-03-05 03:59:35 +0000
>>> +++ nt/emacs.rc 2013-04-08 16:55:12 +0000
>>> @@ -1,6 +1,6 @@
>>>    Emacs ICON   icons/emacs.ico
>>>    32649 CURSOR icons/hand.cur
>>> -#ifdef WIN64
>>> +#if defined (WIN64) || defined (__x86_64__)
>>>    1 24 "emacs-x64.manifest"
>>>    #else
>>>    1 24 "emacs-x86.manifest"
>>>
>>> Right?
>>
>> Yes.  But what about that linker directive in configure.ac -- is that
>> still required?  (I never understood why Cygwin needs that in the
>> first place.)
>
> The "-Wl,-bpe-i386" part is not necessary.  I just checked this on both
> Cygwin32 and Cygwin64.  I think it should be removed, since it's
> confusing to have an unnecessary directive (especially on Cygwin64,
> where the format is actually pe-x86-64).  But if both directives are
> removed, the icon doesn't show up.  I guess the comment right before
> that line in configure.ac explains why.

I've removed "-Wl,-bpe-i386" and made the change to nt/emacs.rc as bzr 
revision 112251.  I'm closing the bug report.  Thanks again for your 
help, Eli.

Ken

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.