Date: Thu, 11 Sep 2008 02:01:35 -0400
From: Christopher Faylor <cgf-use-the-mailinglist-please AT cygwin DOT com>
To: cygwin AT cygwin DOT com
Subject: Re: New experimental package: gcc4-4.3.0-1
Message-ID: <20080911060135.GA11495@ednor.casa.cgf.cx>
Reply-To: cygwin AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
References: <001001c91336$91e70d10$9601a8c0 AT CAM DOT ARTIMI DOT COM> <48C86CB3 DOT 8090909 AT byu DOT net> <20080911052824 DOT GA8377 AT ednor DOT casa DOT cgf DOT cx> <48C8AED3 DOT A6BA9F79 AT dessent DOT net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <48C8AED3.A6BA9F79@dessent.net>
User-Agent: Mutt/1.5.16 (2007-06-09)
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Precedence: bulk
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com

On Wed, Sep 10, 2008 at 10:38:27PM -0700, Brian Dessent wrote:
>Christopher Faylor wrote:
>>I haven't tried this with the actual released compiler yet but,
>>assuming it works the same way, mm I wrong or is there something wrong
>>with this compiler?  The code it creates seems to be correct (and
>>Cygwin is noticeably smaller) but I'd like to get rid of all of the new
>>warnings.  I can't get rid of this one because I don't see how to do
>>that.
>
>The warning is correct, technically.  When compiling strsig.cc, you
>want the variable to be exported, not imported.  When compiling
>everything else that includes signal.h, you want it to be imported.
>
>There are actually two things wrong here, as I see it: one, dllexport
>should not be necessary at all as the exports are controlled by the
>.def file.

sys_sigabbrev is not in cygwin.def.  There are several variables which
are not exported via cygwin.def.

>Two, you only want the dllimport attribute to be present when
>signal.h is included by clients, i.e. outside of Cygwin, so the
>__declspec should be turned off #if defined(__INSIDE_CYGWIN__).

Oh well.  I would have sworn that the dllimport/dllexport combination
was an accepted idiom but I can't find any documentation to support
that.  I should have just done some research before sending this email
but I thought I vaguely recalled that this was reported as a bug at some
point.

Thanks for the clarification.

cgf

--
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/