[Bug binutils/4933] windres does not escape path with spaces in them

2016-03-24 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=4933

--- Comment #12 from Nick Clifton  ---
Hi Emmanuel,

> Looking at the patch it seems it would do what it supposed to do. Is there a
> way to get a binary version ?

Sorry - I do not have access to a mingw development environment.  Where do you
get your binaries from - maybe they could help create a new one for you ?

I could possibly build a 64-bit cygwin targeted version of windres.  Would that
help ?

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/19531] ld fails to build proper executables in several cases on x64_64-w64-mingw32

2016-03-24 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19531

--- Comment #19 from Nick Clifton  ---
(In reply to Rainer Emrich from comment #18)

> No, there is no difference for x86_64-w64-mingw32 AFAIS.

Oh bananas!

Any idea what is special about libgomp ?  I am not familiar with it myself, and
I am wondering if the failures are due to something completely unrelated to
linker garbage collection.  (Do the failures go away if tested with
-Wl,-no-gc-sections ?)

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/19803] gc-sections breaks PE DLL variable export

2016-03-24 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19803

Nick Clifton  changed:

   What|Removed |Added

   Attachment #9104|0   |1
is obsolete||
   Attachment #9106|0   |1
is obsolete||
   Attachment #9107|0   |1
is obsolete||
   Attachment #9115|0   |1
is obsolete||

--- Comment #19 from Nick Clifton  ---
Created attachment 9121
  --> https://sourceware.org/bugzilla/attachment.cgi?id=9121&action=edit
Revised patch

Hi Martin,

> First, 41f46ed9fea1a066de95b6a85c56393beef0b8b8 (Fix possible failure in 
> the AVR linker tests) introduced (propably accidentically)

Yes, that was a snafu.  

> In my option, this change is not necessary, as it just would hides broken 
> symbols in the export library.

Actually I think that it is needed in order to prevent garbage collected
symbols from leaking into the export table.

> The prefix name patch would need a xstrdup, as name is freed some lines below.

Doh, yes.

> There is second case missing (symbol matching implemented by 
> pe_fixup_stdcalls).

OK - this revised patch contains an attempt to fix this function as well. 
Please could you give it a try and let me know the results.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils