> +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