Code review comment for lp:~deeptik/linaro-license-protection/publish-to-snapshot

111 +def main():
112 + uploads_path = '/srv3/snapshots.linaro.org/uploads/'
113 + target_path = '/srv3/snapshots.linaro.org/www/'
114 + uploads_path = '/tmp/uploads/'
115 + target_path = '/tmp/www/'

IMHO, it would be better to treat these as the constants, not as the variables, to be defined in the beginning of the script (like they were before r50).