Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lavalink: init at 4.0.8; nixos/lavalink: init #344687

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

nanoyaki
Copy link

Description of changes

A standalone audio sending node based on Lavaplayer and Koe.

This adds the package for Lavalink and a corresponding nixos module.

I tested the program with and without plugins and several different configurations on my local computer, and a remote nixos machine.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: documentation 8.has: changelog 8.has: module (update) This PR changes an existing module in `nixos/` 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Sep 26, 2024
@NixOSInfra NixOSInfra added the 12. first-time contribution This PR is the author's first one; please be gentle! label Sep 26, 2024
@nanoyaki nanoyaki force-pushed the lavalink branch 2 times, most recently from ba653a0 to cbb5a8d Compare September 26, 2024 17:07
@ofborg ofborg bot added 8.has: package (new) This PR adds a new package 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 labels Sep 26, 2024
Copy link
Member

@Bot-wxt1221 Bot-wxt1221 left a comment

Choose a reason for hiding this comment

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

Rename your title to lavalink: init at 4.0.8; nixos/lavalink: init

nixos/modules/services/audio/lavalink.nix Outdated Show resolved Hide resolved
pkgs/by-name/la/lavalink/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/la/lavalink/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/la/lavalink/package.nix Outdated Show resolved Hide resolved
@Bot-wxt1221
Copy link
Member

I haven't check the nixos module.

@nanoyaki nanoyaki changed the title nixos/lavalink: init at 4.0.8 lavalink: init at 4.0.8; nixos/lavalink: init Sep 27, 2024
@nanoyaki nanoyaki force-pushed the lavalink branch 2 times, most recently from 1b5659e to 2a04599 Compare September 27, 2024 10:38
@nanoyaki
Copy link
Author

I addressed all of the above in the recent pushes

];

buildInputs = [
systemd
Copy link
Member

Choose a reason for hiding this comment

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

What is this for?

Copy link
Author

Choose a reason for hiding this comment

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

That was due to this warning during runtime:

o.s.os.linux.LinuxOperatingSystem : Did not find udev library in operating system. Some features may not work.

I just tested it again and it doesn't seem to have fixed the issue.

I'm not sure how to solve this issue.
I tried installing libudev using

(systemd.override {
  buildLibsOnly = true;
})

and systemdLibs but neither seem to work.

Copy link
Member

@Bot-wxt1221 Bot-wxt1221 left a comment

Choose a reason for hiding this comment

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

I suggest we can build from source.

pkgs/by-name/la/lavalink/package.nix Outdated Show resolved Hide resolved
@h7x4 h7x4 added 8.has: module (new) This PR adds a module in `nixos/` 8.has: tests This PR has tests labels Sep 28, 2024
@nanoyaki
Copy link
Author

nanoyaki commented Nov 7, 2024

I suggest we can build from source.

I've seen other packages with the same approach, so I'm not sure to why?

@Bot-wxt1221
Copy link
Member

@nanoyaki According to manual, we'd better build from source. But now it's ok if you don't want to do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog 8.has: documentation 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 8.has: module (new) This PR adds a module in `nixos/` 8.has: module (update) This PR changes an existing module in `nixos/` 8.has: package (new) This PR adds a new package 8.has: tests This PR has tests 10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 11.by: package-maintainer This PR was created by the maintainer of the package it changes 12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants