Skip to content

Conversation

@eth-p
Copy link

@eth-p eth-p commented Dec 18, 2025

Great work on the vinyl theme! I found it a few months ago, and I've been daily driving it since.

I use NixOS for my distro, and due to the way the package manager works on the system, I had to package it for nix before I could install it. It takes a bit of trial and error to do, and it's not as simple as the other available installation methods for vinyl. I felt it might help encourage other NixOS users to try the theme if it was easier to install, so I figured I would create this PR to help with that.

Ideally, I would have just added the package on the official NixOS nixpkgs repo, but their package guidelines suggest that vinyl-theme is just a little bit too niche right now to be accepted:

Before adding a new package, please consider the following questions:

  • [...]
  • How realistic is it that it will be used by other people? It's good that nixpkgs caters to various niches, but if it's a niche of 5 people it's probably too small. A good estimate is checking upstream issues and pull requests, or other software repositories.

As a compromise, I added the package info (flake.nix, package.nix) directly to the repo here, similar to how other projects like vicinae and kwin-effects-better-blur-dx do it.

If you feel that it's too invasive to include the nix packaging within the vinyl-theme repo itself, no worries, just close this PR.

The `start-here-kde.svg` icon under `icons/src/places/scalable`
does not exist, and it's used as the target for the
`start-here-kde-plasma.svg` symlink.

Remove it from `links.txt` so the built package does not contain
any broken symlinks.
This allows users on NixOS to install the theme directly from a
tagged release of the git repo.
@christiantosta
Copy link
Contributor

Please, submit this PR to the devel branch. Them main one is reserved only for releases.

@eth-p eth-p changed the base branch from main to devel December 19, 2025 03:35
@eth-p
Copy link
Author

eth-p commented Dec 19, 2025

My apologies. That's now fixed, and the PR is changed to the devel branch.

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