Project

General

Profile

Actions

action #16538

closed

[easy hack] improve isotovideo command line handling and help

Added by rpalethorpe almost 8 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2017-02-07
Due date:
% Done:

0%

Estimated time:

Description

Typing
$ ./isotovideo lsdjf

Only results in:
Can't open 'vars.json' for reading: 'No such file or directory' at qa/os-autoinst/bmwqemu.pm line 72
1672: EXIT 1

There is no warning that a command line parameter has been ignored, this is an issue with the following:
/isotovideo -d /tmp/vars.json

Intuitively I would expect /tmp/vars.json to be used as the test's parameters, instead it is ignored and the following message is returned
Can't open 'vars.json' for reading: 'No such file or directory' at qa/os-autoinst/bmwqemu.pm line 72
1734: EXIT 1

There are a number of possible actions, including:

  • A1, produce an error and help message on an unexpected command line argument
  • A2, treat the first command line argument (which does not start with '-') as the path to vars.json

Obviously A1 and A2 are not mutually exclusive.


Related issues 1 (0 open1 closed)

Related to openQA Project - action #16506: [easy hack] Use of uninitialized value with isotovideo --helpResolved2017-02-06

Actions
Actions #1

Updated by okurz almost 8 years ago

  • Category set to Feature requests
  • Target version set to future
Actions #2

Updated by okurz almost 8 years ago

  • Status changed from New to In Progress
  • Assignee set to okurz

https://github.com/os-autoinst/os-autoinst/pull/725 provides better command line handling and help. A2 is out of scope and I propose to not do it at all as we also provide load_vars() and save_vars() from the testapi and therefore would need to find a way to load/save vars consistently on the same file configuration. Could be done with saving the parameter to isotovideo in a package variable but … meh.

Actions #3

Updated by okurz over 7 years ago

  • Status changed from In Progress to Resolved

PR merged

Actions #4

Updated by dheidler over 7 years ago

  • Related to action #16506: [easy hack] Use of uninitialized value with isotovideo --help added
Actions #5

Updated by okurz over 6 years ago

  • Target version changed from future to future
Actions

Also available in: Atom PDF