-
Notifications
You must be signed in to change notification settings - Fork 233
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
Package introduced into Debian Unstable #373
Comments
this sounds awesome @LasseF-H ! an open issue might be a good way to track the process, so I will just leave this here. Please let me know if there are any problems with the building/installing process. There is #200 and #144 to consider. btw https://karlstav.github.io/cava/ is no longer maintained. Just use the github front page as "homepage" |
updated ubuntu/debian package information #373
Great! I'll let you know if I have any problems. I have applied a couple of minor patches to the packaged version, you can look at them here, if you want to consider upstreaming them. I think one of them accidentally got a wrong/weird description though. I'll fix the homepage issue after the next point release here. It's a minor issue after all. The package has transitioned to Debian testing. So it'll most likely make it into Ubuntu 20.10 later this year, and Debian 11, whenever that ends up releasing.
I agree, communication between upstream and the people packaging is beneficial for all parties, and this will do fine. I'll let you know if I have any packaging related trouble that makes sense fixing at the source. |
hi @LasseF-H I tested the package in Ubuntu 20.10 and I noticed that it did not install the example config file. Is it possible to make it happen? |
The example file should be included, these are the files that are included in the package:
As you can see the default config file is included, and I can see that cava creates an empty file in |
yes that is correct @LasseF-H, cava just creates an empty file if no file exists. When building from source it's actually the autogen.sh script that copies the example config to the home directory. But this is not ideal either. I guess the best approach would be to populate the config file (if it doesn't exist) with the stuff from the example file at runtime. That would mean not having the example config as a separate text file at all, but putting it in the code somewhere. |
Following the XDG specification, and moving the example config to $sysconfdir/xdg/cava might also be an option. As in, use the config in /etc/xdg/cava when a user specific config file doesn’t exist. |
create a hex dump of example config file with xxd in autogenerate.sh use this to create a default config file in home dir on first launch if no file is found. this only applies if the path to config file is not specified
Hi, I have been working on packaging Cava for Debian officially for a while, and the package got accepted into Debian unstable last week. You can have a look at the package here. This means that it should move into Debian testing and stable over time, and will therefore also move into the repositories of Ubuntu and its derivatives. I plan on maintaining the package for the foreseeable future.
I don't know if this is the place to inform you, or if I should have sent an email. Either way, I was hoping it could be added to the list of distro packages in the README.md.
The text was updated successfully, but these errors were encountered: