Pull request is send.

 

From: Rob Mensching [mailto:r...@firegiant.com] 
Sent: Monday, August 11, 2014 10:23 PM
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] wix-devs@lists.sourceforge.net

 

Can you please send this as a pull request? It can be merged into the project 
correctly then.

 

_______________________________________________________________

FireGiant  |  Dedicated support for the WiX toolset  |   
<http://www.firegiant.com/> http://www.firegiant.com/

 

From: Norbert Schulze [ <mailto:norbert-schu...@online.de> 
mailto:norbert-schu...@online.de] 
Sent: Monday, August 11, 2014 1:14 PM
To:  <mailto:wix-devs@lists.sourceforge.net> wix-devs@lists.sourceforge.net
Subject: [WiX-devs]  <mailto:wix-devs@lists.sourceforge.net> 
wix-devs@lists.sourceforge.net

 

On 24-Nov-2013 I filed a bug that „Firewall port list or range is not possible“ 
http://wixtoolset.org/issues/4206/. This bug was than relabeled as feature 
request. IMO it’s nevertheless a bug, because all the other code that handles 
the firewall port parameter can already handle port lists as it uses the string 
type for the parameter. Only in firewall.cpp an integer is used. So can 
somebody with write access to the repository please check in the small attached 
patch which changes the type of the variable iPort from integer to string.

 

Best regards,

Norbert

 

------------------------------------------------------------------------------
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to