Bugs item #1687513, was opened at 2007-03-24 12:31 Message generated for change (Comment added) made by justinrockwood You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=642714&aid=1687513&group_id=105970
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: msbuild Group: v3.0 >Status: Pending >Resolution: Fixed Priority: 5 Private: No Submitted By: Dave Cluderay (daveclud) >Assigned to: Justin Rockwood (justinrockwood) Summary: SuppressIces does not accept multiple ICEs. Initial Comment: I am trying to use the following property definition in my MSBuild Project: <PropertyGroup> ... <SuppressIces>ICE90;ICE61</SuppressIces> ... </PropertyGroup> If I just include one of the ICEs, everything is OK, and the LIGHT command line shown in the MSBuild output is correct. I expected though that the above syntax would result in two separate -sice arguments, i.e.: -sice:ICE90 -sice:ICE61 Instead, the command-line just inserts the property as-is: -sice:ICE90;ICE61 LIGHT doesn't understand this and does not suppress either ICE. I can't see any workaround for this, except to alter the source and recompile WixTasks.dll. Note that fudging my MSBuild project file to say... <SuppressIces>ICE90 -sice:ICE61</SuppressIces> ...does not work, since quotes are used to escape the value: -sice:"ICE90 -sice:ICE61" Note: In case you are interested, the reason I am suppressing ICE61 is that there seems to be a bug whereby it only allows the build version to be in the range 0-32767 (should be 0-65535). ---------------------------------------------------------------------- >Comment By: Justin Rockwood (justinrockwood) Date: 2007-04-06 10:33 Message: Logged In: YES user_id=1054914 Originator: NO Cullen fixed this for me. Thanks, Cullen. ---------------------------------------------------------------------- Comment By: Dave Cluderay (daveclud) Date: 2007-03-26 10:33 Message: Logged In: YES user_id=1752125 Originator: YES Note: I have since had a chance to test the code change to WixTasks. Changing the data type of the SuppressIces property to a string array and altering the GenerateCommandLineCommands method to use CommandLineBuilder.AppendArrayIfNotNull instead of CommandLineBuilder.AppendSwitchIfNotNull fixes the problem. I also tracked down the problem with ICE61 in darice.cub - the VBScript was incorrectly using a CInt instead of a CLng to parse the build portion of the version number. This may be fixed in a newer Platform SDK I guess. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=642714&aid=1687513&group_id=105970 ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ WiX-devs mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/wix-devs
