Virtual disk has no icon

[dsjonny]

If I create a virtual disk using diskimage, the disk has no icon in the mount menu or in the DriveSetup.

After I mount it, it has an icon in the Tracker, but in the mount menu there is still no icon.

I use the latest nightly image: 45092

[anevilyak]

Oops, didn't see the DriveSetup part. Most likely a problem in the mime database then.

[dsjonny]

Using hrev46447 nightly anyboot still exists.

[umccullough]

Please don't go through every bug you own and confirm that it's not fixed yet... unless you plan to provide additional details which help narrow down the issue, all it does is spams everyone with useless ticket updates.

[korli]

Replying to umccullough:

Please don't go through every bug you own and confirm that it's not fixed yet... unless you plan to provide additional details which help narrow down the issue, all it does is spams everyone with useless ticket updates.

I find it can be helpful, especially for hardware related bugs and one time a year is fine btw :)

[dsjonny]

Replying to korli:

Replying to umccullough:

Please don't go through every bug you own and confirm that it's not fixed yet... unless you plan to provide additional details which help narrow down the issue, all it does is spams everyone with useless ticket updates.

I find it can be helpful, especially for hardware related bugs and one time a year is fine btw :)

I'm sorry for that. I didn't want to spam, I just saw that, I have many old tickets without any update since months. Usually I check the most of my (bug) tickets with the latest nightly images because there are many commits to the source. So, I don't know if there is some related to these tickets (I don't understand many). And many times I see "Is this still exists?" questions in the tickets, so that's why I updated sometimes the tickets. Next time I will only update if there is some "really useful" info.

[pulkomandy]

How is this supposed to work? For other devices the icon is retrieved using an ioctl and the drivers in add-ons/kernel/drivers/disk provide an implementation with their respective icon(s). However the disk image file driver seems to be implemented directly in the disk device manager and I see no provision for ioctls there?

[axeld]

You may want to look at ​http://cgit.haiku-os.org/haiku/tree/src/system/kernel/device_manager/FileDevice.cpp#n248.

The disk device manager just uses devfs_publish_file_device() which gets served by this class.

[pulkomandy]

I added B_GET_VECTOR_ICON there but it didn't work. Still no icon in DriveSetup nor Tracker mount menu. Attaching the patch if someone can spot what Idid wrong.

[axeld]

Does it work now, since you actually committed it already? In any case, I would check that the hooks are really called, otherwise, I couldn't spot anything wrong.

[pulkomandy]

No, it doesn't work yet and I mixed up things in git again, I didn't mean to push these changes yet.

[pulkomandy]

So, after further investigation:

• Sending the B_GET_DEVICE_ICON ioctl works.
• Getting the icon using get_device_icon also works.
• When the volume is mounted, BVolume::GetIcon returns the standard "disk" icon for it. This is what is shown in Tracker as well. When the volume is not mounted, BVolumeRoster does not enumerate it.

I get the correct icon in DriveSetup now, but I get the standard "disk" icon in Tracker mount menu. I also see that DriveSetup "mounted on" and "file system" columns are not filled properly after initializing the volume. So there are more missing links here.

[dsjonny]

I think ticket can close.

[nielx]

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone