black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Signed-off-by: Patrick Will

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I944f1915ece753f72a3fa654902d445a9749d0f9

show more ...

Revert "black: re-format"

This reverts commit 5731818de0ce446ceaafc7e75ae39da1b69942ae.

Signed-off-by: George Keishing <gkeishin@in.ibm.com>
Change-Id: Ie61cdc8c7f2825b0d9d66be87a6a3a058de2b372

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: If1010ead857d413

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: If1010ead857d41364c024bf8145a979a9377d382
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

Update python code for python 3.8 or higher compatibility

Changes:
- Update shebang directive to use python3 explicitly.
- remove preexec_fn feature which is deprecated in

Update python code for python 3.8 or higher compatibility

Changes:
- Update shebang directive to use python3 explicitly.
- remove preexec_fn feature which is deprecated in higher
python version 3.8 and above.
- bufsize fix, from documentation bufsize 1 means line buffered
(only usable if universal_newlines=True i.e., in a text mode)
refer: https://docs.python.org/3/library/subprocess.html
- Fix SyntaxWarning: "is" with "=="

Change-Id: Ib96c1ba8c2327d6774d0d731b48ca7e61a4a297a
Signed-off-by: George Keishing <gkeishin@in.ibm.com>

show more ...

Minor changes to python_pgm_template

Change-Id: Ia27858a0b3d45fba263c1b6725f8c2271231f12c
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

python_pgm_template exit_function fix

It is not correct for exit_function to have arguments since no
arguments are ever passed to it.

Change-Id: I4981e6071927e0ec8cb3337d2487a4b

python_pgm_template exit_function fix

It is not correct for exit_function to have arguments since no
arguments are ever passed to it.

Change-Id: I4981e6071927e0ec8cb3337d2487a4bd61857777
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

show more ...

Change several python and robot files to 110 chars

Taking advantage of current team limit of 110 chars.

Change-Id: If7ab51fe894889967b8c8bb2f2fa4664f01117d5
Signed-off-by: Micha

Change several python and robot files to 110 chars

Taking advantage of current team limit of 110 chars.

Change-Id: If7ab51fe894889967b8c8bb2f2fa4664f01117d5
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

show more ...

Improved py template

- General streamlining
- exit_function: New comments to indicate that this function is
optional
- signal_handler: Removed this function. This is now hand

Improved py template

- General streamlining
- exit_function: New comments to indicate that this function is
optional
- signal_handler: Removed this function. This is now handled by
gen_args' gen_signal_handler.
- validate_parms: New comments to indicate that this function is
optional
- Replaced old calls with a single call to gen_setup().

Change-Id: I8b19ea4242429330ec8a989269ea1fa5043bed26
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

show more ...

Alter main doc string for python_pgm_template

Change-Id: I068a2acbc53c60d5048807765b694ba5ffae9f0f
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

Change python_pgm_template to use exit_on_error mode

Change-Id: Ib7d64cfbcefe110a9187e4e62bcce7946b5a2931
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

Fixed PEP 8 style using autopep8

Used autopep8 to correct styling.
Ran autopep8 on openbmc-test-automation, autopep8 is not
able to fix all styling issues. There is still around

Fixed PEP 8 style using autopep8

Used autopep8 to correct styling.
Ran autopep8 on openbmc-test-automation, autopep8 is not
able to fix all styling issues. There is still around
1000 styling violations in openbmc-test-automation.
More information on autopep8 can be found here,
https://pypi.python.org/pypi/autopep8

Change-Id: Iddc131da1d74d978eb3dd0fdd6ce5d0a0e49b0f8
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>

show more ...

Change to ArgumentDefaultsHelpFormatter.

Changed from this:
formatter_class=argparse.RawTextHelpFormatter

To this:
formatter_class=argparse.ArgumentDefaultsHelpFormatter

Change to ArgumentDefaultsHelpFormatter.

Changed from this:
formatter_class=argparse.RawTextHelpFormatter

To this:
formatter_class=argparse.ArgumentDefaultsHelpFormatter

This greatly improves the formatting of the help text.

Change-Id: Ie3ad6afe0b3cda50e1b032c37a20ff22733de9bd
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

show more ...

Made updates to some template files.

Change-Id: Ib9ee0046f235040f647b975c78c618c6562501d6
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

bash_simple_template as model for bash programs.

Also, moved python_pgm_templates to new templates subdir.

Note: It should be the very rare case when we choose to code in bash.

bash_simple_template as model for bash programs.

Also, moved python_pgm_templates to new templates subdir.

Note: It should be the very rare case when we choose to code in bash.

Change-Id: I81813b8b3ecdd86a54bf8525fa8d4adb34a51bff
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>

show more ...