Mailing-List: contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-apps-owner AT cygwin DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT cygwin DOT com>
List-Help: <mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Mail-Followup-To: cygwin-apps AT cygwin DOT com
Delivered-To: mailing list cygwin-apps AT cygwin DOT com
Date: Mon, 20 May 2002 01:00:15 -0400
From: Christopher Faylor <cgf AT redhat DOT com>
To: cygwin-apps AT cygwin DOT com
Subject: Re: Packaging error with findutils-4.1.7-3
Message-ID: <20020520050015.GA17338@redhat.com>
Reply-To: cygwin-apps AT cygwin DOT com
Mail-Followup-To: cygwin-apps AT cygwin DOT com
References: <kvwuu0712t DOT fsf AT vzell DOT de DOT oracle DOT com> <20020520034102 DOT GA16589 AT redhat DOT com> <3CE87DA6 DOT 8070804 AT ece DOT gatech DOT edu>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <3CE87DA6.8070804@ece.gatech.edu>
User-Agent: Mutt/1.3.23.1i

On Mon, May 20, 2002 at 12:37:58AM -0400, Charles Wilson wrote:
>Christopher Faylor wrote:
>
>>On Sun, May 19, 2002 at 02:51:22PM +0200, Dr. Volker Zell wrote:
>>
>>>See subject.
>>>findutils-4.1.7-3 installs .mo files under /lib/locale/
>>>instead of /usr/share/locale/
>>>
>>
>>Thanks for the heads up.  Should be fixed.
>>
>>Enabling --auto-import for ld solved this problem.
>
>
>Huh? Wha...?
>
>How does enabling '--auto-import' for ld solve a problem with the 
>installation location of language translation tables?

Can't you imagine all sorts of scenarios where a configure script, which
assumes normal unix linking of external symbols, becoming confused if
--auto-import wasn't on.

>Second, why do you have to specify --auto-import for anything? 
>'--enable-auto-import' IS the default...

I applied the following patch to the sources:

Index: emultempl/pe.em
===================================================================
RCS file: /cvs/src/src/ld/emultempl/pe.em,v
retrieving revision 1.59
diff -u -p -r1.59 pe.em
--- emultempl/pe.em     15 Feb 2002 02:11:05 -0000      1.59
+++ emultempl/pe.em     20 May 2002 04:51:53 -0000
@@ -173,7 +173,7 @@ gld_${EMULATION_NAME}_before_parse()
 #ifdef DLL_SUPPORT
   config.dynamic_link = true;
   config.has_shared = 1;
-/* link_info.pei386_auto_import = true; */
+  link_info.pei386_auto_import = true;
 
 #if (PE_DEF_SUBSYSTEM == 9) || (PE_DEF_SUBSYSTEM == 2)
 #if defined TARGET_IS_mipspe || defined TARGET_IS_armpe

Trying this on Windows, I can see that it is on by default there.  I
guess this must have gotten wiped out in my sandbox when I rebuilt
binutils for linux.  Don't know how that happened.

So, nevermind about the question about turning it on by default.

The real question is whether this should be on in CVS.  Obviously
it should be.

cgf