Skip to content

overlay hunt: remove unused libfm overlay#1845

Merged
brianmcgillion merged 1 commit intotiiuae:mainfrom
vadika:overlay-hunt-libfm
Mar 23, 2026
Merged

overlay hunt: remove unused libfm overlay#1845
brianmcgillion merged 1 commit intotiiuae:mainfrom
vadika:overlay-hunt-libfm

Conversation

@vadika
Copy link
Copy Markdown
Contributor

@vadika vadika commented Mar 22, 2026

Summary

  • Remove the libfm overlay — it patches a library that nothing in Ghaf depends on
  • libfm is the core library of PCManFM file manager, but Ghaf uses COSMIC desktop with cosmic-files which has no libfm dependency
  • Verified: nix why-depends nixpkgs#cosmic-files nixpkgs#libfm → "does not depend on"
  • Zero references to pcmanfm or libfm anywhere in the Ghaf codebase
  • As a library overlay, this was needlessly changing the derivation hash of any transitive dependent, forcing rebuilds from source

Test plan

  • Verify nix build .#lenovo-x1-carbon-gen11-debug succeeds (libfm not in closure)
  • Verify COSMIC file manager still works (it uses cosmic-files, not pcmanfm/libfm)

libfm is the core library of PCManFM file manager, but Ghaf switched
to COSMIC desktop which uses cosmic-files (no libfm dependency).

Verified with `nix why-depends`: cosmic-files does not depend on libfm,
and there are zero references to pcmanfm or libfm in the Ghaf codebase.

This overlay was needlessly patching a library in the global pkgs set.
As a library, any package that transitively depends on libfm would get
a new derivation hash and rebuild from source instead of using the
binary cache.

Signed-off-by: vadik likholetov <vadikas@gmail.com>
Copy link
Copy Markdown
Collaborator

@kajusnau kajusnau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to remove IMO

Copy link
Copy Markdown
Contributor

@nesteroff nesteroff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe support for PCManFM was removed in 9997d21 so the overlay is no longer needed. It was used to make the file manager automatically refresh folder contents when they were changed from a VM.

@brianmcgillion brianmcgillion merged commit 33fd176 into tiiuae:main Mar 23, 2026
31 checks passed
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.

4 participants