Re: pull request #829 (Reduced vs2010+ file sizes by removing redundant config settings)

[David Gregory]

Hi all,

I'm trying to understand why this change was made?

Old premake prior to this change:

<AdditionalOptions Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">/ZW /EHsc %(AdditionalOptions)</AdditionalOptions>

<AdditionalOptions Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">/ZW /EHsc %(AdditionalOptions)</AdditionalOptions>

New premake

<AdditionalOptions>/ZW /EHsc %(AdditionalOptions)</AdditionalOptions>

(Assuming that there are only Debug and Release configurations)

Although visual studio can certainly build with settings made this way (with reduced Conditional configurations), the IDE does not know how to actually EDIT these settings.

I know, your question would be - "you're using a project generation took like premake, so why would you need to edit the settings in Visual Studio"?  Well, one off changes to experiment before going into premake to hard bake something are a normal part of operation.  We always aim for full compatibility with Visual Studio.

Can someone explain the logic behind this change?

-Dave