You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by aw...@apache.org on 2015/07/28 19:46:49 UTC

[09/10] hadoop git commit: HADOOP-12273. releasedocmaker.py fails with stacktrace if --project option is not specified (Kengo Seki via aw)

HADOOP-12273. releasedocmaker.py fails with stacktrace if --project option is not specified (Kengo Seki via aw)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/7c92f0fe
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/7c92f0fe
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/7c92f0fe

Branch: refs/heads/HADOOP-12111
Commit: 7c92f0fe9176c55f7c16bc20211c854cdbfe0141
Parents: 0b6953a
Author: Allen Wittenauer <aw...@apache.org>
Authored: Tue Jul 28 10:43:22 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Tue Jul 28 10:43:22 2015 -0700

----------------------------------------------------------------------
 dev-support/releasedocmaker.py | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/7c92f0fe/dev-support/releasedocmaker.py
----------------------------------------------------------------------
diff --git a/dev-support/releasedocmaker.py b/dev-support/releasedocmaker.py
index d2e5dda..c59ae99 100755
--- a/dev-support/releasedocmaker.py
+++ b/dev-support/releasedocmaker.py
@@ -400,12 +400,6 @@ def main():
   (options, args) = parser.parse_args()
 
   if (options.versions is None):
-    options.versions = []
-
-  if (len(args) > 2):
-    options.versions.append(args[2])
-
-  if (len(options.versions) <= 0):
     parser.error("At least one version needs to be supplied")
 
   proxy = urllib2.ProxyHandler()
@@ -413,6 +407,8 @@ def main():
   urllib2.install_opener(opener)
 
   projects = options.projects
+  if projects is None:
+    parser.error("At least one project needs to be supplied")
 
   if (options.range is True):
     versions = [ Version(v) for v in GetVersions(options.versions, projects).getlist() ]