Skip to content

New package: navidrome 0.61.1#59763

Open
micro-80 wants to merge 1 commit intovoid-linux:masterfrom
micro-80:navidrome
Open

New package: navidrome 0.61.1#59763
micro-80 wants to merge 1 commit intovoid-linux:masterfrom
micro-80:navidrome

Conversation

@micro-80
Copy link
Copy Markdown

@micro-80 micro-80 commented Apr 5, 2026

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, x86_64 glibc + aarch64 musl

@tranzystorekk tranzystorekk added the new-package This PR adds a new package label Apr 5, 2026
Comment on lines +7 to +8
export ND_CONFIGFILE=/var/lib/navidrome/navidrome.toml
exec chpst -u _navidrome:_navidrome navidrome
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export ND_CONFIGFILE=/var/lib/navidrome/navidrome.toml
exec chpst -u _navidrome:_navidrome navidrome
exec chpst -u _navidrome:_navidrome navidrome ${OPTS:- --configfile "/etc/navidrome/navidrome.toml"}

Its always better to match upstream default locations and services (https://github.com/navidrome/navidrome/blob/4570dec675f904aa0cecd09b2c842685b762527e/contrib/navidrome.service#L14). Config files belong in /etc.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks - changed

post_install() {
vlicense LICENSE
vmkdir var/lib/navidrome
vcopy release/linux/navidrome.toml var/lib/navidrome/navidrome.toml
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
vcopy release/linux/navidrome.toml var/lib/navidrome/navidrome.toml
vinstall release/linux/navidrome.toml 644 etc/navidrome

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks - changed

}

post_install() {
vlicense LICENSE
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If this is GPL, the license file doesn't need to be installed.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks - changed

revision=1
build_style=go
go_import_path=github.com/navidrome/navidrome
go_ldflags=" -X github.com/navidrome/navidrome/consts.gitSha=e7c7cba87374ebe1bace57271bc5e8cf731b7a6e -X github.com/navidrome/navidrome/consts.gitTag=v${version}"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Things like the commit will end up being outdated, either don't set it or set it to the version.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks - changed

go_import_path=github.com/navidrome/navidrome
go_ldflags=" -X github.com/navidrome/navidrome/consts.gitSha=e7c7cba87374ebe1bace57271bc5e8cf731b7a6e -X github.com/navidrome/navidrome/consts.gitTag=v${version}"
go_build_tags="netgo,sqlite_fts5"
hostmakedepends="nodejs pkg-config sqlite-devel taglib-devel"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sqlite-devel taglib-devel sound like they belong in makedepends.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks - changed

@micro-80
Copy link
Copy Markdown
Author

micro-80 commented Apr 9, 2026

Ran new changes both on x86_64 glibc and aarch64 musl, works as expected.

@@ -0,0 +1,7 @@
#!/usr/bin/env sh
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's probably safer to just go with the #!/bin/sh shebang. You can inspect other run files in Void for that.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks, changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-package This PR adds a new package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants