Mailing-List: contact stdcxx-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: stdcxx-dev@incubator.apache.org
Message-ID: <45228275.5080100@roguewave.com>
Date: Tue, 03 Oct 2006 09:32:05 -0600
From: Martin Sebor <sebor@roguewave.com>
Organization: Rogue Wave Software
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.13) Gecko/20060417
MIME-Version: 1.0
To: stdcxx-dev@incubator.apache.org
Subject: Re: [PATCH] Scripts, generating solution and projects for MSVC/ICC
 [2]
References: <F92433E3D38672499C549EB615AFADA08EAFCD@exkiv.kyiv.vdiweb.com>
In-Reply-To: <F92433E3D38672499C549EB615AFADA08EAFCD@exkiv.kyiv.vdiweb.com>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Farid Zaripov wrote:
>>-----Original Message-----
>>From: Martin Sebor [mailto:sebor@roguewave.com] 
>>Sent: Monday, October 02, 2006 6:00 PM
>>To: stdcxx-dev@incubator.apache.org
>>Subject: Re: [PATCH] Scripts, generating solution and 
>>projects for MSVC/ICC [2]
>>
>>
>>>  Should we replace #ifdef _RWBUILD_std to #ifdef _RWSTD_LIB_SRC in 
>>>_defs.h?
>>
>>Yes. It should also key in on _RWSHARED.
> 
>   _RWSHARED is present in outer #ifdef:
> ---------------------
> #if (defined (_WIN32) || defined (_WIN64)) && \
>     (defined (RWDLL)  || defined (_RWSHARED))
> 
>   Commited here: http://svn.apache.org/viewvc?view=rev&revision=452509


Okay. FYI, the _WIN32 || _WIN64 conditional above (and in the
rest of our code) can be simplified to just _WIN32 since the
macro is guaranteed to be #defined when _WIN64 is #defined.

> 
> 
>>>  CXX, LD, AR variables are used at configure step only.
>>
>>And not when bulding the lib? Why not?
> 
>   Because we use MS VisualStudio development envorinment (devenv.exe)
> in build process. Devenv itself selects what tool should be invoked to
> compile
> the partucular source file (i.e. cl.exe for .c, .cpp files; rc for .rc
> files). We
> cannot change this behavior.

Gotcha! It would be good to mention this in comments for each
of these variables.

Martin