> +1

:)

>> +    if ($self->{opts}->{'no-httpd'}) {
>> +        warning "skipping configuration: -no-httpd specified";
> 
> 
> may be it's better to say"skipping httpd configuration"?

sure, that's fine.

>> +        return
> 
> 
> please don't forget ; if } is on the next line.

oops :)

> may be use -nohttpd, so that you don't need to quote the key string?

I did it that way to mirror run-tests and others, thinking that using a
hyphen to join words was the norm for this project.  but either is fine with me.

--Geoff

Reply via email to