Skip to content
New issue

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

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

Already on GitHub? # to your account

Drop most uses of SystemTable #400

Merged

Conversation

nicholasbishop
Copy link
Contributor

Continuing to smooth the way for updating to uefi-0.32.

Temporarily use `table::system_table_boot` to keep this change focused
on removing parameters; `SystemTable` will be fully removed from
pe_loader in the next commit.
Change the `entry` function pointer type to use a `c_void` pointer
instead of `SystemTable`, and use `table::system_table_raw` to get that
pointer.
Temporarily use `table::system_table_boot` to get the SystemTable. This
is done rather than using `boot::get_image_file_system` because the
`FileSystem` type in uefi-0.31 only supports the old
`ScopedProtocol`. This can be simplified when updating to uefi-0.32.
Copy link
Member

@blitz blitz left a comment

Choose a reason for hiding this comment

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

Thank you for putting so much work in to get the uefi crate updated. It's really appreciated!

@blitz blitz enabled auto-merge October 15, 2024 21:46
@blitz blitz added this pull request to the merge queue Oct 15, 2024
Merged via the queue into nix-community:master with commit bd01eac Oct 15, 2024
2 checks passed
@nicholasbishop nicholasbishop deleted the bishop-replace-system-table branch October 15, 2024 22:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants