bug#50818: closed (28.0.50; .eln files need to be executable on Cygwin)

[GNU bug Tracking System]

Your message dated Mon, 27 Sep 2021 15:26:35 -0400
with message-id <cda9a764-a63b-eeba-053c-dc6ad235628c@cornell.edu>
and subject line Re: bug#50818: 28.0.50; .eln files need to be executable on 
Cygwin
has caused the debbugs.gnu.org bug report #50818,
regarding 28.0.50; .eln files need to be executable on Cygwin
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
50818: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=50818
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 28.0.50; .eln files need to be executable on Cygwin
>
>
>
>
> Date: 
>
> Sun, 26 Sep 2021 11:03:53 -0400
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0
>
>
>
>
> An installed emacs built with native compilation won't start on Cygwin because 
> the executable permissions have been removed from the .eln files.  This is done 
> in the install-eln target of the top-level Makefile.in, which installs the .eln 
> files with 644 permissions even though they were created with executable 
> permissions.
>
> The failure is in Cygwin's dlopen, which calls the windows function 
> GetModuleHandleExW.
>
> Would it hurt anything on other platforms if the .eln files were installed with 
> 755 permissions?  If so, I can submit a patch that does this just on Cygwin.
>
> Ken
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#50818: 28.0.50; .eln files need to be executable on Cygwin
>
>
>
>
> Date: 
>
> Mon, 27 Sep 2021 15:26:35 -0400
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0
>
>
>
>
> On 9/27/2021 3:12 PM, Eli Zaretskii wrote:
> > > From: Ken Brown <kbrown@cornell.edu>
> > > Cc: 50818@debbugs.gnu.org
> > > Date: Mon, 27 Sep 2021 14:52:15 -0400
> > >
> > > I decide to go ahead with a Cygwin-specific patch (attached).  If Andrea or
> > > someone else decides this would be good for all platforms, they can do that 
> > > later.
> > >
> > > OK to push?
> >
> > Yes, please.
>
> Done. Closing.
>
>
> --- End Message ---