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

Add info about HideAuxiliary #338

Closed
wants to merge 6 commits into from
Closed

Conversation

tarbaII
Copy link

@tarbaII tarbaII commented Feb 6, 2022

Added info about HideAuxiliary to all Intel desktop, laptop, HEDT, and AMD config guides.

@1Revenger1
Copy link
Member

Would it be better to leave it enabled and put a note on the installation page? Hiding recovery is generally useful aside from the initial installation.

@tarbaII
Copy link
Author

tarbaII commented Feb 6, 2022

As a post-install step, people usually transfer their EFI to their drive's ESP anyway, which involves deleting the com.apple.recovery.boot folder. Or they can just delete it without transferring their EFI. I also mentioned that if the user would like to hide those entries, they can enable it. If you'd still like me to change it though, I can.

@1Revenger1
Copy link
Member

Recovery would still be shown for installed disks, plus auxiliary options (such as UEFI Shell, NVRAM Reset, etc if set). I think most people would prefer to hide entries they don't normally use, though I'm not sure what the best option is here in this specific case.

You've probably helped more people out recently, I'll defer to you about it I guess.

Also, we already have a note on the install process page. Might be worth upgrading it to be in a warning box though.

@tarbaII
Copy link
Author

tarbaII commented Feb 9, 2022

Yeah you make a good point. I guess the only people who would get screwed with telling people to set it to true would be people who don't fully read everything 😛 I'll update the install process page as well as changing what we tell people to set HideAuxiliary to.

@tarbaII
Copy link
Author

tarbaII commented Feb 11, 2022

Should be good unless you have an issue with the new Comment section I wrote.

@tarbaII tarbaII marked this pull request as draft February 22, 2022 13:17
@tarbaII
Copy link
Author

tarbaII commented Feb 22, 2022

Converting to a draft until #342 and #343 are merged. Will update this PR with changes for coffee-lake.md and alder-lake.md afterwards.

dhinakg added a commit that referenced this pull request Jul 28, 2022
@dhinakg dhinakg closed this Aug 2, 2022
@tarbaII tarbaII deleted the hideauxiliary branch August 3, 2022 01:14
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.

3 participants