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

Difficult to rename entry with empty label #4

Closed
davidmegginson opened this issue Aug 2, 2021 · 4 comments
Closed

Difficult to rename entry with empty label #4

davidmegginson opened this issue Aug 2, 2021 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@davidmegginson
Copy link

For some reason, Grub created an EFI entry with an empty label, and I found it challenging to rename. After hacking on the script for a while, I realised that I needed to use "Unknown Device: " (with the trailing space) to match the regex output.

Suggestion: if let the user include the empty string for the first argument if there's an entry level provided as a third argument, and skip the string matching.

Great tool, BTW -- thanks!

@s-n-ushakov
Copy link
Owner

Hi David, and thank you for the feedback with an interesting case :)

I have implemented your suggestion with a minor amendment, so that the missing / wildlcard label placeholder is '*' rather than empty string as first command line argument. Are you in a position to test the release candidate script version that is attached here?

Also: are you in a position to share your OS or other bootstrap item name, just for the records? :)

rename-efi-entry.bash.20210803-rc1.zip

@davidmegginson
Copy link
Author

Thanks! Unfortunately, while your script worked as advertised (once I figured out the name issue), it turns out my laptop (Acer Nitro 5 running Ubuntu 21.04) doesn't preserve the UEFI changes on reboot, so I can't do an end-to-end test, but I could still confirm that it temporarily changes the UEFI records if that's helpful.

@s-n-ushakov
Copy link
Owner

Hi David again, and thank you for all the clarifications.

I have issued a new release that hopefully addresses and fixes your issue.

You are most welcome to mark the issue as closed if everything looks good to you now... :)

@s-n-ushakov s-n-ushakov added the enhancement New feature or request label Aug 7, 2021
@s-n-ushakov s-n-ushakov self-assigned this Aug 7, 2021
@s-n-ushakov
Copy link
Owner

Closed, as a fix was supplied, and no further comments followed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants