On 2012/03/11 16:57, Gonzalo L. R. wrote:
> Hi,
> 
> Update for zkt, with change of default timers; Bug fixes; New option
> -M for zkt-ls.
> 
> Ok? Comments?

a few comments:

> Content-Type: text/plain; charset=ISO-8859-1; format=flowed

- format=flowed breaks diffs, please see
https://www.kernel.org/doc/Documentation/email-clients.txt
everything in that document applies equally for OpenBSD patches

- this port has a maintainer, you should at least cc them

- you should update the patches (yes, it builds without, but skipping
this usually means extra work in the future)

- doesn't need groff

Reply via email to