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

Uefi support #23

Closed
wants to merge 4 commits into from
Closed

Uefi support #23

wants to merge 4 commits into from

Conversation

jpshivakavi
Copy link

This PR is to add support for UEFI based use cases in XmlCli.
It makes use of the Python UEFI interpreter available in edk2-libc open source project.

@jpshivakavi jpshivakavi requested a review from gahan9 as a code owner December 19, 2023 05:01
@@ -12,10 +12,12 @@

__all__ = ["CommonCliAccess"]

MAX_READ_SIZE = 0x400

Check notice

Code scanning / CodeQL

Unused global variable Note

The global variable 'MAX_READ_SIZE' is not used.

def validate_xmlcli_request(self):
clb.InitInterface()
status = 1

Check warning

Code scanning / CodeQL

Variable defined multiple times Warning

This assignment to 'status' is unnecessary as it is
redefined
before this value is used.

with open(filename, 'rb') as in_file: # opening for [r]eading as [b]inary
data = in_file.read() # if you only wanted to read 512 bytes, do .read(512)
size = len(data)

Check notice

Code scanning / CodeQL

Unused local variable Note

Variable size is not used.
@jpshivakavi
Copy link
Author

it needs additional work from xmlcli code base hence closing it for now

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