Skip to content

Conversation

@MrMeshok
Copy link

@MrMeshok MrMeshok commented Aug 12, 2025

In Debian we got this warning

There are issues with the multiarch metadata for this package.
mangohud conflicts on /usr/share/vulkan/implicit_layer.d/MangoHud.arm.json on armel <-> armhf

If you try to install both armel and armhf packages you will get this error

trying to overwrite shared '/usr/share/vulkan/implicit_layer.d/MangoHud.arm.json', which is different from other instances of package mangohud:armhf

After this change this will work.
To test this change I compiled for amd64, i386, armel, armhf and got this results

arch before after
amd64 MangoHud.x86_64.json MangoHud.x86_64.json
i386 MangoHud.x86.json MangoHud.x86.json
armel MangoHud.arm.json MangoHud.armel.json
armhf MangoHud.arm.json MangoHud.arm7hlf.json

@MrMeshok
Copy link
Author

Looking at i386 results now I realized, that install/uninstall scripts should be adjusted.
I can do this, but maybe this should be Debian only patch 🤔

@flightlessmango
Copy link
Owner

I was racking my brain trying to think of how this could possibly mess with downstream, but I think it should be fine?
I'm curious how other projects tend to handle this

@MrMeshok
Copy link
Author

rm -fv "/usr/share/vulkan/implicit_layer.d/MangoHud.x86.json"

/usr/bin/install -Dvm644 ./usr/share/vulkan/implicit_layer.d/MangoHud.x86.json /usr/share/vulkan/implicit_layer.d/MangoHud.x86.json

This lines would need to be adjusted because now this file would be named "MangoHud.i386.json" and old x86 name should be kept in uninstall so new install/uninstall script can uninstall older version.

I will look into how other distros handle this, but they probably don't have this problem because not many build for armel and armhf

@MrMeshok MrMeshok changed the title Use cpu instead of cpu_family for mangohud.json.in Fix file collision on armel and armhf Sep 1, 2025
@MrMeshok
Copy link
Author

MrMeshok commented Sep 1, 2025

I found very few distros that supports armel and didn't based on Debian and they all have the same problem.
But I thought about it more and rewritten this PR to only affect arm 32 bit architectures, so this change will have very small footprint and didn't need any changes for install/uninstall script

@MrMeshok
Copy link
Author

MrMeshok commented Sep 3, 2025

Also found a similar issue for mesa
https://gitlab.freedesktop.org/mesa/mesa/-/issues/13745

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.

2 participants