Comment 2 for bug 362355

Nice branch!

[1]
+ run_command("smart-update --after %d" % self.smart_update_interval)

In our package, the command will probably be /usr/share/smart/smart-update. So maybe it would be nice to add /usr/share/smart to the PATH. That will be easier with my other suggestion: use twisted.internet.utils.getProcessOutput, which will give you a real Deferred for running a process.

[2] The smart update call in debian/landscape-client.init can be removed now.

[3] The cron file debian/landscape-client.cron.hourly can be removed as well.

Thanks!