Opened 12 years ago

Closed 8 years ago

Last modified 8 years ago

#38304 closed defect (fixed)

i386-mingw32-binutils @2.21-3_0: windres converts resource string to UTF-16 with wrong endianness, PATCH

Reported by: tuffnatty (Phil Krylov) Owned by: landonf (Landon Fuller)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: cooljeanius (Eric Gallager), mojca (Mojca Miklavec)
Port: i386-mingw32-binutils

Description

This is an old upstream bug: http://sourceware.org/bugzilla/show_bug.cgi?id=13409 The upstream maintainers are too slow, the patch is available for half a year already. The patch is attached.

Attachments (2)

Portfile-i386-mingw32-binutils.diff (496 bytes) - added by tuffnatty (Phil Krylov) 12 years ago.
Portfile patch
patch-binutils-winduni.c.diff (670 bytes) - added by tuffnatty (Phil Krylov) 12 years ago.
Source code patch

Download all attachments as: .zip

Change History (15)

Changed 12 years ago by tuffnatty (Phil Krylov)

Portfile patch

Changed 12 years ago by tuffnatty (Phil Krylov)

Source code patch

comment:1 Changed 12 years ago by tuffnatty (Phil Krylov)

Cc: phil.krylov@… added

Cc Me!

comment:2 Changed 12 years ago by larryv (Lawrence Velázquez)

Cc: phil.krylov@… removed
Keywords: haspatch maintainer added; mingw32 windres binutils removed
Owner: changed from macports-tickets@… to landonf@…
Port: i386-mingw32-binutils added; i386-mingw32msvc-binutils removed
Summary: i386-mingw32msvc-binutils @2.21.3_0 windres converts resource string to UTF-16 with wrong endianness, PATCHi386-mingw32-binutils @2.21-3_0: windres converts resource string to UTF-16 with wrong endianness, PATCH

Thanks for the ticket; in the future, please Cc relevant port maintainers. Also, as the ticket reporter, don’t have to Cc yourself.

comment:3 Changed 12 years ago by mf2k (Frank Schima)

Keywords: maintainer removed

comment:4 Changed 12 years ago by tuffnatty (Phil Krylov)

Thank you; I forgot to fill in the Cc when creating the ticket, and after creation, I could only click "Cc Me", without any possibility to Cc the port maintainer. Should I file another ticket regarding this Trac issue?

comment:5 in reply to:  4 Changed 12 years ago by jmroot (Joshua Root)

Replying to phil.krylov@…:

Thank you; I forgot to fill in the Cc when creating the ticket, and after creation, I could only click "Cc Me", without any possibility to Cc the port maintainer. Should I file another ticket regarding this Trac issue?

Not in our Trac, but you could report it at http://trac.edgewall.org/ if you think the permissions model should be more flexible.

comment:6 Changed 10 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:7 Changed 8 years ago by mojca (Mojca Miklavec)

Cc: mojca added

comment:8 Changed 8 years ago by mojca (Mojca Miklavec)

Version: 2.1.3

The ports seems to build fine on the buildbot at the moment.

See also:

#40174
MinGW-w64: add win64 cross-compiler
#51935
binutils: upgrade to 2.27 -> 2.29.1
#53061
i386-mingw32: remove ports, replaced by i686-w64-mingw32

I would be grateful for further feedback & whether or not this is still a problem.

Then again I would be in favour of removing this port, you can install i686-w64-mingw32-binutils now.

comment:9 Changed 8 years ago by tuffnatty (Phil Krylov)

1) This port did always build fine. The bug reported reveals itself on runtime of a program built using this binutils/windres version. 2) If mingw-w64-* work well finally, there is probably no need for mingw32. I am using a self-built port of mingw-w64 recently. 3) On the other hand, I see no reason to not accept the patch until all mingw32 users switch to mingw-w64, because it is in upstream, tested for ages already, and adding the patch to macports does not seem difficult.

comment:10 Changed 8 years ago by mojca (Mojca Miklavec)

Resolution: fixed
Status: newclosed

In ed25e98e/macports-ports:

i386-mingw32-binutils: upgrade to 2.25.1-1

  • Upgrade to version 2.25.1-1
  • Add license GPL-3+
  • Remove $Id$ line

Closes: #38304
See: #51935
See: #53061

comment:11 Changed 8 years ago by mojca (Mojca Miklavec)

I checked and the patch is already applied in the version of binutils used by i686-w64-mingw32-binutils as well as in the version of binutils as shipped by the MinGW project (2.25.1). I updated the port even (if it's going to be removed afterwards). I didn't bother modernizing it in any other way. I guess I could install it from binutils directly, but then again you would get exactly the same port as i686-w64-mingw32-binutils provides already, only using different names.

Last edited 8 years ago by mojca (Mojca Miklavec) (previous) (diff)

comment:12 Changed 8 years ago by mojca (Mojca Miklavec)

It fails to build on 10.6:

Undefined symbols:
  "_wcsncasecmp", referenced from:
      _rsrc_sort_entries in libbfd.a(peigen.o)

but I'm not going to worry why as the port from MinGW-w64 builds just fine.

comment:13 Changed 8 years ago by mojca (Mojca Miklavec)

In dd6de2b3/macports-ports:

i386-mingw32-binutils: upgrade to 2.25.1-1

  • Upgrade to version 2.25.1-1
  • Add license GPL-3+
  • Remove $Id$ line

Closes: #38304
See: #51935
See: #53061

Note: See TracTickets for help on using tickets.