X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-3.1 required=5.0 	tests=AWL,BAYES_00,J_CHICKENPOX_82,RCVD_IN_DNSWL_LOW,SPF_PASS
X-Spam-Check-By: sourceware.org
Message-ID: <49CB0579.9090701@cwilson.fastmail.fm>
Date: Thu, 26 Mar 2009 00:32:57 -0400
From: Charles Wilson <cygwin AT cwilson DOT fastmail DOT fm>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.21) Gecko/20090302 Thunderbird/2.0.0.21 Mnenhy/0.7.6.666
MIME-Version: 1.0
To: Cygwin Mailing List <cygwin AT cygwin DOT com>
Subject: Re: libtool: two cygwin patches
References: <49CAC092 DOT 8030600 AT users DOT sourceforge DOT net>
In-Reply-To: <49CAC092.8030600@users.sourceforge.net>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

Yaakov (Cygwin/X) wrote:
> I attached two patches for libtool for your consideration:
> 
> 1) 2.2-export-all-symbols.patch:
> 
> On Cygwin, the --export-all-symbols linker flag is required;
> --export-dynamic has no effect (see
> http://sourceware.org/bugzilla/show_bug.cgi?id=6744).
> 
> This patch fixes two problems:
> 
> 1) LT_SYS_DLOPEN_SELF returns a false negative;

Thanks for looking in to this; this failure has always bugged me because
I know cygwin supports it, but I've never had the time to track it down.

> 2) Using the -export-dynamic libtool flag does not affect the resulting
> binary.

I don't know of a better solution; I'm leery of turning on
--export-all-symbols willy-nilly, because that auto-exports everything
even if you've carefully declspec'ed only specific symbols.

But, better working than not.

> 2) 2.2-gcc4-libs.patch:
> 
> With gcc4 providing shared libs, it should now perfectly legitimate to
> add any of these libs to the libtool link command.  Right now, though,
> libtool can't find these libraries because $sys_lib_search_path_spec is
> hard-coded to ignore it, and libtool will refuse to link against any
> library it can't find (even though the linker itself can).
> 
> Instead, this patch simply adds /usr/lib/w32api to the default
> gcc-specific search path.  The m4_if() makes sure that w32api is added
> only once, as is done with Darwin a few lines later.

Works for me.  Thanks for these patches.  Did you already send them
upstream?

I'd planned to respin libtool relatively soon, to remove some of Peter
Rosin's patches that I included in our version, but which he has now
said are not really ready for widespread distribution.  I'll bump that
higher on my priority queue.

Unless we go to package freeze.

--
Chuck

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/