Skip to content

Conversation

@wolfpackmars2
Copy link
Contributor

This edit aligns the thumbnailer feature with PR #1673. Specifically, it modifies the solvespace.thumbnail file to point to the installed executable. This means that if a user installs solvespace to ~/.local, the thumbnailer will work for that user.

This is necessary because ~/.local/bin is not normally included in the default PATH environment variable.

@phkahler
Copy link
Member

phkahler commented Feb 4, 2026

I've been thinking about this thumbnailer stuff and I don't think we want it by default for builds - only for "make install" and then preferably as an option. I still think it's more of a packaging thing. I just don't like the idea of saving a file from email or off the internet and having software open it by default.

@ruevs what do you think?

@ruevs
Copy link
Member

ruevs commented Feb 4, 2026

This is quite far from my daily computer use since I do not use (desktop) Linux regularly.

But it definitely sounds like a distro/packaging problem.

Whether the default make install without sudo should be able to set up a "thumbnailer" I have no opinion whatsoever on... It it were my personal machine I couldn't care less - I would never run make install if compiling from source.

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.

3 participants