On Thu, Dec 03, 2020 at 10:26:20AM +0800, Lu Weitao wrote:
[Why] It's more friendly to user that command-line tool have a help option
Signed-off-by: Lu Weitao luweitaobe@163.com
sbin/mail-job | 29 +++++++++++++++++++++++++---- 1 file changed, 25 insertions(+), 4 deletions(-)
diff --git a/sbin/mail-job b/sbin/mail-job index 954c149..85e37a1 100755 --- a/sbin/mail-job +++ b/sbin/mail-job @@ -3,11 +3,32 @@ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved. # frozen_string_literal: true
-# Usage: -# mail-job job_id
+require 'optparse' require_relative '../lib/mail_job_result.rb'
Maybe it's better to use absolute paths.
Thanks, Chenglong
-job_id = ARGV[0] +# mail job results to job submitter +# require the job had submitted job.yaml include email
+opt_parser = OptionParser.new do |opts|
- opts.banner = 'Usage: mail-job job_id'
- opts.separator 'eg: mail-job crystal.123012'
- opts.separator ''
- opts.separator 'options:'
- opts.on_tail('-h', '--help', 'show this message') do
- puts opts
- exit
- end
+end
+argv = if ARGV == []
['-h']elseARGVend+opt_parser.parse!(argv)
+job_id = argv[0] mail = MailJobResult.new(job_id) mail.send_mail -- 2.23.0