X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:subject:to:references:from:reply-to:message-id
	:date:mime-version:in-reply-to:content-type
	:content-transfer-encoding; q=dns; s=default; b=rL88UTavcecqEpUL
	JmVsv7Nsx7X3qIOmctFtTE44NqZbLQj0q7RPIY2mITlgG7p09NkhXTqnjEqbYzml
	9VUzzk6ZMlay5n+67CEQ2e3mLpMCYp7hngZdflJpeSv3/8IikjZ+gbIVFZz5WEcx
	1PuWrV2U5Ng8PNCfa5VUrwtM/yk=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:subject:to:references:from:reply-to:message-id
	:date:mime-version:in-reply-to:content-type
	:content-transfer-encoding; s=default; bh=8Ru7vsVQCl/k9AVzrjWtBK
	1LLqY=; b=xqGR7H4eiOp5f5RqNo97PTWnQWdOzRXeAioI+pyWxrzGPrJ6aBqeIp
	oht9JkK2ArwiZ/VEijF3uVjmMkdZxIQlMlZTGc0XLW/fkzPMH60LgVghPGHJjHyD
	l6rhz+Fp9jG43o/WpZ6s8bV6tqzFexJnADjTG9qpuwPQTS8NhW7E4=
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
Authentication-Results:	sourceware.org; auth=none
X-Virus-Found:	No
X-Spam-SWARE-Status:	No, score=-1.9 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.2 spammy=
X-HELO:	mail-it0-f43.google.com
X-Google-DKIM-Signature:	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:from:reply-to:message-id :date:user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=wHk9I9cdCjBr/9X4ol5m0Ej/Gw1JMiicJW3Fuyma/G4=; b=cwLyXNlLLovbF6L2ExNYh2UNbVQp21dwShVP4tJp/NXsUmAqHqYFw3Gb9t/MYSZKi9 +9+Kg0f5lc2LWg9f7y3+E3gJ/yiAnP4Sh8ZhnqYAy+cWt/xNoi9QxrnkVH6d/g5PfiY9 FEVqjKDiX0SuOSTRKCqZZG20YuPX0Wh73He1DZc9WuQWYqjiZNJLbK4hlRKyxjzK22QS aXP08yVrFb4lhlwdN/E5KkWMC4qBQREhWxOpMnf7ff/DeoyILy0aRJz+uSKM6fjdZjvn iIsGBmxlthltI3kWzpA3TB5EmhKGTDGnSw0ppfMuuQUQanXRGjTJCzDxKaGn33aUsY+8 F0rg==
X-Gm-Message-State:	APf1xPD6nTBdatr2TnEh/ucVjAAYI9x0+956EzQxvAWUewIfa5+NRt0X RUGsvYI8lb2Ge4X/UGAI24w4vg==
X-Google-Smtp-Source:	AG47ELsVwWg/4oqy8hVopN8Wwut1DTTLepQK6AkEq5IbQ0xFDR7lkWjdga6BL1UzdsPwRpJcMnYFEA==
X-Received:	by 10.36.148.201 with SMTP id j192mr4965192ite.139.1519740650085; Tue, 27 Feb 2018 06:10:50 -0800 (PST)
Subject:	Re: environ: fix link error on 64-bit Cygwin
To:	cygwin AT cygwin DOT com
References:	<36131639 DOT liFsmjX0ip AT omega> <0d2666a8-1037-fc9c-a17f-72cc59e9d977 AT redhat DOT com> <20180131144046 DOT GD4223 AT calimero DOT vinschen DOT de> <0d34d107-43ff-18f5-4284-b8a6ada9a927 AT cornell DOT edu>
From:	cyg Simple <cygsimple AT gmail DOT com>
Reply-To:	cygwin AT cygwin DOT com
Message-ID:	<8219adcd-85df-a6bb-b8d5-e6bf2fdb57f6@gmail.com>
Date:	Tue, 27 Feb 2018 09:10:36 -0500
User-Agent:	Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0
MIME-Version:	1.0
In-Reply-To:	<0d34d107-43ff-18f5-4284-b8a6ada9a927@cornell.edu>
X-IsSubscribed:	yes

On 2/26/2018 4:28 PM, Ken Brown wrote:
> On 1/31/2018 9:40 AM, Corinna Vinschen wrote:
>> On Jan 31 07:52, Eric Blake wrote:
>>> Forwarding from the gnulib list; is this something we should fix in
>>> cygwin proper?
>>
>> Why does anybody remove the -Wl,--disable-auto-import flag?  What for?
>> The idea was to never use this flag on Cygwin and to get rid of the
>> dllimport/dllexport nonsense...
>>
>>> On 01/31/2018 04:42 AM, Bruno Haible wrote:
>>>> On 64-bit Cygwin, a libunistring build fails like this:
>>>> [...]
>>>> libtool: link: x86_64-pc-cygwin-gcc -g -O2 -Wl,--disable-auto-import
>>>> -o .libs/test-environ.exe test-environ.o  -L/usr/local/cygwin64/lib
>>>> libtests.a ../lib/.libs/libunistring.dll.a -liconv libtests.a
>>>> -L/usr/local/cygwin64/lib
>>>> test-environ.o:test-environ.c:(.rdata$.refptr.environ[.refptr.environ]+0x0):
>>>> undefined reference to `environ'
>>>> collect2: error: ld returned 1 exit status
>>>>
>>>> This fixes it.
>>>> [...]
>>>> +_GL_EXTERN_C __declspec(dllimport) char **environ;
>>
>> But, other than that, an equivalent newlib patch would be ok.
> 
> Before patching newlib, please note that Bruno's patch doesn't work on
> 32-bit Cygwin.  Projects (like emacs and clisp) that use gnulib's unistd
> module with that patch will get link errors like this:
> 
>   undefined reference to `_imp__environ'
> 
> I have no idea why this error occurs only on 32-bit.
> 
> I've made Bruno aware of the problem, and I'm waiting to see what he
> comes up with.
> 

_imp__environ?  Shouldn't that be _imp_environ for Cygwin?
_imp__environ to me indicates the VC version of char **_environ instead.
 Is the correct header being used?

-- 
cyg Simple

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

- Raw text -