On Fri, Oct 1, 2010 at 15:07, Peter Volkov <[email protected]> wrote: > [a very thorough review of the openvpn ebuild]
Thanks for reviewing, I've fixed most of the issues.
>> if [[ -n $(ls /etc/openvpn/*/local.conf 2>/dev/null) ]] ; then
>
> I'd suggested [ -e /etc/openvpn/*/local.conf ] here, but probably there
> are better alternatives. Also ${ROOT} is missed here.
I've put ${ROOT} in, are there no better alternatives? I don't think
anyone mentioned any.
Cheers,
Dirkjan
