Hi,

On Sat, Oct 29, 2016 at 9:27 AM, Jan Just Keijser <janj...@nikhef.nl> wrote:

> That apart, NAK to this patch as its not correct
>
> I'll see if I can refactor the patch - but your suggestion to put it at
> the top of add_option did not work for me initially.


Its not about refactoring. The patch simply causes definition of an env
variable named "opt" and does not treat what follows "setenv opt" as an
option. To handle the option correctly, setenv and opt should be stripped
before the parsing starts as is done when processed from config file.

Regards,

Selva
------------------------------------------------------------------------------
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive. 
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to