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

ima_emulator: specify sys.argv as the named parameter argv in main() #809

Merged
merged 1 commit into from Dec 16, 2021

Conversation

sergio-correia
Copy link
Contributor

Fixes the following issue, which followed-up from b0dcaa8:

Traceback (most recent call last):
  File "/usr/local/bin/keylime_ima_emulator", line 33, in <module>
    sys.exit(load_entry_point('keylime==0.0.0', 'console_scripts', 'keylime_ima_emulator')())
TypeError: main() missing 1 required positional argument: 'argv'

Other utilities under cmd/ also use this approach.

Signed-off-by: Sergio Correia scorreia@redhat.com

Fixes the following issue, which followed-up from b0dcaa8:

Traceback (most recent call last):
  File "/usr/local/bin/keylime_ima_emulator", line 33, in <module>
    sys.exit(load_entry_point('keylime==0.0.0', 'console_scripts', 'keylime_ima_emulator')())
TypeError: main() missing 1 required positional argument: 'argv'

Other utilities under cmd/ also use this approach.

Signed-off-by: Sergio Correia <scorreia@redhat.com>
Copy link
Member

@THS-on THS-on left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I missed that during my rewrite of that script.

@mpeters mpeters merged commit 1c811c9 into keylime:master Dec 16, 2021
@sergio-correia sergio-correia deleted the ima-emulator branch December 16, 2021 14:33
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.

None yet

5 participants