dakanji / RefindPlus

A Boot Manager for Mac and PC
GNU General Public License v3.0
326 stars 68 forks source link

Incorrect icons shown for Elementary OS partitions #74

Closed MarioG-X closed 3 years ago

MarioG-X commented 3 years ago

Describe the Bug: This is a minor issue but thought since original refind displays the correct icons so should RP. There are 2 linux Opensuse BTRFS and 2 Linux elementary OS EXT4 partitions. Also 5 macOS volumes in APFS. All display the correct icons except the 2 Elementary OS partitions.

To Reproduce: Boot RP.

Expected Behaviour: All menu items should display the same as Original Refind does, using the correct icons for Elementary OS.

Environment: RefindPlus Version : v0.13.2.AH beta, AG cannot test because it does not display linux partitions. Hardware Computer : Mac mini 2014, Macmini7,1 with internal an SSD and hard drive (originally fusion drive) Motherboard : Dual-Core Intel Core i7 GPU Model : Intel Iris Affected OS : All macOS, Opensuse Linux, Elementary OS Linux

Log Files 21r18u2006.log

Screenshots RP menu, notice the penguin and dimmed elementary os icon: IMG_1984

Original Refind menu, both elementary os icons blue and bright white: IMG_1976

Additional Context: I found .VolumeIcon.icns and .VolumeIcon.png in "/" (top level root). These display in the original refind menu. I found it odd that although both elementary partitions are identical (one cloned from the other), RP displays a penguin on one and a dimmed elementary icon on the other.

dakanji commented 3 years ago

Please use a released version for this issue. The test version of 0.13.2.AH you have access to has things disabled in terms of icons as was explained and is not a valid basis for this report.

MarioG-X commented 3 years ago

As I mentioned above I used the AG version first, it does not display linux partitions, that's why we just worked on AH. Here I tried the AG debug version and it crashed because there was no config.conf: 21r18v4659.log

Then I added config.conf and it did not crash, but it did not display linux partitions. No sense working on AG the it has so many linux issues on my machine. Here's the log anyway: 21r18v5059.log

This is not important so it can wait until AH is further along it you want to look into it.

dakanji commented 3 years ago

Basically means the report is premature at this point as you have not yet actually been able to definitely confirm that the issue exists since AG does not load Linux and the AH you have is not suitable.

Will close as invalid. You can test when possible to test

MarioG-X commented 3 years ago

OK I get it now. No reports of problems in beta, puzzles me because if someone tests beta there is no way to report a new issue. So we wait until you release AH, just thought you may want to prevent a known issue going into AH when it's released.

dakanji commented 3 years ago

OK I get it now.

Not quite. What you had is not a beta but a file created to test a specific issue which had some distracting items (with respect to the desired test) disabled.

Specifically, this included the loading of some types of icons.

  1. Explained here: https://github.com/dakanji/RefindPlus/issues/72#issuecomment-918924613
  2. And here: https://github.com/dakanji/RefindPlus/issues/72#issuecomment-921044730

When that item is closed, you can build the code base to get a proper beta that will load Llnux and see whether this issue actually exists.

dakanji commented 3 years ago

The codebase has been updated with the Linux fixes.

I believe you may have Homebrew installed. If correct, you can build on the current code to verify the issue by following the instructions here: https://github.com/dakanji/RefindPlus#roll-your-own

MarioG-X commented 3 years ago

Luckily I have an Xcode "junk" partition to play with, I followed your instructions and built GOP.

RP now displays the same elementary OS icons same as original refind. I booted Opensuse, elementary OS and macOS fine. Great, no work needed on this! If you're interested here is a log: 21r19s5637.log

Don't know if you're interested: I noticed the MBR line for the second drive line was removed, it now has nothing before it instead of showing partition type GUID, seems not to harm anything, just cosmetic: 3:102 0:007 00000000-0000-0000-0000-000000000000 : 00000000-0000-0000-0000-000000000000 : Unknown : : Unknown Volume 3:109 0:007 00000000-0000-0000-0000-000000000000 : 00000000-0000-0000-0000-000000000000 : Whole Disk : : Whole Disk Volume

You have a typo on https://github.com/dakanji/RefindPlus/blob/GOPFix/BUILDING.md first sentence has buidling, should be building.

Crossing fingers for SyncAPFS enhancement!

Thank you for RP and all your hard work!

dakanji commented 3 years ago

I noticed the MBR line for the second drive line was removed

That is just down to the logging level

You have a typo on https://github.com/dakanji/RefindPlus/blob/GOPFix/BUILDING.md first sentence has buidling, should be building.

Will fix. Thanks

BTW, I force pushed a change to the last commit shortly after the original commit. Shouldn't really do that but the repo is not busy and not likely to affect anyone but guess what? Seems you forked the repo between the two pushes and now your fork is out of sync.

Follow the instructions here to bring it back in line:- https://github.com/dakanji/RefindPlus/blob/GOPFix/BUILDING.md#syncing-your-repositories-with-source-repositories

MarioG-X commented 3 years ago

Thanks for the info but I got this message and will forget about syncing for now, I will set it up a token when I really need it. I can always rebuild RP from scratch if needed (I think) but not likely until SyncAPFS since that was the whole point of converting: remote: Support for password authentication was removed on August 13, 2021. Please use a personal access token instead.

BTW, I noticed this sentence in the .md, since I verified on Catalina you may want to change it from: These steps have been verified on Mac OS v10.12 (Sierra) and Mac OS v10.14 (Mojave) and should also work on Mac OS v10.13 (High Sierra) and Mac OS v10.15 (Catalina).

TO: These steps have been verified on macOS v10.12 (Sierra), macOS v10.14 (Mojave) and macOS v10.15 (Catalina). They should also work on macOS v10.13 (High Sierra), macOS v11 (Big Sur), and macOS v12 (Monterey).

Having said that you may want to remove v12 and v13 because I have found once Apple drops support so does HB and you can no longer build HB packages (most anyway) from scratch.