Project

General

Profile

Bug #652

windres and ranlib not detected when cross-compiling

Added by Carlo Bramini over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Minor
Assignee:
-
Category:
win32
Target version:
Start date:
July 18, 2016
Due date:
% Done:

100%

Estimated time:
Affects version:

Description

The mingw compiler provided with CYGWIN is called with i686-w64-mingw32 prefix.
The configure script is not able to detect windres and runlib here (but in my opinion it happens also with MSYS2): I think that you should replace AC_PATH_PROG with AC_PATH_TOOL in configure.ac, when it tries to check these two tools (lines 146-147).
I did this easy change, I regenerated configure and now it works fine.

Sincerely,

History

#1 Updated by John Lindgren over 8 years ago

  • Status changed from New to Rejected

We do not support Cygwin.

#2 Updated by Carlo Bramini over 8 years ago

John Lindgren wrote:

We do not support Cygwin.

Excuse me... my intention was not to ask a strict support for CYGWIN, probably I did a mistake in my request. What I wanted to say is that using AC_PATH_PROG instead of AC_PATH_TOOL for testing the presence of a tool is wrong, independently by the fact that it is running on CYGWIN, linux, Mac, etc.

Sincerely.

#3 Updated by John Lindgren over 8 years ago

  • Subject changed from winres and runlib not detected. to windres and ranlib not detected when cross-compiling
  • Category set to win32
  • Status changed from Rejected to Closed
  • Target version set to 3.8
  • % Done changed from 0 to 100
  • Affects version 3.7.2 added

Okay, I see the difference now, AC_PATH_TOOL is needed for cross-compiling.

#4 Updated by Carlo Bramini over 8 years ago

John Lindgren wrote:

Okay, I see the difference now, AC_PATH_TOOL is needed for cross-compiling.

Thank you for the fix.
It escaped to my attention that also AR needs to be tested with AC_PATH_TOOL because it's part of binutils. Perhaps I did not noticed it because AR from CYGWIN and AR from MINGW produce the same files, but it may be possible that it won't work with different architectures. I do not know if it could cause an error in some conditions, however I report it here as reminder.

#5 Updated by John Lindgren over 8 years ago

Fixed.

Also available in: Atom PDF