cvs commit: ports/sysutils/atop Makefile distinfo ports/sysutils/atop/files atop.in pkg-message.in

Alex Samorukov samm at os2.kiev.ua
Sun Jul 3 12:23:36 UTC 2011


Hello,

Thank you for comments.

On 07/03/2011 02:33 AM, Doug Barton wrote:
> Overall this looks good, just 2 small comments. It's not clear why it 
> uses:
>
> # REQUIRE: DAEMON
> # BEFORE: LOGIN
>
> Instead of just
>
> # REQUIRE: LOGIN
>
> like most ports scripts do.
Probably my copypaste from some other script. Better to replace with 
REQUIRE: LOGIN.

 > Also the default empty assignment for _flags is neither necessary or 
desirable.

Agree. Do i need to submit PR to fix this or you can do it without it?



More information about the cvs-all mailing list