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

build: add debian packaging files #411

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

Conversation

drlkf
Copy link

@drlkf drlkf commented Jul 19, 2024

This pull request introduces debian packaging files. There are 2 most important items:

  1. Icons and server download are added to the cmake build
  2. An edge-case is introduced in the server load path to avoid putting a server.js in /usr/bin on debian package installs

I've added the instructions to build the package in the DEBIAN.md file, let me know if something is missing or unclear.

cheers

@drlkf
Copy link
Author

drlkf commented Dec 23, 2024

Any chance we could get some attention to this?

@kzdnk
Copy link

kzdnk commented Dec 27, 2024

I know very little of debian packaging, so I can't talk about correctness, but I tested this on bookworm and it seems to be working fine.

One thing I had to do, that is not mentioned, is to enable the non-free archive for libfdk-aac2.

@kzdnk
Copy link

kzdnk commented Dec 27, 2024

Also, desktop entry is displayed with the default icon.

@drlkf
Copy link
Author

drlkf commented Jan 1, 2025

Both should be fixed now

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