From: Alexander Kanevskiy Date: Tue, 13 Aug 2013 10:50:40 +0000 (+0300) Subject: Use common code for parsin/logging X-Git-Tag: submit/devel/20190730.075356~14 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fc7035cb50ef2ef43ddd7f5e8b0fa07b8e67cb30;p=services%2Fgerrithooks.git Use common code for parsin/logging --- diff --git a/gerrithooks/patchset_created.py b/gerrithooks/patchset_created.py index 60b912f..8248e39 100644 --- a/gerrithooks/patchset_created.py +++ b/gerrithooks/patchset_created.py @@ -20,11 +20,9 @@ patchset-created hook entry module """ -import os import sys import argparse -from gerrithooks.misc import configure_logging from gerrithooks.common import run_plugin_hooks def parse_cmdline(argv): @@ -51,20 +49,7 @@ def parse_cmdline(argv): def main(): """ patchset-created hook entry """ - # get parameters from command line - params = parse_cmdline(sys.argv[1:]) - - # set up logger - logger = configure_logging(os.path.basename(sys.argv[0])) - - logger.info("patchset-created: project %s : branch %s, commit %s, change: "\ - "%s (draft: %s url: %s) topic: %s uploader: %s patchset: %s", - params.project, params.branch, params.commit, params.change, - params.is_draft, params.change_url, params.topic, - params.uploader, params.patchset) - - - return run_plugin_hooks("patchset-created", params, logger) + return run_plugin_hooks("patchset-created", parse_cmdline) if __name__ == "__main__": sys.exit(main())