I set logs_limit to a positive value so that it's more user friendly, and the idea is that the fact that internally read_file is used with a negative value should be an implementation detail.
The docstring for logs_limit sounds clear enough to me:
+ @cvar logs_limit: When reporting upgrade-tool logs to the server, only the
+ last C{logs_limit} lines will be sent.
Sidnei, maybe the code is confusing, but it's actually the *end* of the file:
+ content = read_file(filename, - self.logs_limit)
I set logs_limit to a positive value so that it's more user friendly, and the idea is that the fact that internally read_file is used with a negative value should be an implementation detail.
The docstring for logs_limit sounds clear enough to me:
+ @cvar logs_limit: When reporting upgrade-tool logs to the server, only the
+ last C{logs_limit} lines will be sent.