Re: RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Corinna Vinschen
Hi Nick, On Apr 23 10:41, Nicholas Clifton wrote: > Hi Corinna, > > >However, we know that the act of merging will currently result in broken > >resources in the executable. Wouldn't it be better to apply the above > >patch only after the resource merge fix? > > No. Well not in my opinion. :-)

Re: RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Nicholas Clifton
Hi Corinna, However, we know that the act of merging will currently result in broken resources in the executable. Wouldn't it be better to apply the above patch only after the resource merge fix? No. Well not in my opinion. :-) The reason is that this patch only makes a difference if the d

Re: RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Corinna Vinschen
[This time to everyone, not only to Kai, sorry] Hi guys, On Apr 23 11:08, Kai Tietz wrote: > 2014-04-23 11:06 GMT+02:00 Nicholas Clifton : > > Hi Kai, > > > > > >>>The default manifest file contains a resource section (.rsrc) holding > >>>information necessary for the binary to be run und

Re: RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Kai Tietz
2014-04-23 11:06 GMT+02:00 Nicholas Clifton : > Hi Kai, > > >>>The default manifest file contains a resource section (.rsrc) holding >>>information necessary for the binary to be run under Windows 8. It is >>>placed last on the linker command line so that a user provided >>>manifes

Re: RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Nicholas Clifton
Hi Kai, The default manifest file contains a resource section (.rsrc) holding information necessary for the binary to be run under Windows 8. It is placed last on the linker command line so that a user provided manifest, if there is one, will take precedence over the default mani

Re: RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Kai Tietz
Hello Nick, 2014-04-23 10:53 GMT+02:00 Nick Clifton : > Hi Guys, > > Please could I have permission to apply the patch below ? Ideally for > both mainline and the 4.9 branch. > > The patch adds a file called "default-manifest.o" to the end of a > final link command line for the Cygwin and

RFA: x86 backend: Add default-manifest to Cygwin/MinGW links

2014-04-23 Thread Nick Clifton
Hi Guys, Please could I have permission to apply the patch below ? Ideally for both mainline and the 4.9 branch. The patch adds a file called "default-manifest.o" to the end of a final link command line for the Cygwin and MinGW targets. The file is only added if it can be found in the