Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrected documentation errors #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jwrn3
Copy link

@jwrn3 jwrn3 commented Aug 12, 2021

  • Changed use_firmwarepassword to use_fwpw to correct KeyError exception.
  • Tidied boolean from No to false

Using the use_firmwarepassword as documented in README.md results in a KeyError:

Traceback (most recent call last):
File "/Users/uisadmin/./firmware_manager.py", line 1169, in
main()
File "/Users/uisadmin/./firmware_manager.py", line 1165, in main
FWPM_Object(args, logger, master_version)
File "/Users/uisadmin/./firmware_manager.py", line 211, in init
self.master_control()
File "/Users/uisadmin/./firmware_manager.py", line 229, in master_control
self.fwpwd_change()
File "/Users/uisadmin/./firmware_manager.py", line 554, in fwpwd_change
if not self.config_options["flags"]["use_fwpw"]:
KeyError: 'use_fwpw'

- Changed use_firmwarepassword to use_fwpw to correct KeyError exception.
- Tidied boolean from No to false
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant