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

Cannot compile bacon - missing sys library alsa.pc #319

Closed
ondrej-ivanko opened this issue Jan 28, 2025 · 4 comments
Closed

Cannot compile bacon - missing sys library alsa.pc #319

ondrej-ivanko opened this issue Jan 28, 2025 · 4 comments

Comments

@ondrej-ivanko
Copy link

Hi,

I cannot seem to compile bacon, since the latest release v3.9.0. I'm using cargo install --locked bacon command and it fails on missing system library alsa. I have alsa installed, just not the alsa.pc file. $(which alsactl) returns cli.

The compilation log is part of this issue.

One question I have is why does bacon have to use alsa sound library? It's just linter/formatter no?

compilationlog.txt

OS specs:
OS: Fedora Linux 40 (Workstation Edition) x86_64
Kernel: Linux 6.12.9-100.fc40.x86_64
DE: GNOME
WM: Mutter (X11)
CPU: 12th Gen Intel(R) Core(TM) i7-12700H (20) @ 4.70 GHz
GPU: NVIDIA GeForce RTX 3060 Mobile / Max-Q [Discrete]

@Canop
Copy link
Owner

Canop commented Jan 28, 2025

This was also mentioned here, and a survey was opened to ask whether the "sound" feature should be enabled by default or not, but unfortunately there's a visibility problem for such discussion so it stalled.

With your report, I think I'll make the feature disabled by default.

@Canop
Copy link
Owner

Canop commented Jan 28, 2025

Don't close the issue but an immediate solution for you will be to compile bacon with --no-default-features.

@ondrej-ivanko
Copy link
Author

This was also mentioned here, and a survey was opened to ask whether the "sound" feature should be enabled by default or not, but unfortunately there's a visibility problem for such discussion so it stalled.

With your report, I think I'll make the feature disabled by default.

Ah cool, should have checked the closed issues before :D The compilation works now. TBH, I have nver ever used the sound feature, so disabling it by default is good idea.

@Canop Canop closed this as completed in 108520b Jan 28, 2025
@Canop
Copy link
Owner

Canop commented Jan 28, 2025

3.9.1 released

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

No branches or pull requests

2 participants