Page MenuHome

Wrong icon for Movie Clip Editor
Closed, ResolvedPublicBUG

Description

System Information
Operating system: Linux-5.10.0-1-amd64-x86_64-with-debian-bullseye-sid 64 Bits
Graphics card: Quadro RTX 6000/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 460.27.04

Blender Version
Broken: version: 2.92.0 Beta, branch: master, commit date: 2021-01-13 23:55, hash: rB5804bf25bdfe
Worked: 2.79, all local builds (all the way up to the latest master)

Short description of error

Movie clip editor is using wrong icon in the space menu.

Exact steps for others to reproduce the error

  • Download Blender from buildbot
  • Open space menu
  • Observe icon of the clip editor (it is supposed to be shot clicky thing, not a crosshair)

It is caused by multiple .dat files in the icons folder defining image for the clip editor's icon. The files are traversed in an undefined order (since readdir does not perform any sorting), this is why buildbot behaves different from local builds.

The check against icons conflicts was implemented in D9715, but the check can not go in since it will break builds, because icons are already in an inconsistent state.

Rebuilding icons also turns out to be not so trivial, as there seems to be inconsistency between icons defined in the UI_icons.h header and editors/datafiles/CMakeLists.txt files.

Event Timeline

Philipp Oeser (lichtwerk) changed the task status from Needs Triage to Confirmed.Jan 19 2021, 5:42 PM
Julian Eisel (Severin) changed the subtype of this task from "Report" to "Bug".Jan 25 2021, 11:27 AM

Could be considered a todo since it's an issue in the resource files more than a code issue. But since there are differences in platforms, I'll mark this as actual bug.

Is there any progress here? Not sure why such a simple-sounding task takes months to be addressed (it was verbally communicated before D9715).

It's not entirely clear to me what needs to be done. So there are two overlapping icons in a cell in the SVG? Plus, the icon enum is out of sync with the SVG?

I also couldn't run the icon update script for a while until c549d736cff0, not sure if that's related to what Dalai reported.

If something is not clear is the best to ask for clarification, than to wait for, not sure what ;)

In practical terms what I see needs to be done is to remove all existing .dat files from blender_icons{16,32}, regenerate the new ones using make icons. This will ensure that the generated .dat files are in perfect sync with what Blender expects, solving the conflicting .dat files.

The steps above will work, however, when you'll attempt to do a fresh build you'll see something like this:

make[2]: *** No rule to make target '/home/sergey/Developer/blender/blender/release/datafiles/blender_icons16/icon16_clip.dat', needed by 'release/datafiles/blender_icons16.png'.  Stop.

So while it all sounds simple, the issue goes deeper into some fragility of the way how things are set up.