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

Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

71 + if not (os.path.isdir(uploads_path) or os.path.isdir(build_dir_path)):
72 + print "Missing build path", build_dir_path
73 + return FAIL

The error message is incorrect - it checks 2 dirs, but reports problem as if applies to only one. Such cases are always great source of confusion and suffering (the latest one, (c) by Google: https://bugs.launchpad.net/bugs/949100)

review: Needs Fixing

« Back to merge proposal