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

OcAfterBootCompatLib: Avoid verbose boot overwriting graphics mode #424

Closed
wants to merge 1 commit into from

Conversation

mikebeaton
Copy link
Contributor

Addresses acidanthera/bugtracker#2225

I believe this is preferable to both the previous and the new behaviour, in that there is no overlap of verbose lines with graphics mode in any circumstance.

I also believe it is preferable to continue to omit the clear screen removed in 2abe076 (which is what is at issue here; NB this visible change is not being caused by a189bd5), as in normal boot progress it is unwanted, and a noticeable slow down on older, slower systems.

@mikebeaton mikebeaton marked this pull request as draft February 14, 2023 21:50
@mikebeaton
Copy link
Contributor Author

Apologies - give me a minute - I'm rushing - this doesn't appear to be launching correctly, in modes I tested before uploading it.

@mikebeaton
Copy link
Contributor Author

Redone to use existing code to properly check for -v in nvram or OC provided args.

@mikebeaton mikebeaton marked this pull request as ready for review February 14, 2023 22:23
@mikebeaton mikebeaton changed the title OcBootManagementLib: Avoid verbose boot overwriting graphics mode OcAfterBootCompatLib: Avoid verbose boot overwriting graphics mode Feb 14, 2023
@mikebeaton
Copy link
Contributor Author

Combining this into #426 .

@mikebeaton mikebeaton closed this Feb 16, 2023
@mikebeaton mikebeaton deleted the verbose branch February 16, 2023 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant